Add current tallies for bounded surfaces and cells#436
Conversation
…s to run with numba, modified surface tests to run with pytest.
…ion tests to run with numba, modified surface tests to run with pytest." This reverts commit f86e993.
|
Thanks, @massimolarsen. Apologies for just reviewing this. This is a very good feature. But I think we may need to reorganize it a bit to follow the tally design core idea: The currently existing proper tally types—surface, tracklength, and collision—correspond to the MC estimators used. The grouping is advantageous as it is directly related to the triggering event and scoring mechanism and thus the code implementation. The new tally type What do you think? |
Yea, let me fix that. I misread TallySurface as "Surface" meaning the physical surface, so I created a new TallyCell for the physical "Cell" region. In reality "Surface" is referring to the surface crossing estimator, which makes a lot more sense along with tracklength and collision. That is my bad. I'll rewrite the tally as the TallySurface with an extra cell filter. |
massimolarsen
left a comment
There was a problem hiding this comment.
Modified the previous TallyCell implementation to be a spatial filter within the existing surface crossing tally TallySurface. Updated testing and docs.
Summary of changes
This PR adds cell-based current tallies and bounded surface-current tallies, while expanding unit testing coverage for existing tally behavior.
The new cell tally supports:
net-currentcurrent-incurrent-outcurrent-inandcurrent-outare reported as positive partial currents, whilenet-currentcarries the sign convention.Types of changes
TALLY_CELLand current score constants forcurrent-inandcurrent-out.mcdc.Tally(cell=..., scores=[...])with current scores into a dedicatedTallyCellimplementation.y=[...]/z=[...]on aPlaneX.current-in/current-outscoresDeveloper Checklist
Associated Issues and PRs
Associated Developers