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

Demo Numpy v2 #6035

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Demo Numpy v2 #6035

wants to merge 11 commits into from

Conversation

trexfeathers
Copy link
Contributor

🚀 Pull Request

Description

We already know that Iris' syntax is NumPy v2 compliant, thanks to Ruff, but that does not prove that the code will all function with NumPy v2.

A vanilla Conda install of Iris does get NumPy v2, but unfortunately our full test environment does not (some of the dependencies must be pinned). So I have forced the issue by manually editing requirements/locks/py312-linux-64.lock, which does thankfully produce a working environment since none of NumPy's dependencies have changed.

The test failures show us what work we need to do for Iris to be compliant with NumPy v2. It seems to mostly be around changes in:

  • Handling of array dtypes
  • Array string representation
  • Array difference tolerances

Consult Iris pull request check list


Add any of the below labels to trigger actions on this PR:

  • benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts

@github-actions github-actions bot added the benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts label Jul 2, 2024
trexfeathers added a commit to trexfeathers/iris that referenced this pull request Jul 2, 2024
bjlittle pushed a commit that referenced this pull request Jul 3, 2024
* NEP29 NumPy pin.

* Pin back from NumPy v2 (#6035).

* PyPI pins.
@pp-mo pp-mo mentioned this pull request Jul 5, 2024
1 task
@trexfeathers trexfeathers added this to the v3.11 milestone Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
benchmark_this Request that this pull request be benchmarked to check if it introduces performance shifts
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

1 participant