Skip to content

Conversation

@bjlittle
Copy link
Member

🚀 Pull Request

Description

This PR introduces a minimum pin for numpy>=1.18 in order to honour NEP-29.

numpy is approximately 24 months old, and the latest version is 1.21, so we're supporting greater than the latest 3 minor versions.


Consult Iris pull request check list

@rcomer
Copy link
Member

rcomer commented Oct 21, 2021

#4129 states that our anticipated release date for Iris 3.2 is January. By that time, numpy 1.18 will have dropped out of the 24-month window, so we could pin to >=1.19 if we want.

Edit: I don’t really have an opinion either way though!

@bjlittle
Copy link
Member Author

bjlittle commented Oct 22, 2021

@rcomer I like your thinking... I'm happy to go with that 👍

BTW I'm not playing NEP-29 on to any of our other dependences. I personally think it makes sense to at least ensure numpy and python conform, particularly numpy as a change in numbers affects the science.

Copy link
Member

@rcomer rcomer left a comment

Choose a reason for hiding this comment

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

It’s been a week and nobody has objected in principle to this pin. Change itself looks fine to me.

@rcomer rcomer merged commit 1ec48bc into SciTools:main Oct 30, 2021
tkknight added a commit to tkknight/iris that referenced this pull request Nov 3, 2021
* main: (44 commits)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4395)
  min pin for numpy (nep29) (SciTools#4386)
  Updated environment lockfiles (SciTools#4393)
  Extend stock.mesh api (SciTools#4389)
  Updated environment lockfiles (SciTools#4388)
  Integrate ASV with Nox (SciTools#4378)
  NetCDF save - stream ALL lazy arrays. (SciTools#4375)
  adopt flake8 maccabe complexity metric (SciTools#4380)
  Accept inverse_flattening = 0 for spherical ellipsoid (closes SciTools#4146) (SciTools#4368)
  Updated environment lockfiles (SciTools#4379)
  Prevent warning in `test_Saver` (SciTools#4376)
  drop pyugrid in site.cfg (SciTools#4373)
  `flake8` dependency (SciTools#4371)
  update latest whosnew (SciTools#4372)
  Allow `check_graphic` to be more flexible (SciTools#4370)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4365)
  Updated environment lockfiles (SciTools#4364)
  Update latest.rst (SciTools#4362)
  More clarity on setting `iris-test-data` location. (SciTools#4359)
  update whatsnew (SciTools#4361)
  ...
@bjlittle bjlittle deleted the nep29-numpy branch January 10, 2022 08:29
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