Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update 1_find_dpa.ipynb #270

Closed
wants to merge 1 commit into from

Conversation

Oliverator
Copy link
Contributor

As suggested, I have extended the explanation of DPA calculation. I hope this is useful and clarifying for new users. NRT and arc models for DPA are something the new generation of neutroneers need to learn about :)

As suggested, I have extended the explanation of DPA calculation. I hope this is useful and clarifying for new users. NRT and arc models for DPA are something the new generation of neutroneers need to learn about :)
@shimwell shimwell self-assigned this Feb 14, 2024
@shimwell
Copy link
Member

Thanks for the PR from a quick skim it looks good.

It is a bit hard to see the changes when the code blcok outputs have also been committed.

The code block outputs swamp the git diff

Please could the outputs be cleared.

Here is the button in vs code and jupyterlab
clear_vs
clear_jl

@Oliverator
Copy link
Contributor Author

Hi John,

Sure, let me clear all the outputs. Thank you for your patience.

Cheers,
Mario

@shimwell
Copy link
Member

replaced by pr #271

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants