Skip to content
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

Build against numpy 2.0rc1 #113

Merged
merged 2 commits into from
Apr 12, 2024
Merged

Conversation

djhoese
Copy link
Collaborator

@djhoese djhoese commented Apr 2, 2024

No description provided.

@djhoese djhoese requested a review from mraspaud April 2, 2024 20:14
@djhoese
Copy link
Collaborator Author

djhoese commented Apr 2, 2024

cibuildwheel doesn't seem to show enough verbosity to know that it is using numpy 2 for building the wheel (it uses 1.26 for testing the wheel and passes). But since the pyproject.toml specifies it I think it is safe.

@djhoese
Copy link
Collaborator Author

djhoese commented Apr 8, 2024

As far as I can tell this works. I created a local environment with numpy 2.0.0rc1 and the wheels created by this PR and I can import it. If I install pyresample locally against numpy 2.0.0rc1 and try to import it it fails to import pandas with:

ValueError: numpy.dtype size changed, may indicate binary incompatibility. Expected 96 from C header, got 88 from PyObject

If you're OK with this @mraspaud then I think this is ready for merge and release.

@djhoese
Copy link
Collaborator Author

djhoese commented Apr 12, 2024

FYI @neutrinoceros I moved your Inf commit here, but didn't include the pyproject.toml ones as I like my version better 😉

@djhoese
Copy link
Collaborator Author

djhoese commented Apr 12, 2024

Got approval from Martin on slack...

@djhoese djhoese merged commit 7607815 into storpipfugl:master Apr 12, 2024
6 checks passed
@djhoese djhoese deleted the build-np2rc1 branch April 12, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants