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.
Description
This PR fixes an accuracy issue in the random ray solver stemming from PR #3299 affecting flux tallies in void cells with external source terms. More info is available in the writeup for issue #3315. This issue was originally missed in my testing as this error decayed away as the mesh resolution was refined, so seemed like a legitimate method bias for this use case rather than a logical error. With the new fix, a much coarser mesh is now sufficient to resolve flux tallies in void regions with external source terms.
Fixes #3315
Validation
To test this, I redid the analysis in #3299. This model is a modified 3D Kobayashi dog leg problem, where the fixed source area and duct work have been converted to pure void materials. Below is the flux tally in the void area that contains the external source term, using a 5x5x5 cm^3 mesh:
Formatting change
I also noticed there was an annoying loop index variable name that was causing a discrepancy in formatting between LLVM 15 vs. 19, so changed that variable name to a shorter one to stop triggering the condition that caused the formatting discrepancy.
Checklist