Skip to content

[enhancement] HMS+HMS Mergers rejuvenation according to Schneider+2016#788

Open
ezapartas wants to merge 15 commits intov2.3from
dimitris_masos_MSMSmergers_rejuvenation_averageX
Open

[enhancement] HMS+HMS Mergers rejuvenation according to Schneider+2016#788
ezapartas wants to merge 15 commits intov2.3from
dimitris_masos_MSMSmergers_rejuvenation_averageX

Conversation

@ezapartas
Copy link
Copy Markdown
Contributor

@ezapartas ezapartas commented Jan 16, 2026

  • This PR is changing the default option in the POSYDON .ini file of HMS_HMS_merging_rejuvenation = True to stronger rejuvenation of MS+MS mergers (according to Glebbeek+2023, Schneider+2016, following the recent Wang+2025 too). So it should be considered a change in POSYDON results for few types mergers

-This PR includes already the changes in #785 and builds on it. If accepted, that PR becomes unnecessary

…d the possibility to match with total_mass_h1
@ezapartas ezapartas marked this pull request as draft January 16, 2026 06:55
@ezapartas
Copy link
Copy Markdown
Contributor Author

With @dimsour94 we tested it with various options run.

@ezapartas ezapartas marked this pull request as ready for review January 24, 2026 10:11
added the options and default values of of rel_mass_lost_HMS_HMS and HMS_HMS_merging_rejuvenation in the .ini file of the population.
@maxbriel maxbriel changed the base branch from main to v2.3 February 10, 2026 08:21
@maxbriel maxbriel changed the title HMS+HMS Mergers rejuvenation according to Schneider+2016 [enhancement] HMS+HMS Mergers rejuvenation according to Schneider+2016 Feb 10, 2026
@maxbriel maxbriel added the enhancement New feature or request label Feb 10, 2026
@maxbriel maxbriel added this to the v2.3 milestone Feb 13, 2026
Comment thread posydon/popsyn/population_params_default.ini Outdated
@sgossage
Copy link
Copy Markdown
Contributor

sgossage commented Mar 24, 2026

We should merge PR#785 into main, then main into v2.3, then into this branch to incorporate the latest changes from PR#785, main, and v2.3.

4/23/26: I went ahead and merged v2.3 (which contains PR#785) into this.

@sgossage
Copy link
Copy Markdown
Contributor

sgossage commented Mar 24, 2026

We should update the docs here with these changes.

4/24/26: I added some updates to the docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants