-
Notifications
You must be signed in to change notification settings - Fork 500
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
Allow transfer rates with nuclides. #2564
Allow transfer rates with nuclides. #2564
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for improving this capability @yardasol!
Co-authored-by: Paul Romano <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me now! @church89 do you want to take a look at this before we merge?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @yardasol, looks good thanks. We discussed it in person and I have nothing to add beside this small docstring clarifications.
Co-authored-by: Lorenzo Chierici <[email protected]>
Description
This PR adds capabilities to enable
TransferRates
to work with specific nuclides. While transferring specific nuclides would most likely not occur in a real fuel processing system, it is a useful feature to have for comparison with other codes (i.e. Serpent2'srepr
card).Checklist