i_confinement_time ITPA20-IL Scaling equation fix#4158
i_confinement_time ITPA20-IL Scaling equation fix#4158OceanNuclear wants to merge 2 commits intomainfrom
i_confinement_time ITPA20-IL Scaling equation fix#4158Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #4158 +/- ##
==========================================
+ Coverage 47.97% 49.44% +1.47%
==========================================
Files 143 149 +6
Lines 30125 29796 -329
==========================================
+ Hits 14451 14734 +283
+ Misses 15674 15062 -612 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Before the PR, the documentation page looks like: However, the bottom reference ([23]) shows this equation: |
i_confinement_time ITPA20-IL Scaling equation fix
… matches the table in the reference.
|
Regarding making the roundings more consistent, these are the values (α) and their uncertainties (σ) as recorded in the first paper ([22]):
|
|
@stuartmuldrew can you identify what the (+0059 - 0.032) is doing in the first term of the top equation, and the (+0.030 - 0.018) in the first term of the bottom equation? I can't see these being the upper and lower bounds of that constant |
|
@jonmaddock have you approved this as a maintainer as ready to be merged? Also are you managing this PR and should I put you as the assignee? |
My interpretation from the equation and the table is that is the error on the constant at the start. Admittedly, that is a pretty large upper bound. |
|
Thanks @stuartmuldrew. I think those values contradict with the actual upper and lower bound I calculated using the values given in the table (tabulated in the previous comment). Perhaps the (+0.059 - 0.032) and (+0.030 - 0.018) will remain a mystery forever. It doesn't affect anything above anyways. As for the failing regression test, that's due to the actual constants being changed (due to correction in number of digits rounded). Shall we change the regression test so that it passes then? @timothy-nunn |
jonmaddock
left a comment
There was a problem hiding this comment.
I don't understand why certain coefficients are being rounded to a lower accuracy: can you explain?
|
@stuartmuldrew Asked for a more consistent rounding in the issue #4017. The reduced accuracy reflects the lower confidence/higher uncertainty. |






Description
Closes #4017
reference [^23] is dereferenced into [^22].
Checklist
I confirm that I have completed the following checks: