Added metallicity dependence to martinez26 and added martinez26_ecsn#760
Added metallicity dependence to martinez26 and added martinez26_ecsn#760katiebreivik merged 4 commits intoCOSMIC-PopSynth:developfrom
Conversation
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (40.43%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## develop #760 +/- ##
===========================================
- Coverage 86.91% 77.78% -9.13%
===========================================
Files 40 50 +10
Lines 25542 27555 +2013
Branches 0 921 +921
===========================================
- Hits 22198 21431 -767
- Misses 3344 5837 +2493
- Partials 0 287 +287 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
@TomWagg mentioned that we might consider making this a metallicity-specific limit but I’m happy to merge with the inclusive 6.8 Msun limit for now once that one comment is addressed above. |
…n to fit for min ECSN masses
|
The mass cutoff is the FeCCSN minimum mass for |
|
thanks @MarkGM02 ! |



Changed the primary mass from 8M⊙ to 6.8M⊙ which divides high and low mass stars for the martinez26 orbital period option. This cutoff mass was chosen since it is the minimum mass for an FeCCSN at COSMIC's lowest metallicity, when default winds are used.