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

Use oldest-supported-numpy in pyproject.toml #60

Merged
merged 2 commits into from
Feb 24, 2021

Conversation

jdavies-st
Copy link
Collaborator

@jdavies-st jdavies-st commented Feb 22, 2021

Instead of using the latest version of numpy as the build dependency, use the oldest compatible one via the meta package

https://github.com/scipy/oldest-supported-numpy

so that the compiled code can be used with any new version of numpy in the runtime environment.

This should resolve the issue discussed in spacetelescope/jwst#5744

For more detail, see pypa/pip#9542

@codecov
Copy link

codecov bot commented Feb 22, 2021

Codecov Report

Merging #60 (cc34ed8) into master (577a254) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   79.75%   79.75%           
=======================================
  Files           5        5           
  Lines         326      326           
=======================================
  Hits          260      260           
  Misses         66       66           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 577a254...cc34ed8. Read the comment docs.

@jdavies-st jdavies-st requested a review from eslavich February 22, 2021 18:48
Copy link

@eslavich eslavich left a comment

Choose a reason for hiding this comment

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

LGTM

"setuptools>=42",
"setuptools_scm[toml]>=3.4",
"wheel",
"oldest-supported-numpy",

Choose a reason for hiding this comment

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

Well that is a handy dandy feature isn't it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We may want to do this for other spacetelescope projects that have C code that need numpy installed to build/compile. Maybe we should file issues for the others as well.

@jdavies-st jdavies-st merged commit 80296bb into spacetelescope:master Feb 24, 2021
@jdavies-st jdavies-st deleted the oldest-supported-numpy branch February 24, 2021 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants