Repository navigation
Conversation
reactivity computed the Bosch-Hale thermal reactivity with er, which held reaction energies, instead of the reduced-mass energy m_r c². The correct values were already defined as mc2 but never used. Use mc2 in the formula, define the D-T value under the same name and drop the er lines. The new test pins 24 values of Table VIII of Bosch & Hale 1992, as reprinted in the erratum, Nucl. Fusion 33, 1919. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Member
jmcclena
marked this pull request as ready for review
October 2, 2026 21:14
Contributor
Author
|
Thanks for merging this so quickly, @jmcclena On #311: yes, I think the factor of 1/2 should be there. The Bosch-Hale is per pair, so for In nuclear.jl, D_D_to_He3_reactions and D_D_to_T_reactions return n_D^2 without the 1/2:
One thing worth a release note: this PR changes D+D->He3, D+D->T and D+He3->He4 by constant factors of |
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
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.
reactivitycomputes ⟨σv⟩ wither, which holds reaction energies (18.3e6, 4.03e6 and 0.82e6 eV) for D+He3→He4, D+D→T and D+D→He3. The Bosch-Hale fit needs the reduced-mass energy m_r c². The correct values are already defined asmc2in the same function but are never used. ⟨σv⟩ scales as1/√(m_r c²), so at every temperature D+D→He3 was 6.9 % too high, D+D→T 52 % too low and D+He3→He4 75 % too low. D+T→He4 was correct.The fix uses
mc2in the formula, defines the D-T value (1124656) under the same name and removes theerlines. Nothing else changes.The new
test/runtests_nuclear.jlpins 24 values of Table VIII of Bosch & Hale 1992 (thermal reactivities at 1–50 keV, as reprinted in the erratum, Nucl. Fusion 33, 1919) atrtol = 1e-3. 18 of them fail before the fix and all pass after it. I ran this test file with the package (Pkg.testwithruntests_nuclear.jl), not the rest of the suite.This changes the D-D and D+He3 source terms and powers wherever those reactions are enabled. #311 asks about the ½ factor for identical D-D reactants; that is a separate issue and not part of this PR.
🤖 Generated with Claude Code