ESMValGroup / ESMValTool

ESMValTool: A community diagnostic and performance metrics tool for routine evaluation of Earth system models in CMIP
https://www.esmvaltool.org
Apache License 2.0
210 stars 122 forks source link

Documentation build is failing #3638

Open bouweandela opened 1 month ago

bouweandela commented 1 month ago
WARNING:psyplot.config.rcsetup:Failed to import EntryPoint(name='plugin', value='psy_reg.plugin', group='psyplot')!
WARNING:psyplot:Could not import EntryPoint(name='plugin', value='psy_reg.plugin', group='psyplot')
Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/esmvaltool/conda/3628/lib/python3.11/site-packages/psyplot/__init__.py", line 132, in get_versions
    mod = ep.load()
          ^^^^^^^^^
  File "/home/docs/checkouts/readthedocs.org/user_builds/esmvaltool/conda/3628/lib/python3.11/site-packages/importlib_metadata/__init__.py", line 184, in load
    module = import_module(match.group('module'))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/docs/checkouts/readthedocs.org/user_builds/esmvaltool/conda/3628/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/docs/checkouts/readthedocs.org/user_builds/esmvaltool/conda/3628/lib/python3.11/site-packages/psy_reg/plugin.py", line 36, in <module>
    from psyplot.compat.pycompat import map, filter
ModuleNotFoundError: No module named 'psyplot.compat'

Seen when building the documentation in #3628 (https://readthedocs.org/projects/esmvaltool/builds/24538250/).

bouweandela commented 1 month ago

Related to https://github.com/psyplot/psyplot/issues/62

bouweandela commented 2 weeks ago

Looks like it started working again: https://readthedocs.org/projects/esmvaltool/builds/24673573/.

valeriupredoi commented 1 week ago

nope it did not, with a magic combo of:

(esmvaltool) valeriu@valeriu-PORTEGE-Z30-C:~/ESMValTool$ conda list psy
# packages in environment at /home/valeriu/miniconda3/envs/esmvaltool:
#
# Name                    Version                   Build  Channel
psy-maps                  1.5.0           py311h38be061_1    conda-forge
psy-reg                   1.4.0           py311h38be061_3    conda-forge
psy-simple                1.5.0           py311h38be061_1    conda-forge
psyplot                   1.5.0           py311h38be061_1    conda-forge

it die, like before. @Chilipp and meself are pushing to get psy-reg to 1.5.0 so it supports Python 3.12 too, but until then, it's a bit of a faff :beer:

bouweandela commented 1 week ago

The error message is still there, but the documentation build does not seem to fail because of it. In https://github.com/ESMValGroup/ESMValTool/pull/3628, I thought the build was failing because of this issue here, but it was actually another problem, which I subsequently fixed in #3675.

valeriupredoi commented 1 week ago

Bizarre - that's a serious import error, and RTD detects it, why wouldn't it treat it as a breakage. At any rate, feel free to re-close the issue, bud - but I'd still argue we ought to fix the diagnostic ie via #3687