DAGMC solver object for neutronics running #4150
Draft
Conversation
This file contains hidden or 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
je-cook
force-pushed
the
je-cook/dagmc_solver
branch
from
4042462 to
3ab24cd
Compare
CoronelBuendia
force-pushed
the
matti/je-cook/process_develop_EUDEMO_LAR
branch
from
d1ff474 to
674a0ac
Compare
je-cook
force-pushed
the
matti/je-cook/process_develop_EUDEMO_LAR
branch
from
05afa54 to
6207a2f
Compare
Base automatically changed from
matti/je-cook/process_develop_EUDEMO_LAR
to
develop
November 18, 2025 15:53
je-cook
force-pushed
the
je-cook/dagmc_solver
branch
from
3ab24cd to
e1f7c13
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## develop #4150 +/- ##
===========================================
- Coverage 78.83% 77.45% -1.38%
===========================================
Files 244 244
Lines 29276 29418 +142
===========================================
- Hits 23080 22787 -293
- Misses 6196 6631 +435 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
je-cook
force-pushed
the
je-cook/dagmc_solver
branch
2 times, most recently
from
b74807d to
0243b1a
Compare
* fix upper port height * P_fus_DT and e_mult * ruff stuff * fix total heating and pass in pfus * Created the correct tally for total power. * Fixed a duplicate name * typo correction * Total power is now consistent with what you get when adding up vacuum vessel power + blanket power + divertor power. * Added the appropriate NWL tallies (#4040) * Renamed the Setup, Run, and Teardown classes to something more specific. * Changed the tallies to reflect the NWL properly. * Add comments/renamed functions/added arguments to make the code more readable. * Includes the previously-missed cells in the total power tallies now. * Bug fix * Added comment to clarify flux tally's 'mean' is actually volume integrated. * Added tallies and calculations to get the damage values in DPA/FPY as well. * Removed all traces of the word neutron wall load/NWL because its definition is too ambiguous/not useful. * removed the calculation about fw heating as well as we aren't using it. (Can be undone if necessary). * removed unused function --------- Co-authored-by: matti <[email protected]> * rebase pain * omg no deliberate nans * clean up diagnostic printing * remove duplicate function * ruff * avoid pandas warnings * ruff * change DPACoefficients into function * use * typing --------- Co-authored-by: Matti <[email protected]>
CoronelBuendia
force-pushed
the
je-cook/dagmc_solver
branch
from
906b3ff to
6118f6e
Compare
|
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.
Description
Add a dagmc solver to allow for continuous workflow mechanics
Interface Changes
Checklist
I confirm that I have completed the following checks:
pytest tests --reactorpre-commit run --from-ref develop --to-ref HEADsphinx-build -W documentation/source documentation/build