Fetch ESMValTool recipes in advance#582
Fetch ESMValTool recipes in advance#582bouweandela wants to merge 5 commits intoClimate-REF:mainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
| base_url=DATASET_URL, | ||
| package="climate_ref_esmvaltool.dataset_registry", | ||
| resource="data.txt", | ||
| cache_name=f"climate_ref_esmvaltool/{_DATASETS_REGISTRY_NAME}", |
There was a problem hiding this comment.
This will require redownloading the datasets. I'm not 100% happy with how we handle the various caches, but that is something for another day.
There was a problem hiding this comment.
If you prefer to keep everything in one folder that's also fine with me, might be easier to configure for users. We may want to use a subfolder per provider in that case, to avoid that providers accidentally overwrite each others files.
There was a problem hiding this comment.
Updated the code in 65f9ed9 so all caches live in subdirectories under a single climate_ref cache directory.
|
I think this is ready, but I'll have to have a more careful think. I'll get this in the next release |
2 participants
Description
Closes #468
Checklist
Please confirm that this pull request has done the following:
changelog/