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

Adds distribution data for N-Body distribution #20

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HunterBelanger
Copy link

This PR addresses the issue that Wim mentioned in #16 , and adds the information about the representation of the distribution data given for the fraction of the max energy quantity (referred to as T in the document).

Originally, it was stated that the provided distribution was used for sampling T, but the equation provided was not completely consistent with this. I modified the distribution to be consistent with what the analytic form should be for PDF(T). If you want, I can change this be be the distribution for outgoing energy (as presented in the ENDF manual Equation 6.21).

The information about the distribution data can be found in acefc.f90 near line 7300, and acefc.f90 line 13265 in NJOY2016.

@whaeck
Copy link
Contributor

whaeck commented Apr 5, 2021

@HunterBelanger Let's wait to do this one until the CP production one has been finalised. I have some additional N-body phase space text to go with this as well.

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