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

Degraded precision of evolutionary models when Ledoux switched on #624

Open
parallelpro opened this issue Feb 16, 2024 · 4 comments
Open
Assignees

Comments

@parallelpro
Copy link

parallelpro commented Feb 16, 2024

@mjoyceGR and I noticed that a 1Msun evolution track (r230501) is a bit bumpy on the HR diagram (Lum vs Teff) around the first acent RGB, with use_Ledoux_criterion = .true.. Here I attached the inlist to reproduce this track. For comparision, with use_Ledoux_criterion = .false. the track is much smoother. It would be great to gather some insights on this issue and improve it in future releases.

ledoux

inlist_r230501_Ledoux_on.txt

inlist_r230501_Ledoux_off.txt

@evbauer
Copy link
Member

evbauer commented Mar 5, 2024

Does this persist if you include predictive mixing or convective premixing in your model? Without one of those options or some overshoot, it's very likely that convective boundary evolution will be incorrect, and that might be the cause of some of these features.

@pmocz pmocz self-assigned this Jul 23, 2024
@pmocz
Copy link
Member

pmocz commented Aug 16, 2024

I was able to reproduce this issue.

You can get good results with use_Ledoux_criterion = .true. if you also set num_cells_for_smooth_gradL_composition_term=0 (the default value is 3; changing this parameter should not affect your results when ledoux criterion is false).

An explanation of this parameter and why setting it to 0 helps is found in the docs: https://docs.mesastar.org/en/latest/reference/controls.html#num-cells-for-smooth-gradl-composition-term

which I reproduce below:

Number of cells on either side to use in weighted smoothing of gradL_composition_term. gradL_composition_term is set to the “raw” unsmoothed brunt_B and then optionally smoothed according num_cells_for_smooth_gradL_composition_term. In cases where the Ledoux criterion is used to evaluate the boundary for burning convective cores, you may need to set num_cells_for_smooth_gradL_composition_term = 0 to avoid smoothing the stabilizing composition jump into the convection zone and unphysically causing it to shrink. See section 3.2 in Moore, K., & Garaud, P. 2016, APJ, 817, 54

@mjoyceGR
Copy link
Member

Update: spoke to Yaguang about this. We are not using Brunt smoothing and it sounds like we shouldn't, given the concerns raised that this "trick" doesn't take any physics into account and is based on cells rather than mass coordinate

@evbauer
Copy link
Member

evbauer commented Oct 18, 2024

Just a quick clarifying question: when you say "Brunt smoothing" are you referring to the control num_cells_for_smooth_gradL_composition_term or num_cells_for_smooth_brunt_B? Obviously these two are very related physically, but I believe the former is applied for convection while the latter is applied for reported Brunt profiles.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants