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
MNT: Pin numpy<2 for v5.3.x #15234
MNT: Pin numpy<2 for v5.3.x #15234
Conversation
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
@@ -211,7 +210,7 @@ def test_bounds_gauss2d_lsq(self, fitter): | |||
ctx2 = nullcontext() | |||
if isinstance(fitter, fitting.TRFLSQFitter): | |||
ctx = np.errstate(invalid="ignore", divide="ignore") | |||
if not NUMPY_LT_2_0 or not SCIPY_LT_1_11_2: | |||
if not SCIPY_LT_1_11_2: |
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.
Hmm... Looks like this warning somehow went away again in dev scipy?
that now fails in devdeps with scipy dev. Since this is backport branch and the warning may or may not appear, let's just blanket ignore the unsuccessful fit because test case is not well defined to begin with.
I think this makes a lot of sense! |
@mhvk , wanna approve? 😉 |
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.
@pllim - am happy to approve this, though perhaps good to have someone from the @astropy/astropy-project-release-team approve too? Though, in a way, this really just makes sense...
@@ -2,7 +2,7 @@ | |||
requires = ["setuptools", | |||
"setuptools_scm>=6.2", | |||
"cython==0.29.34", | |||
"oldest-supported-numpy", | |||
"numpy>=1.25,<2", |
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.
Is there a reason to have the minimum build version higher than the minimum runtime version? That’s generally the thing that has caused problems in the past and is the reason for oldest-supported-numpy
.
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.
Numpy 1.25 is the first version that supports backward compatible builds. The release notes explicitly mention that the mechanism makes oldest-supported-numpy not needed
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.
More discussions happened in #14949
@astrofrog and @saimn gave verbal approval on Slack, so merging. Thanks, everyone! |
What about v5.0.x? Should we make the same change there? |
I already did that much earlier due to something else. So now both of them are pinned. |
Description
I think it is reasonable to say that for v5.3.x, we will not support NumPy 2.0 or later? The support for v5.3.x will end anyway when v6.0.x is cut, which is schedule to be by the end of 2023.
Motivations:
main
. Backport is just more churn.