-
Notifications
You must be signed in to change notification settings - Fork 97
Open
Labels
cleanupCode/comment cleanup: Low PriorityCode/comment cleanup: Low Priority
Description
@drewj-tp and @keckler suggest that we remove the nuclide flags feature from ARMI. They put forward the notion that Nuclide Flags do not add value for them, and frequently slow them down.
There are three options for every nuclide flag:
| nuclide flags: | |
| U234: | |
| burn: true | |
| xs: true | |
| expandTo: |
The question is: Is it possible these are only used by one downstream plugin and can be safely moved from ARMI to that project?
armi/armi/reactor/blueprints/isotopicOptions.py
Lines 82 to 91 in 0e830d2
| burn : bool | |
| True if this nuclide should be added to the burn chain. If True, all reachable nuclides via | |
| transmutation and decay must be included as well. | |
| xs : bool | |
| True if this nuclide should be included in the cross section libraries. Effectively, if this | |
| nuclide is in the problem at all, this should be true. | |
| expandTo : list of str, optional | |
| isotope nuclideNames to expand to. For example, if nuclideName is ``O`` then this could be | |
| ``["O16", "O17"]`` to expand it into those two isotopes (but not ``O18``). The nuclides will | |
| be scaled up uniformly to account for any missing natural nuclides. |
Second Question: Is it possible we could solve the user problem by just auto-defining defaults so the system never complains at you for missing nuclides in the list?
Metadata
Metadata
Assignees
Labels
cleanupCode/comment cleanup: Low PriorityCode/comment cleanup: Low Priority