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

Remove the deprecated option use_legacy_supplementaries #2202

Merged
merged 4 commits into from
Sep 28, 2023

Conversation

bouweandela
Copy link
Member

@bouweandela bouweandela commented Sep 22, 2023

Description

Remove the deprecated option use_legacy_supplementaries, scheduled for removal in v2.10.0 (see #1609 (comment)).

Upgrade advice

The recommended upgrade procedure is to remove use_legacy_supplementaries from config-user.yml (if it was there) and remove any mention of fx_variables from the recipe. If automatically defining the required supplementary variables does not work, define them in the variable or (additional_)datasets section as described in the documentation.


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.


To help with the number pull requests:

@codecov
Copy link

codecov bot commented Sep 22, 2023

Codecov Report

Merging #2202 (c57cb32) into main (10bd776) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2202      +/-   ##
==========================================
- Coverage   93.14%   93.06%   -0.09%     
==========================================
  Files         237      237              
  Lines       12831    12654     -177     
==========================================
- Hits        11951    11776     -175     
+ Misses        880      878       -2     
Files Coverage Δ
esmvalcore/_recipe/recipe.py 98.99% <100.00%> (-0.02%) ⬇️
esmvalcore/_recipe/to_datasets.py 100.00% <100.00%> (ø)
esmvalcore/config/_config_validators.py 97.45% <ø> (-0.10%) ⬇️
esmvalcore/preprocessor/__init__.py 95.23% <ø> (ø)
esmvalcore/preprocessor/_supplementary_vars.py 97.67% <ø> (-0.39%) ⬇️

@bouweandela bouweandela marked this pull request as ready for review September 28, 2023 09:47
@bouweandela bouweandela changed the title Remove the deprecated option use_legacy_supplementaries Remove the deprecated option use_legacy_supplementaries Sep 28, 2023
Copy link
Contributor

@valeriupredoi valeriupredoi left a comment

Choose a reason for hiding this comment

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

is ESMValCore so light now that it can ✈️ ? I'd see about CMIP7 project, bud 😁

tests/integration/recipe/test_recipe.py Show resolved Hide resolved
@bouweandela
Copy link
Member Author

Thanks for reviewing!

@zklaus zklaus merged commit 925fce3 into main Sep 28, 2023
@zklaus zklaus deleted the remove-use-legacy-supplementaries branch September 28, 2023 12:53
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.

3 participants