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

CI: add a build against numpy nightly #129

Merged
merged 2 commits into from
Dec 20, 2023

Conversation

jakevdp
Copy link
Collaborator

@jakevdp jakevdp commented Dec 13, 2023

The test is failing, because upstream numpy removed PyArray_TypeNumFromName last week (numpy/numpy#25292). This is a real failure that we'd like this test to catch. When I commented-out that part of the source code, tests passed.

I propose we merge this as-is, and then work to fix the failure by using an alternative API.

Fixed in the second commit on the branch.

@jakevdp jakevdp self-assigned this Dec 13, 2023
@jakevdp
Copy link
Collaborator Author

jakevdp commented Dec 13, 2023

CI failure should be fixed by numpy/numpy#25389

@jakevdp jakevdp force-pushed the ci-numpy-nightly branch 2 times, most recently from 42b9edd to 39edf7f Compare December 13, 2023 17:44
@jakevdp
Copy link
Collaborator Author

jakevdp commented Dec 13, 2023

Looks like failures are due to pybind11 not being compatible with numpy 2.0. We need a release with pybind/pybind11#4857 (2.11.1 is currently the most recent release, and does not include this change).

@jakevdp jakevdp force-pushed the ci-numpy-nightly branch 8 times, most recently from e3e3996 to 3993f56 Compare December 19, 2023 23:49
@jakevdp
Copy link
Collaborator Author

jakevdp commented Dec 19, 2023

Finally hit a real failure! numpy/numpy#25292 removed PyArray_TypeNumFromName, which ml_dtypes uses. I've left a comment on the PR to request guidance.

@jakevdp jakevdp requested a review from hawkinsp December 20, 2023 00:06
@jakevdp jakevdp force-pushed the ci-numpy-nightly branch 2 times, most recently from 96f50c0 to bfa7f3b Compare December 20, 2023 00:43
@jakevdp jakevdp force-pushed the ci-numpy-nightly branch 4 times, most recently from 5c99d88 to e5bd04f Compare December 20, 2023 17:12
@jakevdp jakevdp force-pushed the ci-numpy-nightly branch 2 times, most recently from e5da001 to 818a50f Compare December 20, 2023 17:19
@jakevdp jakevdp requested a review from hawkinsp December 20, 2023 17:20
ml_dtypes/_src/custom_float.h Show resolved Hide resolved
@copybara-service copybara-service bot merged commit 29edcb3 into jax-ml:main Dec 20, 2023
11 checks passed
@jakevdp jakevdp deleted the ci-numpy-nightly branch December 20, 2023 19:14
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