-
Notifications
You must be signed in to change notification settings - Fork 39
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
Code freeze for v2.2.0 #972
Comments
@jvegasbsc I see you are more lenient than @bouweandela and myself for last release, we decided to freeze it in the afternoon 😁 For the freeze, you know the drill - create a branch called something like |
Branch |
I don't think I have the right to do that. I can't access the settings of the repo. My guess is that only @axel-lauer can do that at the moment. |
Done! |
cheers @axel-lauer - for the sake of redundancy it'd be very good if you allowed @remi-kazeroni and at least another core member privileges for these types of operations (of your choice, could be @bouweandela , myself or @jvegasbsc ) 👍 |
@valeriupredoi good point. I propose to give @remi-kazeroni as "successor" of @mattiarighi admin rights right now but to leave it up to the new steering committee to decide on how admin rights are handled in the future. |
Please, @ESMValGroup/esmvaltool-developmentteam go ahead and test the branch |
cheers @axel-lauer - it all sounds very reasonable to me 🍺 |
@ESMValGroup/esmvaltool-developmentteam to be able to test, the easiest way is to install the ## first install esmvalcore
cd ESMValCore
git fetch -v
git checkout release_2.2.0
conda env create -n test_core_release -f environment.yml
conda activate test_core_release
pip install -e .[develop]
## then install esmvaltool in development
cd ESMValTool
git fetch -v
git checkout iris3
## comment out the "- esmvalcore>=2.1.0,<2.2" line in environment.yml
conda env update -n test_core_release -f environment.yml
conda activate test_core_release
pip install -e .[develop] |
I'm getting the following error now when installing the tool in development mode with the current
|
this is a common hurdle when installing numba, but we've got rid of it, all goes fine on my machine, can you pls tell me which wheel is building while this is being executed? (add a few more lines of stdout before the error kicks in) |
I'm repeating the installation with the |
OK I have replicated your issue - and that's because you should always read screen stdout 🔫 - installing ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
esmvaltool 2.1.1 requires esmvalcore<2.2,>=2.1.0, but you have esmvalcore 2.2.0 which is incompatible. and it rightly does, then going further to install ESMValTool results in your error |
It doesn't say it for me, I swear 😬 Here's the full log: Just got the same error with the release branch. Apparently it tries to install I did the following:
The last command results in the error... |
you missing a couple steps: |
No, the core needs to be installed after the tool, otherwise you end up with the pip package and not the development version (at least it did this some months ago). I'm installing the tool like this for months now, it always worked |
(This is also the order given by the documentation: https://docs.esmvaltool.org/en/latest/quickstart/installation.html) |
why do you think you're commenting out the environment line before updating it , Sherlock 😁 Trust me, am an engineer 🔨 |
this is an unorthodox manner, the documentation has its limitations but they're not exposed due to the workflow following a logical mainstream path. This type of installation as we are doing here is highly dubious if a user would have to do it on a regular basis |
also note that if you installed it after, in pip devel mode, with esmvalcore initially being downloaded and installed from conda, you will run in a hefty dependency issue with iris3 not bein installed, and you'd have to update the env from core again, but that'll get you in a catch22 because esmvaltool will not be compatible with some deps - just follow my instructions and we'll all be happy 😁 |
Okay, now I get your point (and why you commented out esmvalcore in the environment.yml). I agree that this is okay for testing purposes now, but shouldn't we have two branches (in the ideal case the two |
yep, the current core-master is at odds with tool-master coz of the upping of version number in core (something that @jvegasbsc pointed out well and I agree with). For now consider the core-master to be |
Okay, apparently the installation with my (traditional) method works just fine with the You should be careful with the tips you're giving, @valeriupredoi 😂 |
tips-shnips, of course it works, coz I realized I had just committed the pinning for iris3 this morning in the |
But the conda esmvalcore environment IS built for iris3, no? Line 10 in fd70226
|
@jvegasbsc @valeriupredoi Could one of you check the problems reported by @remi-kazeroni please? |
@bouweandela @remi-kazeroni on me to-do list, will do it in a bit 👍 |
I will check this evening, it's being a crazy day |
@remi-kazeroni the second issue is not impacting us - The second issue (first, in your post) is a bit more tricksy - I get it too, but in a slightly different shape (I am running with --skp_nonexistent), in my case the tool is trying to run concatenation on 0-len cubes list, will investigate a bit more... |
OK I think I know what the problem with the 0-len cube list is - |
OK - I dug deeper in the fiona issue: it seems that the pypi fiona (that we install currently) has been built with gdal=3 earlier that still had that dud file, installing it from conda-forge completely solves the issue, I am not going to open an issue on the fiona github page since last time I did that I had to haggle and sneer at them to actually pay correct attention to it and fix it, we'll just have to install it from conda-forge. There was a note in the setup file I left saying that we'd eventually have to install fiona from conda since they kinda gave up uploading wheels on pypi, so might as well we do it now |
@remi-kazeroni it'd be best if you opened an issue about this supplying the command you ran (if you used or not running with esmvaltool=2.1.1 and esmvalcore=2.1.0 (off the JASMIN central installation): esmvaltool run examples/recipe_preprocessor_derive_test.yml --skip_nonexistent returns:
This is much better than what we have now, since the current codebase is actually attempting derivation (hence going and trying to concatenate). In any case, it's just a matter of understanding where the stop point has moved, and outputting a reasonable error message, but I'm not gonna try fix that on a Friday evening and it's really a corner case that would fail anyway, so no reason to halt the release 👍 🍺 |
I will look into it. I probably messed it up when changing the error messages for the missing data |
I am also having trouble getting an environment installed. I followed @valeriupredoi 's instructions (except for adding quotes to the pip commands) but no success. pip install for ESMValTool gives me this error
I also tried various other ways to create a new environment but all attempts have failed. I think by now I tried everything that comes to my mind. Why is it so complicated to create a new environment for testing v2.2? |
@axel-lauer the trace tells me you are using |
also pls make sure you are on |
I just repeated @valeriupredoi's instructions
and got a successful installation of the tool (I tried One question though: Does this ensure that the usual installation in development mode works after the release of ESMValCore the and the merge of the |
cheers Manu! yes, the usual development installation procedure will work fine after the release provided that we make sure the |
also note that the |
Will do so. I also had the |
I just tried to follows @schlunma's steps using a brand new anaconda installation. Everything seems to work fine until the second "pip install":
I am close to giving up. This is a nightmare. |
ESMPy is provided by the ESMF package from conda-forge, that should be installed when you use the environment.yml file from ESMValCore to create or update the environment (you can check that it is listed there). |
@axel-lauer for the sake of sanity, I have repeated the exact same steps @schlunma posted above and the ones you followed too; I have no issues installing ESMValTool (Manu - side note, having looked closer at the commands - I recommend not mixing |
@valeriupredoi just ran into the same problem with a fresh miniconda installation. Tomorrow I'll try switching from tcsh to bash. Maybe that does the trick... |
@axel-lauer it might be that your |
BTW it'd be nice if you could document the fixes you apply and the steps you take to make it work - don't worry about this particular case with conda update etc, just try and install esmvalcore in pure development mode then esmvaltool, and make sure all deps are there where they should be - this way we can document the procedure and include it in the documentation for the next oddball that's using |
BTW the main reason I'm using |
@axel-lauer Sometimes conda says it activates the environment, but it actually doesn't. You could try running |
Version 2.2.0 publish on both Conda and PyPI! |
Can any of you delete the branch |
Just deleted branch release_2.2.0. |
Today we freeze the code for the 2.2.0 release. I ask for your help to review and merge as many pending issues in the milestone before today at 22:00 CES. Thanks!
@ESMValGroup/scientific-lead-development-team
@ESMValGroup/technical-lead-development-team
The text was updated successfully, but these errors were encountered: