COSMIC-PopSynth / COSMIC

COSMIC (Compact Object Synthesis and Monte Carlo Investigation Code)
GNU General Public License v3.0
48 stars 59 forks source link

Possible bug in the if conditions for white dwarf dynamical mass transfer #572

Open claireshiye opened 2 years ago

claireshiye commented 2 years ago

On line 2486 elseif(kstar(j1).eq.10.or.kstar(j2).gt.10)then in evolv2.f https://github.com/COSMIC-PopSynth/COSMIC/blob/561e8c6db75bc71e011ddd4e8f0b2dd247850993/cosmic/src/evolv2.f#L2486 the 'or' should probably be 'and' in this if condition according to the comments for this block.

If using the default COSMIC, when there is a dynamical mass transfer from a CO WD, for example, to a ONe WD, the ONe WD will temporarily become a type 8 giant star. This will cause the ONe WD to be destroyed (turned into type 15) instead of forming an NS if the WD accretes a lot of mass during dynamical mass transfer (e.g., when eddfac>=10).

Here are a few outputs as examples for comparison. All outputs are for this binary initial condition single_binary = InitialBinaryTable.InitialBinaries(m1=1.2, m2=1.37999, porb=0.2, ecc=0., tphysf=13700.0, kstar1=11, kstar2=12, metallicity=0.002) If change the 'or' in the if condition to 'and':

image

if use eddfac=10:

image

If using the default as in COSMIC V3.4:

image

and then if use eddfac=10:

image