-
Notifications
You must be signed in to change notification settings - Fork 91
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
Add 1D Cylindrical Modeling to the XS Group Manager #1238
Changes from all commits
f9056a9
6e9ec8d
d5ca761
7b42b48
df32dec
601e6c1
241f817
2e9ea40
32e84ef
d222d4e
eabf578
a47d386
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -81,13 +81,6 @@ def __init__(self, r, cs): | |
|
||
# Set to True by default, but should be disabled when perturbed cross sections are generated. | ||
self._updateBlockNeutronVelocities = True | ||
# Geometry options available through the lattice physics interfaces | ||
self._ZERO_DIMENSIONAL_GEOM = "0D" | ||
self._ONE_DIMENSIONAL_GEOM = "1D" | ||
self._TWO_DIMENSIONAL_GEOM = "2D" | ||
self._SLAB_MODEL = " slab" | ||
self._CYLINDER_MODEL = " cylinder" | ||
self._HEX_MODEL = " hex" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Are these removed because they are unused? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I see they were used downstream but no longer are. Ok. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. They have been replaced by the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @sombrereau @drewj-usnctech tagging for visibility and a heads up |
||
self._burnupTolerance = self.cs[CONF_TOLERATE_BURNUP_CHANGE] | ||
self._oldXsIdsAndBurnup = {} | ||
self.executablePath = self._getExecutablePath() | ||
|
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.
Why the median block temperature?
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.
I'm not sure, this is what @jakehader originally implemented. We do have a
MedianBlockCollection
but it's based on the median burnup, not the median temperature. Average probably does make more sense for temperature.Edit: Although I guess it doesn't matter because then we just
deepcopy
it and then set the number densities based on the whole block collection. So it's probably not too important which block gets selected here.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.
Agreed. You could probably simplify
_selectCandidateBlock
to use just the first block ingetCandidateBlocks()
. For what it's worth, the unit test within this PR still works with that change.