-
Notifications
You must be signed in to change notification settings - Fork 145
Numpy 20 fixes #565
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
Numpy 20 fixes #565
Conversation
Currently blocked by multiple regular but not deterministic segmentation faults, that seem to be most commonly from
No such segmentation faults on Numpy 1.26.4, which tests clean. |
@alexis-roche - do you have any time to have a look at what's going on here with Procedure is: virtualenv nipy-env
. ./nipy-env/bin/activate
git clone https://github.com/matthew-brett/nipy
cd nipy
git checkout numpy-20-fixes
pip install meson-python ninja cython numpy pytest spin
spin install
pytest nipy/labs/bindings/tests/test_numpy.py If it doesn't segfault first time, try running the In case it's useful - Numpy 2.0 changes : https://numpy.org/doc/stable//numpy_2_0_migration_guide.html |
9a4729d
to
66e5c52
Compare
I'm not getting segfaults, but I am getting:
I don't have any insights into what's causing this. |
This is a nasty macro that includes a return statement.
Thanks to Sebastian's comments at: numpy/numpy#26765 (comment)
66e5c52
to
3ea60ee
Compare
numpy.rec is not a module, but a synonym for a module, so cannot be imported, at least for some versions of Numpy.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #565 +/- ##
==========================================
- Coverage 84.70% 84.61% -0.09%
==========================================
Files 297 293 -4
Lines 27590 27479 -111
Branches 3361 3362 +1
==========================================
- Hits 23370 23252 -118
- Misses 3266 3273 +7
Partials 954 954 ☔ View full report in Codecov by Sentry. |
Were getting Windows error
c6d8fd6
to
934515f
Compare
Given Py3.8 is end-of-life next week, might make more sense to just remove support and end testing on it. |
Good call - dropped. Tests passing now. |
Ah - that turns out to be because Python 3.8 pulls in Numpy < 2, which is the actual problem. |
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.
LGTM
OK - added back 3.8 to trigger a Numpy < 2 build - and added some Numpy 1.2x fixes to the original Numpy 2.0 Nipy port. I think this is ready now. And it will fix a few of the outstanding issues. |
Numpy issue with description, discussion, resolution of segfault: numpy/numpy#26765 |
Working on build / compile and test for Numpy 2.0.