Options write_plots and write_netcdf will be removed in ESMValCore v2.4

[bouweandela]

The options in config-user.yml write_plots and write_netcdf are deprecated in ESMValCore v2.2 and will be removed in v2.4. See ESMValGroup/ESMValCore#93 and ESMValGroup/ESMValCore#808 for previous decisions on the topic.

This means that for those diagnostic scripts that support these options, they will have to be moved to the diagnostic script settings in the recipe.

@ESMValGroup/tech-reviewers Please keep this in mind when reviewing new recipes.

@ESMValGroup/esmvaltool-recipe-maintainers Please update your recipes before the ESMValTool v2.4 release (likely somewhere in autumn 2021).

[stefsmeets]

Note that the shared submodule also has instances of cfg['write_netcdf'] / cfg['write_plots'] which will cause the tool to crash with KeyError if these keys are not available in the config-user.yml.

[bouweandela]

Note that the shared submodule also has instances of cfg['write_netcdf'] / cfg['write_plots'] which will cause the tool to crash with KeyError if these keys are not available in the config-user.yml.

That should be fixed by #2018.

[bouweandela]

@ESMValGroup/esmvaltool-recipe-maintainers Please note that this feature will be removed in the upcoming v2.4 release planned for October. Please update your recipes and their documentation if they support the write_netcdf or write_plots setting (or even require it to run successfully).

[axel-lauer]

Just opened a new pull request that removes all instances of "write_plots" and "write_netcdf" from all NCL diagnostics that were still supporting these options: #2331

[valeriupredoi]

cheers @axel-lauer - I can review it techy-wise if you want me to 😁

[axel-lauer]

@valeriupredoi Sure!