-
Notifications
You must be signed in to change notification settings - Fork 128
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
obs4MIPs dataset and version are outdated in recipes #2974
Comments
@ESMValGroup/esmvaltool-coreteam I could make a pull request to update those recipes, if there are no objections? |
Thanks for opening the discussion @bouweandela. I think it would be good to align our recipes with the current specifications. Do I understand it correctly that we would not necessarily specify the obs4MIPs dataset versions (i.e. |
Yes
Our broken recipe policy states that recipes can use only data that is publicly available, so recipes that use data that is not or no longer publicly available will need to be updated or marked as broken. To get a better overview of the issue, I created the following table. Here is a mapping table from datasets and versions currently used in recipes to ESGF datasets:
Note that the And here is a list of affected recipes: recipe_smpi.yml #2991 recipe_perfmetrics_CMIP5.yml recipe_ecs_scatter.yml recipe_quantilebias.yml recipe_clouds_bias.yml recipe_schlund20esd.yml recipe_flato13ipcc.yml recipe_perfmetrics_CMIP5_4cds.yml recipe_wenzel16jclim.yml recipe_lauer13jclim.yml recipe_cmug_h2o.yml recipe_ecs_constraints.yml recipe_autoassess_landsurface_surfrad.yml recipe_smpi_4cds.yml recipe_validation.yml #3002 recipe_clouds_ipcc.yml recipe_ocean_quadmap.yml recipe_deangelis15nat.yml recipe_perfmetrics_land_CMIP5.yml bock20jgr/recipe_bock20jgr_fig_6-7.yml bock20jgr/recipe_bock20jgr_fig_8-10.yml bock20jgr/recipe_bock20jgr_fig_1-4.yml ipccwg1ar6ch3/recipe_ipccwg1ar6ch3_atmosphere.yml recipe_radiation_budget |
@remi-kazeroni Would you prefer a single pull request with all updates, or a pull request per recipe? And what about the outdated |
I would prefer to have one pull request per recipe or group of recipes. There are 26 recipes in this list. I think that if we have a single PR for all recipes and need to wait for all recipes to be reviewed by their maintainers/experienced users, this would probably take too long for the release. It would be a good start if we could change a few of these recipes for v2.8. Also, I now realize that this could potentially have a large impact on our obs4MIPs data pool at DKRZ which has more than 3000 files (90 GB) and which are shipped to JASMIN. Given that these data pools can be used by recipe developers or users of previous versions of the ESMValTool modules, I do not plan to make changes to these data pools. Instead, I am thinking to benefit from the auto-download feature of the Tool to generate a new pool of obs4MIPs data at DKRZ. This could be in our shared
I'm not really knowledgeable about this facet and would suggest to ask advice from the @ESMValGroup/scientific-lead-development-team. |
Sounds good to me! If you're concerned about the file size and you're sure it's the same file, you could also create some symlinks so the directory with the new version name points to the directory containing the old data. |
It looks to me that it is getting a bit late in the release process to start a major update of the OBS4MIPs entries in our recipe. I would prefer to focus on fixing broken recipes, properly documenting the release and retesting recipes, rather than this work. I think it would be great to "ESGFize" our recipes so that ESMValTool could download more data automatically and the tool could be more easily ported on different clusters and data pools. But this would take more than the one week we have left for v2.8. I'd be happy to contribute to these efforts once the release is finished. @bouweandela, would it be agreeable to bump this issue and related PRs to v2.9? |
Hi All, I encountered that issue trying to set up ESMValTool on our system here at ACCESS-NRI. |
Hey @rbeucher , I assume you're aware of |
Hi @hot007, Yes I am. I think that using a pool of data aligned with the ESMValTool group is probably easier to get for now... |
@bouweandela @remi-kazeroni . What do you do for derived variables like |
Hi @rbeucher, thanks for your interest in the Tool. My recommendation to set up an pool of obs4MIPs data would be to create it by using the automatic download from ESGF. The problem here is that quite a few of our obs4MIPs entries in recipes do not follow ESGF standards but older, outdated ESGF ones. In many cases, that can be fixed by changing the dataset name (as explained in this #2974 (comment)) and removing the But I suspect we may first need to clarify |
Happy to help with this. It affects quite a lot of recipes so I think it is important we solve the issue. |
Pull requests with updates would be most welcome! You may want to have a look at our contribution guidelines to get more familiar with the process. |
Sure. I'll submit PRs. I'm looking into it. |
Thanks for the massive effort @rbeucher! With these changes, there are 10 recipes remaining that are still affected by this issue. I'll re-open the issue so we can keep track of it. @ESMValGroup/esmvaltool-recipe-maintainers If your recipe is listed in this comment, could you please update your recipes according to the table above? #2974 (comment)
|
massive massive good work from @rbeucher - very many thanks! This can moved to M2.11 since there are still a few open PRs - even though those are nearly ready, am fairly sure the release boys @zklaus and @bouweandela would prefer to release this week before 🎅 comes along |
Hi, we are currently working on the ESMValTool release for v2.11.0. We're wondering if you'd be able to finalise this issue by the end of next week (Friday 10th May). Otherwise, please let us know, and we'll move it into the next milestone for you 🙂 |
Many of the recipes in ESMValTool use "legacy" names for the dataset and version number of obs4MIPs datasets. The obs4MIPs specification is from 2017, so the naming scheme in those recipes is considered "legacy" for over 5 years now. I think it's time we update those recipes so they are in line with the current specification. Updating has the advantage that downloading the data automatically from ESGF will actually work.
The text was updated successfully, but these errors were encountered: