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

docs build failing on Pygments lexter warning #2067

Closed
matthewfeickert opened this issue Nov 29, 2022 · 2 comments · Fixed by #2068
Closed

docs build failing on Pygments lexter warning #2067

matthewfeickert opened this issue Nov 29, 2022 · 2 comments · Fixed by #2068
Assignees
Labels
bug Something isn't working docs Documentation related

Comments

@matthewfeickert
Copy link
Member

Hm. Something related to spatialaudio/nbsphinx#24 is breaking the docs build. We're getting

WARNING: Pygments lexer name 'ipython3' is not known

for all the notebooks during the docs build and we fail on warnings.

Originally posted by @matthewfeickert in #2066 (comment)

@matthewfeickert
Copy link
Member Author

matthewfeickert commented Nov 29, 2022

Given that

$ python -m pip show Pygments | grep Required-by
Required-by: ipython, jupyter-console, nbconvert, nbdime, qtconsole, readme-renderer, rich, Sphinx

and between the last working build of the docs on 2022-11-17 and now (2022-11-28) the difference in installed dependencies includes

$ diff --color -u /tmp/pass.txt /tmp/fail.txt
...
-ipykernel                     6.17.1
-ipython                       8.6.0
+ipykernel                     6.18.1
+ipython                       8.7.0

I'm going to assume there is something different with ipython and look there.

@matthewfeickert matthewfeickert added bug Something isn't working docs Documentation related labels Nov 29, 2022
@matthewfeickert
Copy link
Member Author

matthewfeickert commented Nov 29, 2022

This shows up in ipython/ipython#13845 as well, so I think that this is going to exist for a bit and that PR #2066 should get merged in while this all gets figured out.

It does seem that ipython/ipython#13845 (comment) is correct though about

this is due to the Pygments entry points not being correctly installed with 8.6.0 8.7.0 as

$ python -m pip list | grep 'setuptools\|ipython'                                                
ipython                       8.7.0
ipython-genutils              0.2.0
setuptools                    65.6.3
$ python -c 'from importlib.metadata import entry_points; print(entry_points(group="pygments.lexers"))'
[]
$ python -m pip install --upgrade 'ipython==8.6.0'
$ python -m pip list | grep 'setuptools\|ipython'                                                                                                         
ipython                       8.6.0
ipython-genutils              0.2.0
setuptools                    65.6.3
$ python -c 'from importlib.metadata import entry_points; print(entry_points(group="pygments.lexers"))'                                                   
[EntryPoint(name='ipython', value='IPython.lib.lexers:IPythonLexer', group='pygments.lexers'), EntryPoint(name='ipython3', value='IPython.lib.lexers:IPython3Lexer', group='pygments.lexers'), EntryPoint(name='ipythonconsole', value='IPython.lib.lexers:IPythonConsoleLexer', group='pygments.lexers')]

so to get around this just disallow ipython v8.7.0 for the time being

ipython!=8.7.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working docs Documentation related
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant