Closed wtbarnes closed 5 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Comparison is base (
65130bb
) 92.00% compared to head (04c674d
) 92.12%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Owee, I'm MrMeeseeks, Look at me.
There seem to be a conflict, please backport manually. Here are approximate instructions:
git checkout 0.2
git pull
Cherry pick the first parent branch of the this PR on top of the older branch:
git cherry-pick -x -m1 9d531da3b0cf4691160491c14919b6deda31c91c
You will likely have some merge/cherry-pick conflict here, fix them and commit:
git commit -am 'Backport PR #262: Use total recombination data if available to calculate recombination rate'
git push YOURFORK 0.2:auto-backport-of-pr-262-on-0.2
"Backport PR #262 on branch 0.2 (Use total recombination data if available to calculate recombination rate)"
And apply the correct labels and milestones.
Congratulations — you did some good work! Hopefully your backport PR will be tested by the continuous integration and merged soon!
Remember to remove the Still Needs Manual Backport
label once the PR gets merged.
If these instructions are inaccurate, feel free to suggest an improvement.
This fixes a bug where the total recombination rate data in the
trparams
files should be used if available. Previously, I was just assuming that the recombination rate is always the sum of the methods that calculate the radiative and dielectronic recombination rates and thus ignoring the trparams data altogether. However, if this total recombination rate data is available, it should take precedence as it is perceived as being more accurate (though there is some controversy around this).