-
Notifications
You must be signed in to change notification settings - Fork 34
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
Astropy updates 2023.07 #381
Astropy updates 2023.07 #381
Conversation
Codecov Report
@@ Coverage Diff @@
## main #381 +/- ##
==========================================
+ Coverage 76.80% 76.84% +0.03%
==========================================
Files 78 78
Lines 6984 6974 -10
==========================================
- Hits 5364 5359 -5
+ Misses 1620 1615 -5
|
@mkelley can you give me some context for this PR? I remember seeing a relevant message from someone astropy, but didn't keep track of it. Thanks. |
This address issue #372, and simplifies the Afrho/Efrho code. |
And removes some code from the testing setup in conftest.py Since we do not support astropy < 4.3, none of these lines are executed in our tests, which is why the coverage is increased by this PR. |
Thanks Mike. All looks good to me. I also linked this PR with issue #372 if that's OK with you. |
Perfect! |
Currently supported astropy is >=4.3. Remove code branches for astropy < 4. Remove commented out reference to a deprecated astropy method.