Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Bump minimum NumPy version to 1.23#26800
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
ffc43e7
to5366eab
CompareSeems like it is not trivial to install numpy 1.26 (probably because of Meson) on CygWin (hence, the pinning in the install step). However, since build isolation is used, it will try to reinstall an unpinned version. The minver test is probably related, although I cannot really find anything specific in the 1.22 release notes that seems related. The PR that introduced the failing test is:#26253 |
Cherry-picked out ofmatplotlib#26800
Cherry-picked out ofmatplotlib#26800; also unpin setuptools.
Cherry-picked out ofmatplotlib#26800; also unpin setuptools.
Cherry-picked out ofmatplotlib#26800; also unpin setuptools.
Cherry-picked out ofmatplotlib#26800; also unpin setuptools.
Cherry-picked out ofmatplotlib#26800; also unpin setuptools.
oscargus commentedOct 15, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Seems like there are two floating-point related errors with 1.22. Since these are really small, should I just add a bit of tolerance (and a note that these are for NumPy 1..2)? An option is, I guess, to try 1.22.1 etc to see if this is a regression that was fixed. (Have checked the change log briefly, but not really sure what the underlying problem is anyway...) |
I've had to bump tolerance on at least one of those failing tests in Fedorafedora-python@d429c3d (though I cheated a little bit and put it in the non-x86 commit). I think it's likely to be the SIMD optimization issue that we've run into with newer NumPy. |
We discussed on the call today and decided to go directly to 1.23, so I've pushed that, and also updated the missed |
PR summary
I take it that we will stick with 1.22 for MPL 3.9. Probably we could have gone to 1.23, but I recall that we are a bit conservative here:https://numpy.org/neps/nep-0029-deprecation_policy.html
PR checklist