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

Save binary inclination and phase at each SN to ensure reproducibility #115

Merged
merged 7 commits into from
May 7, 2024

Conversation

TomWagg
Copy link
Owner

@TomWagg TomWagg commented May 6, 2024

Inclinations and phases are now all drawn together (faster!) and saved in the initC table (reproducible!). The new columns are phase_sn_X and inc_sn_X which are the phase and inclination at the moment of SN X where X is 1 or 2.

This will fix #114 .

@TomWagg TomWagg added the bug Something isn't working label May 6, 2024
@TomWagg TomWagg self-assigned this May 6, 2024
Copy link

github-actions bot commented May 6, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1549 1549 100% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
cogsworth/events.py 100% 🟢
cogsworth/kicks.py 100% 🟢
cogsworth/pop.py 100% 🟢
TOTAL 100% 🟢

updated for commit: a4a0d1c by action🐍

@TomWagg TomWagg merged commit 8762b1b into main May 7, 2024
2 checks passed
@TomWagg TomWagg deleted the save_inc branch May 7, 2024 00:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

We don't track the inclination and phase that we draw during kick
1 participant