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

Fixing energy ordering #757

Merged
merged 2 commits into from Sep 3, 2015
Merged

Fixing energy ordering #757

merged 2 commits into from Sep 3, 2015

Conversation

elliottbiondo
Copy link
Member

In PARTISN, energy-dependent quantities are ordered from high-energy to low-energy. I could not find explicit documentation of this, but I was about to confirm this by experiment. I ran transport problems with source intensity only in the last energy group (i.e. the lowest energy group in PARTISN), and confirmed that fluxes in all other groups were zero. I also did the reverse --- only flux in the first group --- and confirmed that particles were indeed downscattering in the other groups.

In PyNE we have been using the opposite convention --- low-energy to high-energy --- consistent with MCNP. The PR fixes PARTISN functionality to be consistent with the rest of PyNE. For unit tests, I was able to simply reverse the order of input spectra, minimizing what needed to be changed.

@scopatz scopatz added the bug label Sep 3, 2015
@scopatz scopatz added this to the v0.6 milestone Sep 3, 2015
@scopatz
Copy link
Member

scopatz commented Sep 3, 2015

Thanks @elliottbiondo!

scopatz added a commit that referenced this pull request Sep 3, 2015
@scopatz scopatz merged commit 3e4f248 into pyne:develop Sep 3, 2015
@elliottbiondo elliottbiondo deleted the low_to_high branch September 5, 2015 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants