Skip to content

Small fix to CO core masses for AGB stars#762

Merged
katiebreivik merged 2 commits into
COSMIC-PopSynth:developfrom
TomWagg:snfix
Feb 19, 2026
Merged

Small fix to CO core masses for AGB stars#762
katiebreivik merged 2 commits into
COSMIC-PopSynth:developfrom
TomWagg:snfix

Conversation

@TomWagg

@TomWagg TomWagg commented Feb 18, 2026

Copy link
Copy Markdown
Collaborator

In #756 we changed the conditions in assign_remnant.f to use mc_co instead of mc. However, we didn't ensure that mc_co was set to mcmax if we overshot it. This meant that there was a timestepping issue that gave you a nonmontonic result of an SN occuring with smooth changes in initial masses and metallicity.

Here's the testing script:
min_sn_crit.py

Before:
image

After:
image

@TomWagg
TomWagg requested a review from katiebreivik February 18, 2026 23:11
@TomWagg TomWagg self-assigned this Feb 18, 2026
@TomWagg TomWagg added the bug label Feb 18, 2026
@codecov

codecov Bot commented Feb 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 77.83%. Comparing base (8772c07) to head (78f87a7).
⚠️ Report is 137 commits behind head on develop.

Files with missing lines Patch % Lines
src/cosmic/_version.py 0.00% 1 Missing ⚠️

❌ Your patch status has failed because the patch coverage (50.00%) 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     #762      +/-   ##
===========================================
- Coverage    86.91%   77.83%   -9.08%     
===========================================
  Files           40       50      +10     
  Lines        25542    27517    +1975     
  Branches         0      921     +921     
===========================================
- Hits         22198    21417     -781     
- Misses        3344     5813    +2469     
- Partials         0      287     +287     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@katiebreivik katiebreivik left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @TomWagg !!

@katiebreivik
katiebreivik merged commit b49872c into COSMIC-PopSynth:develop Feb 19, 2026
5 of 7 checks passed
@TomWagg
TomWagg deleted the snfix branch June 1, 2026 19:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants