You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, we use a sharp transition when building the 1D profile for the wdmerger. It is based on M_enclosed = mass_want - envelope_mass, where mass_want is the total mass of the WD and envelope_mass is the He envelope mass.
We want to make this transition smooth. I used the same tanh profile we use in subchandra 1D profile. The temperature profile is still isothermal as used in the default set up. However, since we are changing the composition as we move outward through the transition, the he mass needs to be integrated to converge to the requested envelope mass.
I'm confused how the original code was working -- where was it checking for convergence for the desired envelope mass? Are we sure that the envelope mass was correct originally? We don't seem to print it out.
So the original code does not check for convergence separately. It starts assigning He comp once M_enclosed = mass_want - envelope_mass is reached. It then outputs the requested He shell mass in the output file when the model is generated. For example -
Creating primary with CO core with mass fractions C = 0.5and O = 0.5 and a He shell of solar mass =5.9652e+31.
Creating secondary with CO core with mass fractions C = 0.5and O = 0.5 and a He shell of solar mass =5.9652e+31.
Generated initial model for primary WD of mass 1.05 solar masses, central density 4.767e+07 g cm**-3, and radius 5.047e+08 cm.
Generated initial model for secondary WD of mass 0.7 solar masses, central density 6.274e+06 g cm**-3, and radius 7.641e+08 cm.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Currently, we use a sharp transition when building the 1D profile for the wdmerger. It is based on
M_enclosed = mass_want - envelope_mass, wheremass_wantis the total mass of the WD andenvelope_massis the He envelope mass.We want to make this transition smooth. I used the same tanh profile we use in
subchandra1D profile. The temperature profile is still isothermal as used in the default set up. However, since we are changing the composition as we move outward through the transition, the he mass needs to be integrated to converge to the requested envelope mass.