-
Notifications
You must be signed in to change notification settings - Fork 25
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
Cython3 #1421
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1421 +/- ##
==========================================
- Coverage 99.92% 99.92% -0.01%
==========================================
Files 37 37
Lines 20871 20868 -3
==========================================
- Hits 20855 20852 -3
Misses 16 16 ☔ View full report in Codecov by Sentry. |
@@ -19,7 +19,7 @@ dependencies: | |||
- pytest-cases>=3.8.3 | |||
- pytest-cov | |||
- pytest-xdist | |||
- cython | |||
- cython>=3.0 |
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.
This should also be updated in the readme and maybe in setup.py?
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.
Good call thanks for pointing those out.
for more information, see https://pre-commit.ci
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.
Looks good, I think there's just one more place to update, the pyuvdata_min_versions_tests.yml
. Sorry I didn't notice that one before.
oh good catch. I must have missed that when I was searching through. |
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.
Looks good, thanks!
bumps the cython version to 3.0. It's been stable since October 2023.
Not sure if it is possible to change some string thins in corr fits so i'd hold off on full PR for time being.