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

Fixes initialisation in MS_gt_07::Initialise() for CORE_MASS_PRESCRIPTION::SHIKAUCHI #1326

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

jeffriley
Copy link
Collaborator

MS_gt_07::Initialise() now allows for CH stars that spin down when initialisation done for CORE_MASS_PRESCRIPTION::SHIKAUCHI.

@brcekadam I haven't tested this other than for compilation - can you check that the CORE_MASS_PRESCRIPTION::SHIKAUCHI code functions as you expect. If you can find a CH star that spins down, you could check for that too (but don't worry too much if you can't find one easily).

…TION::SHIKAUCHI (now allows for CH stars that spin down)
@jeffriley jeffriley added bug Something isn't working severity_minor This bug is not very severe urgency_low This issue is not urgent labels Jan 28, 2025
@jeffriley jeffriley self-assigned this Jan 28, 2025
Copy link
Collaborator

@ilyamandel ilyamandel left a comment

Choose a reason for hiding this comment

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

Looks good to me!
@jeffriley , I leave it to you and @brcekadam to decide whether to merge now or wait for Adam's review. :)

Copy link
Collaborator

@brcekadam brcekadam left a comment

Choose a reason for hiding this comment

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

Thanks @jeffriley! I tested it and everything looks good. I wasn't able to find a CH star that spins down, but there shouldn't be an issue anymore.

@jeffriley jeffriley merged commit acc2772 into TeamCOMPAS:dev Jan 30, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working severity_minor This bug is not very severe urgency_low This issue is not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants