-
Notifications
You must be signed in to change notification settings - Fork 492
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 decay only nuclides to be specified in Material
for a depletion simulation
#2616
Allow decay only nuclides to be specified in Material
for a depletion simulation
#2616
Conversation
materials.xml
for a depletion simulation
materials.xml
for a depletion simulationMaterial
for a depletion simulation
Co-authored-by: Jonathan Shimwell <drshimwell@gmail.com>
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.
@yardasol Thanks so much for this improvement. You'll be happy to know that a request for this very feature came in today during the OpenMC monthly meeting! Sorry it took me a long time to look it over properly.
Co-authored-by: Paul Romano <paul.k.romano@gmail.com>
…s in coupled_operator.py
@paulromano I implemented you suggested change. Let me know what you think. |
Looks like the CI is failing. This is due to my new tests. I suspect this has to do with the most recent commit based on @paulromano's suggestion, as now the decay-only nuclides are passed to the C API which would cause the simulation to crash. |
60d7b61
to
6f0969d
Compare
@paulromano I was incorrect above, the issue was simpler than that. |
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 the updates @yardasol! Approved ✔️
…on simulation (openmc-dev#2616) Co-authored-by: Jonathan Shimwell <drshimwell@gmail.com> Co-authored-by: Paul Romano <paul.k.romano@gmail.com>
Description
This PR enables users to specify decay-only nuclides in a
Material
when running transport-coupled depletion simulations. This could be useful for coupling with external interface codes, as well as for simulating scenarios where there may be buildup of these decay-only nuclides.Checklist
[ ] I have run clang-format on any C++ source files (if applicable)