-
Notifications
You must be signed in to change notification settings - Fork 25
New ccsn model: Takata_2025 #418
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
Conversation
JostMigenda
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for adding this new model! ✨
I’ve got two initial comments at first glance; haven’t looked at the code in any detail yet, but will try to do this next week, if I find time during a conference.
.DS_Store
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like you added this file (and a few similar ones in subdirectories) accidentally. Can you please remove them?
(And, optionally, feel welcome to add this to the .gitignore file; to help everyone avoid this in the future.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I did not realize git add -A will stage these files as well. I will remove them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you revert these changes please? They're unrelated to this PR.
(Together with the .DS_Store files, this makes me guess that you used git commit -a? Please be very careful with that or try to get out of that habit completely—it makes it really easy to accidentally include unintended files …)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do, thanks for pointing it out.
|
I have made the changes suggested in the above two comments, and have added .DS_Store to the .gitignore file as well.
I got the same message for TestFlavorTransformations.test_EarthMatter_IMO. Seems there is mismatch between |
|
The unit test module
All the other unit tests look OK, either passing or with expected failures. |
|
After discussion in the SNEWS2 signal prediction telecon on 2025-12-16, I think we'll just merge the change and then fix the two broken unit tests. |
|
Noted. Thank you. |
…3/new_model_Takata2025
Added a new 1-D core-collapse supernova model, Takata_2025, simulated in recent work by Takata, T. et al, 2025 PhysRevD.111.103028 in the presence of axion-like particles. The corresponding model files have been added to
snewpy-models-ccsn.