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

Documentation build is failing #3638

Closed
bouweandela opened this issue May 30, 2024 · 6 comments
Closed

Documentation build is failing #3638

bouweandela opened this issue May 30, 2024 · 6 comments

Comments

@bouweandela
Copy link
Member

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
Copy link
Member Author

Related to psyplot/psyplot#62

@bouweandela
Copy link
Member Author

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

@valeriupredoi
Copy link
Contributor

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 🍺

@bouweandela
Copy link
Member Author

The error message is still there, but the documentation build does not seem to fail because of it. In #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
Copy link
Contributor

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

@Chilipp
Copy link

Chilipp commented Jul 17, 2024

FYI: the PR of psy-reg v1.5.0 (conda-forge/psy-reg-feedstock#19) has been merged, so it should be available soon via the conda-forge channel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants