fix #7279 full support for PNNL material database #7428
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- renamed stateful compute check_animation_dir() to verify_geometry()
e-carlin
requested changes
Jan 21, 2025
|
Good point - the PNNL references nuclides which aren't present in the older ENDFB-7.0 source. I've updated the examples to use the more recent version ENDFB-8.0. You should be able to "discard changes to example" and then use the material lookup. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added a type-ahead lookup list option when defining materials.
A condensed hdf5 file can be created from the full PNNL json file using pkcli.openmc.create_standard_materials_file()
The hdf5 file is stored in the cloudmc data repo sirepo-data-cloudmc.
During geometry initialization, the PNNL cache is checked and the file is retrieved from the data repo if needed.