Skip to content
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

Inability to get through helium flash for 1.05 - 1.4 Msolar #563

Open
mjoyceGR opened this issue Jul 21, 2023 · 1 comment
Open

Inability to get through helium flash for 1.05 - 1.4 Msolar #563

mjoyceGR opened this issue Jul 21, 2023 · 1 comment
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request test_suite Issue in test_suite case

Comments

@mjoyceGR
Copy link
Member

mjoyceGR commented Jul 21, 2023

MESA version 23051

Comparing this inlist (attached as .txt) for a science case against inlist_1.3M_ms_high_Z from the test_suite case of the same name, I have systematically removed each difference in physics (replace with basic.net, use standard opacities, etc) to try to isolate the reason the science inlist fails. I have not found any change that enables successful passage through the helium flash
inlist_issue.txt

However, introduction of the following control parameters does enable passage through the he flash:
convergence_ignore_equL_residuals = .true. relax_use_gold_tolerances = .true.

Despite confirmation from other devs that this control is needed to get models in this regime through the he flash (as of some point after release 10398 and the solver controls overhaul),
there is no documentation to accompany convergence_ignore_equL_residuals = .true.

I think a few things need to happen:

  1. the effect of this and similar controls needs to be documented. As far as I can tell, the meaning of this control is to shut off consideration of discrepancies in the luminosity equation entirely (?).
    a. Corollary: is that safe??

  2. It needs to be made loud and clear that this setting is necessary for (most) AGB models. I advocate for introducing it in inlist_1.3M_ms_high_Z even though it is not required in that case, just so users are aware of it. They will almost certainly consult this test_suite case when trying to figure out their own convergence issues for AGBs. It has been pointed out to me that this setting is in the 1M_pre_ms_to_wd test_suite case, but that is not the first place I personally would look for usable AGB settings.

  3. We need a test case for AGBs (@giulia-cinquegrana and I are doing this, stand by)

  4. Ideally, we need a better solution than (even a well-documented) magic flag for dealing with this. If the era of tighter tolerances made getting through the he flash impossible without telling the solver to ignore one of the stellar structure equations(?)...that seems like something we should fix.

end rant!

@evbauer evbauer added documentation Improvements or additions to documentation enhancement New feature or request test_suite Issue in test_suite case labels Jul 21, 2023
@evbauer
Copy link
Member

evbauer commented Jul 21, 2023

If the era of tighter tolerances made getting through the he flash impossible without telling the solver to ignore one of the stellar structure equations(?)...that seems like something we should fix.

Prior to the era of tighter tolerances, the MESA default was basically to ignore residuals after struggling for a certain number of iterations. So I think this control basically says to go back to that behavior for one specific equation that is likely to be the source of struggles during phases like the core flash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request test_suite Issue in test_suite case
Projects
None yet
Development

No branches or pull requests

3 participants