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

Update environment: pin iris>=3.11, unpin cartopy and allow for numpy >=2 #3811

Merged
merged 8 commits into from
Nov 14, 2024

Conversation

valeriupredoi
Copy link
Contributor

@valeriupredoi valeriupredoi commented Nov 13, 2024

Description

  • We are now using iris=3.11 which is what we need to be able to unpin cartopy, see @schlunma 's comment Unpin cartopy #3769 (comment)
  • to not risk getting cartopy==0.24 with an older iris eg iris==3.10, and hence get the bug we pinned cartopy for in the first place, we should pin iris>=3.11
  • iris==3.11 works well with numpy>=2 so we remove the upper pin in our own environment; note, however, that numpy2 is not yet available to us, due to ncl (that holds esmpy from going past 8.6, and makes us keep that silly importlib_metadata<8 pin, see Pin importlib_metadata <8 #3699)

Closes #3769
Contributes to #3812


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

@valeriupredoi valeriupredoi changed the title Update environment (unpin cartopy and numpy >=2 now allowed) Update environment: unpin cartopy and allow for numpy >=2 Nov 14, 2024
@valeriupredoi valeriupredoi changed the title Update environment: unpin cartopy and allow for numpy >=2 Update environment: pin iris>=3.11, unpin cartopy and allow for numpy >=2 Nov 14, 2024
Copy link
Contributor

@schlunma schlunma left a comment

Choose a reason for hiding this comment

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

Thanks V, just have a couple of minor comments.

environment.yml Outdated Show resolved Hide resolved
environment.yml Show resolved Hide resolved
environment_osx.yml Show resolved Hide resolved
valeriupredoi and others added 2 commits November 14, 2024 16:04
Co-authored-by: Manuel Schlund <32543114+schlunma@users.noreply.github.com>
@valeriupredoi
Copy link
Contributor Author

cheers for the review, Manu 🍺

@schlunma schlunma added this to the v2.12.0 milestone Nov 14, 2024
Copy link
Contributor

@schlunma schlunma left a comment

Choose a reason for hiding this comment

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

Thanks!!

@valeriupredoi valeriupredoi merged commit de43833 into main Nov 14, 2024
8 checks passed
@valeriupredoi valeriupredoi deleted the update_environment_Nov24 branch November 14, 2024 16:35
ehogan added a commit that referenced this pull request Nov 19, 2024
* main: (31 commits)
  Update environment: pin `iris>=3.11`, unpin `cartopy` and allow for `numpy >=2` (#3811)
  Fix issue related to removal/change of private function imported in `diag_scripts/shared/_supermeans.py` (deprecation in iris=3.11) (#3810)
  Remove recipe filler utility (#3777)
  [Condalock] Update Linux condalock file (#3809)
  change authors name (#3806)
  [Condalock] Update Linux condalock file (#3798)
  Fix contourf plots for masked data (#3797)
  [Condalock] Update Linux condalock file (#3796)
  Add next release schedule (#3794)
  Use `transform_first=True` for contourf plots with Robinson projection to avoid cartopy bug (#3789)
  Pin pys2index >=0.1.5 in osx environment (#3792)
  Adding a CMORiser for CMAP data for pr (#3766)
  Adding pr, tauu, tauv, tos to NCEP2 CMORISer (#3765)
  Readthedocs configuration/builds: revert to miniconda before miniforge is available (#3785)
  Adapt ESMValTool to new configuration (#3761)
  Adding pr, tauu, tauv NOAA-CIRES-20CR-V2 CMORISER (#3763)
  update comment in conda lock creation Github action (#3788)
  [Condalock] Update Linux condalock file (#3786)
  update Docker builds badge in README (#3783)
  Pin mamba<2 for conda-lock: solution by Ben Mares @maresb (#3771)
  ...
ehogan added a commit that referenced this pull request Nov 19, 2024
…Tool into 3723_remove_rose_cylc

* '3723_remove_rose_cylc' of github.com:ESMValGroup/ESMValTool: (31 commits)
  Update environment: pin `iris>=3.11`, unpin `cartopy` and allow for `numpy >=2` (#3811)
  Fix issue related to removal/change of private function imported in `diag_scripts/shared/_supermeans.py` (deprecation in iris=3.11) (#3810)
  Remove recipe filler utility (#3777)
  [Condalock] Update Linux condalock file (#3809)
  change authors name (#3806)
  [Condalock] Update Linux condalock file (#3798)
  Fix contourf plots for masked data (#3797)
  [Condalock] Update Linux condalock file (#3796)
  Add next release schedule (#3794)
  Use `transform_first=True` for contourf plots with Robinson projection to avoid cartopy bug (#3789)
  Pin pys2index >=0.1.5 in osx environment (#3792)
  Adding a CMORiser for CMAP data for pr (#3766)
  Adding pr, tauu, tauv, tos to NCEP2 CMORISer (#3765)
  Readthedocs configuration/builds: revert to miniconda before miniforge is available (#3785)
  Adapt ESMValTool to new configuration (#3761)
  Adding pr, tauu, tauv NOAA-CIRES-20CR-V2 CMORISER (#3763)
  update comment in conda lock creation Github action (#3788)
  [Condalock] Update Linux condalock file (#3786)
  update Docker builds badge in README (#3783)
  Pin mamba<2 for conda-lock: solution by Ben Mares @maresb (#3771)
  ...
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.

Unpin cartopy
2 participants