ESMValGroup / ESMValTool

ESMValTool: A community diagnostic and performance metrics tool for routine evaluation of Earth system models in CMIP
https://www.esmvaltool.org
Apache License 2.0
217 stars 126 forks source link

Update proj in conda environment, see #3214 #3448

Closed rbeucher closed 9 months ago

rbeucher commented 9 months ago

Description

The cmoriser for NSDIC fails (see #3214). The problem has been solved for proj > 9.3. The conda environment should be fixed


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.

New or updated recipe/diagnostic

New or updated data reformatting script


To help with the number of pull requests:

valeriupredoi commented 9 months ago

wow that CircleCI link did not format nicely at all

valeriupredoi commented 9 months ago

OK @rbeucher here's the deal: mamba refuses to solve the environment even if one grabs the latest conda=23.10 and mamba=1.5.3. On the flipside though, proj=9.3 can be installed post-facto, after the env has been created; the env gets altered heavily with mamba updating about 85 dependencies and changing builds for a good 50 others (most striking is changing from nompi to mpich builds, which is not necessarily evil but not good either), however, the tool installs and behaves well. So how about we just mention that eg "to get your NSDIC cmorization work done please upgrade proj to such and such" for now; once our env has been updated, and proj=9.3 is a bit more mature, we'll have it in no doubt. Which also made me gasp a little, @bouweandela and @zklaus - we have a set of fairly old dependencencies in our env, some stuff that's 2 or 3 minor versions below the latest, mambaforge being stuck at 2 clicks older is not helping, but can we maybe update the Circle conda/mamba? I'll have a look at that. Also, am hoping we can get Python 3.12 soon, so we modernize proper (am working on that on multiple fronts) :+1:

rbeucher commented 9 months ago

OK sure. Works for me

valeriupredoi commented 9 months ago

want to add that bit of info to the docs, @rbeucher, and close this then? :beer:

rbeucher commented 9 months ago

Yes I will add that to the doc. Thanks @valeriupredoi

rbeucher commented 9 months ago

See #3457 Closing this one for now