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

Unpin Numpy version for building wheels #201

Merged
merged 2 commits into from
Nov 27, 2023

Conversation

lpsinger
Copy link
Contributor

Allow using any version of Numpy between 1.25 and 2. This will allow us to use the latest release of Numpy on PyPI and take advantage of prebuilt wheels for more platforms and architectures.

Use the new NPY_TARGET_VERSION macro to peform a backward-compatible supporting versions of Numpy as old as 1.19.

@lpsinger lpsinger added the Build wheels Run the publish workflow label Nov 20, 2023
@lpsinger lpsinger closed this Nov 20, 2023
@lpsinger lpsinger reopened this Nov 20, 2023
Copy link

codecov bot commented Nov 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (855a8f9) 88.27% compared to head (f95a90d) 88.27%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #201   +/-   ##
=======================================
  Coverage   88.27%   88.27%           
=======================================
  Files           6        6           
  Lines         503      503           
=======================================
  Hits          444      444           
  Misses         59       59           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Allow using any version of Numpy between 1.25 and 2. This will
allow us to use the latest release of Numpy on PyPI and take
advantage of prebuilt wheels for more platforms and architectures.

Use the new `NPY_TARGET_VERSION` macro to peform a
backward-compatible supporting versions of Numpy as old as 1.19.
@lpsinger lpsinger force-pushed the unpin-numpy-build-requires branch from 25b68cb to 6b98327 Compare November 21, 2023 13:35
@lpsinger lpsinger requested review from astrofrog and pllim November 21, 2023 13:36
Copy link
Member

@pllim pllim left a comment

Choose a reason for hiding this comment

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

I'll let @astrofrog approve since I am not a maintainer of this repo but the diff LGTM. Thanks!

pyproject.toml Outdated Show resolved Hide resolved
Co-authored-by: P. L. Lim <2090236+pllim@users.noreply.github.com>
@astrofrog astrofrog merged commit 9ad66ad into astropy:main Nov 27, 2023
17 checks passed
@lpsinger lpsinger deleted the unpin-numpy-build-requires branch November 27, 2023 15:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build wheels Run the publish workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants