8000 Use Numpy 1.22.2 instead of 1.22.0 by lpsinger · Pull Request #58 · scipy/oldest-supported-numpy · GitHub
[go: up one dir, main page]

Skip to content

Use Numpy 1.22.2 8000 instead of 1.22.0 #58

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

Merged
merged 3 commits into from
May 28, 2022
Merged

Conversation

lpsinger
Copy link
Contributor

Numpy 1.22.0 had a regression that prevented building packages against the Python limited C API, that was fixed in Numpy 1.22.2. See numpy/numpy#20818.

This is causes build failures for my Astropy affiliated package (https://git.ligo.org/lscsoft/ligo.skymap), which like many Astropy affiliated package, depends on oldest-supported-numpy.

lpsinger and others added 2 commits May 27, 2022 18:28
Numpy 1.22.0 had a regression that prevented building packages
against the Python limited C API, that was fixed in Numpy 1.22.2.

See numpy/numpy#20818.
Copy link
Member
@rgommers rgommers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks @lpsinger. I've added a comment with this rationale.

@rgommers rgommers merged commit 4aace24 into scipy:main May 28, 2022
@rgommers
Copy link
Member

@lpsinger lpsinger deleted the numpy-1.22 branch May 30, 2022 01:09
lpsinger added a commit to lpsinger/oldest-supported-numpy that referenced this pull request Jun 2, 2022
This is a follow up to scipy#58. The Numpy regression fixed in numpy/numpy#20818
was introduced in Numpy 1.21.0, not 1.22.0. My mistake. Sorry!
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
0