-
Notifications
You must be signed in to change notification settings - Fork 12
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
Update setup.py / pyproject.toml to follow latest numpy and cython docs #271
Conversation
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.
I had to read a bit to figure out why in pyproject.toml
you require numpy>=1.25
and in setup.py
you require numpy>=1.21
and NPY_1_21_API_VERSION
, but I think I understood and it's not a bug, so I approve (the one comment I left is not critical).
Dropping python 3.8 required for the numpy >= 1.25 requirement. But according to the NEP 29 schedule, this was overdue anyway and e.g. ctapipe has already done it. |
Yes, this is a bit confusing, but it boils down to:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #271 +/- ##
=======================================
Coverage 86.58% 86.58%
=======================================
Files 24 24
Lines 2222 2222
=======================================
Hits 1924 1924
Misses 298 298 ☔ View full report in Codecov by Sentry. |
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.
All good now!
Remove all the magic for allowing compilation of bundled c files without Cython, this was obsolete since we python
Cython
into the build requirements inpyproject.toml
. Supporting the case where people don't have cython doesn't make sense, since pip ensures it.Numpy 1.25 made
oldest-supported-numpy
obsolete, we can chose the lowest version of numpy we want to support with a definition as long as we compile using at least numpy 1.25.