Read config-user.yml using esmvalcore.config module

[bouweandela]

Description

Reduce the use of private functions from ESMValCore.

Partial solution to #1975.

---

Before you get started

• ☝ Create an issue to discuss what you are going to do

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.

• 🛠 This pull request has a descriptive title
• 🛠 Code is written according to the code quality guidelines
• 🛠 Tests run successfully
• 🛠 The list of authors is up to date
• 🛠 All checks below this pull request were successful

---

To help with the number of pull requests:

• 🙏 We kindly ask you to review two other open pull requests in this repository

[remi-kazeroni]

Hi @bouweandela, would this be something to consider for the v2.8 release? Otherwise, we will continue to get the deprecation warning with the released version:

ESMValCore/esmvalcore/_config/__init__.py:13: ESMValCoreDeprecationWarning: The private module `esmvalcore._config` has been deprecated in ESMValCore version 2.8.0 and is scheduled for removal in version 2.9.0. Please use the public module `esmvalcore.config` instead.

[remi-kazeroni]

@esmvalbot Please run examples/recipe_python.yml

[esmvalbot]

Since @remi-kazeroni asked, ESMValBot will run recipe examples/recipe_python.yml as soon as possible, output will be generated here

[esmvalbot]

ESMValBot is happy to report recipe examples/recipe_python.yml ran OK, output has been generated here

[remi-kazeroni]

Thanks for fixing this @bouweandela. Deprecation warnings no longer show up unless one uses deprecated options in their config_file. I tried a couple of recipes and CMORizers and all looks good.

Is it also planned to remove import esmvalcore._config from the tests in the Tool? This seems to be the case in tests/integration/test_recipes_loading.py and tests/system/data_simulator.py.

[bouweandela]

Let's do that at a later point. The test tests/integration/test_recipes_loading.py is written to work with the old and the new version, but it has become a bit of a mess due to this. tests/system/data_simulator.py is more of an idea than something that is actually used.

[remi-kazeroni]

@ESMValGroup/technical-lead-development-team could someone have a final look and merge that if all good?

[schlunma]

Thanks both! Bouwe - would you mind opening an issue about updating these two files you mentioned? Thanks!

[bouweandela]

Cf. #3109