-
Notifications
You must be signed in to change notification settings - Fork 157
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 2.0 #210
Comments
@da-wad Thanks for the heads up. We've not yet looked at the impact of Numpy 2.0. Are you aware of anything in particular that will require changes to zfpy? |
Nothing more than what is in the summary text in that issue. But: "the NumPy C ABI will change in 2.0, so any compiled extension modules that rely on NumPy are likely to break, they need to be recompiled." may mean that compiling new wheels is all we need to do. However, I think this highlights the need for some distance between zfpy and zfp versioning... |
JFYI 3 weeks ago, NumPy 2.0.0rc1 was tagged. Packages were built for conda & wheels: numpy/numpy#24300 (comment) Would add NumPy has put together a migration guide. More details are in the release notes If zfpy make use of NumPy's C API (and produces wheels that use it), having a release of zfpy with wheels built against NumPy 2.0.0rc1 would be helpful to ensure NumPy 1 & 2 compatible wheels (as wheels built against NumPy 1 won't be compatible with NumPy 2). More details in this NumPy 2 ABI doc Also as NumPy is tracking ecosystem support for NumPy 2.0, it would be helpful to share zfpy's current support status (with any plans) in this issue: numpy/numpy#26191 |
@jakirkham Thanks for the heads up. We are currently looking for someone to help out with maintaining zfpy and building wheels as we're short on staff and expertise in this area. Any contributions would be welcome and appreciated. |
Would recommend using whatever build script is used currently and try NumPy 2 instead (maybe drop Given the light usage of NumPy in zfpy, it is possible that is all that is needed |
Had a bit of a look around, and I think @jakirkham is right. Assuming the current build setup works (it doesn't look very "formal"): all that should be needed is replacing |
I added #231, let's see what happens. |
Update from #231 (comment). I see two warnings:
I don't think those are new (due to 2.0) though. Tests are passing, so I think it's ok? |
Yes they are ancient, but we didn't enforce them now, there was just little point. |
This is breaking with numpy 2.0.0. Looks like a change to numpy's array struct.
If I build using (after updating setup.py / pyproject.toml)
the problem disappears. |
Yes this is a known issue. The builds are not compatible with NumPy 2. Would stick with NumPy 1 for now if you need zfpy |
Numpy 2.0 is coming: numpy/numpy#24300
There will need to be a
zfpy
release built for this, as 1.0.0 wheels will (very likely) not be compatible with numpy 2.xIs anyone looking into this yet?
The text was updated successfully, but these errors were encountered: