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

Automatically create a pynbodyrc file if a user tries to load FIRE snapshot #141

Merged
merged 3 commits into from
Nov 5, 2024

Conversation

TomWagg
Copy link
Owner

@TomWagg TomWagg commented Nov 5, 2024

cogsworth now detects if a user is missing a pynbodyrc file that fixes the units for a FIRE snapshot loaded with pynbody.

This will fix #137.

@TomWagg TomWagg added the documentation Improvements or additions to documentation label Nov 5, 2024
@TomWagg TomWagg self-assigned this Nov 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1723 1708 99% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
cogsworth/_version.py 100% 🟢
cogsworth/hydro/utils.py 69% 🟢
TOTAL 84% 🟢

updated for commit: ad23a66 by action🐍

@TomWagg TomWagg merged commit d9a9a0d into main Nov 5, 2024
2 checks passed
@TomWagg TomWagg deleted the pynbody branch November 5, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error loading snapshot
1 participant