Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@cournoyercloutierc
Copy link
Contributor

This fix ensures that the stellar properties (in particular the relative_mass) are preserved when restarting with SeBa. The other properties (such as the radius, luminosity, and temperature) are also correctly calculated during the stellar evolution step. The changes presented in this pull request also require the change proposed in commit 01dcff8da08833622a2b52c507f2275ea4f3117e to be implemented in the SeBa repository.

@spzwart
Copy link
Contributor

spzwart commented Jan 30, 2025

We checked this together, and it seems to be the right thing to do to realize restarts with evolved stars.

@spzwart spzwart closed this Jan 30, 2025
@rieder
Copy link
Member

rieder commented Jan 30, 2025

Then you don't want to close it but to merge it, I guess?

@rieder rieder reopened this Jan 30, 2025
@cournoyercloutierc
Copy link
Contributor Author

@rieder Yes, I think Simon was trying to merge it -- we were discussing it live during the Torch meeting. The corresponding pull request in the SeBa repo was just merged.

@rieder rieder merged commit 5dff550 into amusecode:main Jan 30, 2025
2 checks passed
@rieder
Copy link
Member

rieder commented Jan 30, 2025

the download.py still needs to be updated to the new SeBa version.

@cournoyercloutierc
Copy link
Contributor Author

cournoyercloutierc commented Jan 30, 2025

@rieder I can do that now and submit a new pull request, or would you prefer to re-open this one?

@cournoyercloutierc
Copy link
Contributor Author

cournoyercloutierc commented Jan 30, 2025

@rieder @spzwart I have prepared a pull request with the changed SeBa version, but there is an issue. Although the SeBa worker compiles without issues with the changes from the last pull request, any SeBa commit from 382b5904609b1a44577f1d652c2eba34f9192616 on includes changes that prevent the worker from compiling. How would you like to proceed?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.