-
Notifications
You must be signed in to change notification settings - Fork 40
Changing default to exclude recombination terms in neutrino cooling #1793
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changing default to exclude recombination terms in neutrino cooling #1793
Conversation
|
you'll need to regenerate the benchmarks. You can see how they are created and where they are stored by looking at the files in |
|
Benchmarks regenerated |
|
diffs are small for all tests except for the nova: |
Merge branch 'recomb_change' of github.com:khanakbhargava/Microphysics into recomb_change
This is what I get if I decrease
|
|
but |
|
With those numbers this is what we get - variable name absolute error relative error
|
In this case first I had changed both atol and rtol which increased the errors. So I tried changing on atol and kept rtol same. |
|
I'm confused. Building if I rerun them with the tolerances I suggest above, I see: so it seems much closer with the tighter tolerances |
|
All checks pass now except for castro, conflicts resolved. |
unit_test/burn_cell/ci-benchmarks/triple_alpha_plus_cago_FE_unit_test.out
Show resolved
Hide resolved
|
yes, please restore the ones that did not fail. that way the git history will reflect what actually needed to be updated for this change. |
No description provided.