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.

Sign up for GitHub

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

Jump to bottom

IPython installs without a lexar entry point for pygments #1430

Open
ResidentMario opened this issue Feb 7, 2017 · 3 comments
Open

IPython installs without a lexar entry point for pygments #1430

ResidentMario opened this issue Feb 7, 2017 · 3 comments

Comments

@ResidentMario
Copy link

I had an issue with the nbconvert component of jupyter, whereupon my code blocks were "swallowed" upon emission to HTML in sphinx. I filed this as an issue over at nbconvert, and it was fixed by running:

conda update ipython -c conda-forge

E.g. moving ipython over to the conda-forge build.

@takluyver hypothesizes this is because of a configuration miss on the default channel:

I don't think it's an issue with pygments, but the IPython package from the default channel appears to be built in a way that doesn't install the entry points (which are how we register lexers for pygments). In conda-forge, where we control the ipython package, it is built using pip, which works.

For more details see the bug report and fix over @ nbconvert.
This was referenced Feb 7, 2017
Closed
Closed
@takluyver
Copy link

I'd be happy for Anaconda to use the conda-forge recipe for IPython, if that's easy for you.

@marscher marscher mentioned this issue Mar 2, 2017
Closed
@mgeier mgeier mentioned this issue Mar 23, 2017
Merged
danking pushed a commit to danking/hail that referenced this issue Apr 19, 2017
work around a bug in conda
8b32af2 
Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430
danking pushed a commit to danking/hail that referenced this issue Apr 19, 2017
work around a bug in conda
3c64207 
Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430
tpoterba pushed a commit to hail-is/hail that referenced this issue Apr 19, 2017
@danking @tpoterba
Fix all docs warnings (#1693)
a205789 
* eliminate warning in getting_started.rst

* all warnings are errors

* use anonymous links instead of named links

* fix indentation of list item

* fix a variety of list item syntax issues

* remove illegal excess indentation

* several list item syntax fixes

* variety of list item fixes and a link fix

* remove extraneous character from string [] docs

* favicons must end in ico

* work around a bug in conda

Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430

* fix link to tutorial

* fix broken links

* remove broken link to sample

* remove more broken links to sample

* remove references to sec-objects

* fix bad reference syntax

* two small fixes

1. Remove reference to sample
2. Move ico to correct directory

* remove unnecessary html_extra_path entry

* fix doctest skip syntax
cseed pushed a commit to hail-is/hail that referenced this issue Apr 21, 2017
@danking @cseed
in docs, improve three-way linking between type and representations (#…
30016eb 
…1691)

* eliminate warning in getting_started.rst

* all warnings are errors

* use anonymous links instead of named links

* fix indentation of list item

* fix a variety of list item syntax issues

* remove illegal excess indentation

* several list item syntax fixes

* variety of list item fixes and a link fix

* remove extraneous character from string [] docs

* favicons must end in ico

* work around a bug in conda

Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430

* fix link to tutorial

* fix broken links

* remove broken link to sample

* remove more broken links to sample

* remove references to sec-objects

* fix bad reference syntax

* two small fixes

1. Remove reference to sample
2. Move ico to correct directory

* remove unnecessary html_extra_path entry

* initial attempt

* add missing file

* improve hailType blurb

* further improvements from tpoterba

* Update hailType.rst

* sphinx lists must not have leading space

This triggers an error message:

    Error in "include" directive: no content permitted.

because the one-space-indented text is assumed to be an argument to the
`.. include` directive

Also fix location of hailType.rst

* use anonymous links instead of named links

* simplify docs
@tshead2 tshead2 mentioned this issue Apr 28, 2017
Closed
liameabbott pushed a commit to liameabbott/hail that referenced this issue May 3, 2017
@danking @liameabbott
Fix all docs warnings (hail-is#1693)
f20d23a 
* eliminate warning in getting_started.rst

* all warnings are errors

* use anonymous links instead of named links

* fix indentation of list item

* fix a variety of list item syntax issues

* remove illegal excess indentation

* several list item syntax fixes

* variety of list item fixes and a link fix

* remove extraneous character from string [] docs

* favicons must end in ico

* work around a bug in conda

Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430

* fix link to tutorial

* fix broken links

* remove broken link to sample

* remove more broken links to sample

* remove references to sec-objects

* fix bad reference syntax

* two small fixes

1. Remove reference to sample
2. Move ico to correct directory

* remove unnecessary html_extra_path entry

* fix doctest skip syntax
liameabbott pushed a commit to liameabbott/hail that referenced this issue May 3, 2017
@danking @liameabbott
in docs, improve three-way linking between type and representations (h…
7a80ac5 
…ail-is#1691)

* eliminate warning in getting_started.rst

* all warnings are errors

* use anonymous links instead of named links

* fix indentation of list item

* fix a variety of list item syntax issues

* remove illegal excess indentation

* several list item syntax fixes

* variety of list item fixes and a link fix

* remove extraneous character from string [] docs

* favicons must end in ico

* work around a bug in conda

Discussion of issue:

  spatialaudio/nbsphinx#24 (comment)

Root cause:

  ContinuumIO/anaconda-issues#1430

* fix link to tutorial

* fix broken links

* remove broken link to sample

* remove more broken links to sample

* remove references to sec-objects

* fix bad reference syntax

* two small fixes

1. Remove reference to sample
2. Move ico to correct directory

* remove unnecessary html_extra_path entry

* initial attempt

* add missing file

* improve hailType blurb

* further improvements from tpoterba

* Update hailType.rst

* sphinx lists must not have leading space

This triggers an error message:

    Error in "include" directive: no content permitted.

because the one-space-indented text is assumed to be an argument to the
`.. include` directive

Also fix location of hailType.rst

* use anonymous links instead of named links

* simplify docs
@fperez
Copy link

fperez commented Sep 5, 2017

ping on this... I just ran into this same issue and the only workaround was to use the conda-forge channel as @takluyver suggested. It's a bummer that the default IPython package seems to have this problem; anything we can do to help?

@csoja
Copy link
Contributor

csoja commented Nov 16, 2017

@mingwandroid or @msarahan this should be fixed for the packages in main now, right?

@jgosmann jgosmann mentioned this issue Nov 28, 2017
Merged
8 tasks
leezu added a commit to leezu/gluon-nlp that referenced this issue Apr 28, 2018
@leezu
Install ipython from conda-forge
cae7480 
Workaround ContinuumIO/anaconda-issues#1430
szhengac pushed a commit to dmlc/gluon-nlp that referenced this issue Apr 29, 2018
@leezu @szhengac
Fix sphinx docs (#92)
d463407 
* Fix pygments lexers

`log` (used in beam_search_generator.rst) is not a valid lexer. `bash` is for
shell scripts. We are presenting interactive sessions and should use `console`.

* Add .ipynb_checkpoints to sphinx exclude_patterns

This is required: https://media.readthedocs.org/pdf/nbsphinx/latest/nbsphinx.pdf
Otherwise we will get a lot of warnings if the environment contains
ipynb_checkpoints..

* Declare development requirements

This may not be exhaustive yet, but are the requirements needed on my machine to
build the documentation.

* Make CI deploy stage fail if there are any Sphinx warnings

* Add 'release' dependency to 'docs' in Makefile

'docs' depends on .zip archives created in the 'release' step

* Declare word embedding evaluation extended_results.ipynb as orphan

Sphinx won't warn about orphan elements not being included in the TOC.

* Fix docstrings causing sphinx warnings

* Install ipython from conda-forge

Workaround ContinuumIO/anaconda-issues#1430
cgraywang pushed a commit to cgraywang/gluon-nlp-1 that referenced this issue May 2, 2018
@leezu @szhengac
Fix sphinx docs (dmlc#92)
56824c7 
* Fix pygments lexers

`log` (used in beam_search_generator.rst) is not a valid lexer. `bash` is for
shell scripts. We are presenting interactive sessions and should use `console`.

* Add .ipynb_checkpoints to sphinx exclude_patterns

This is required: https://media.readthedocs.org/pdf/nbsphinx/latest/nbsphinx.pdf
Otherwise we will get a lot of warnings if the environment contains
ipynb_checkpoints..

* Declare development requirements

This may not be exhaustive yet, but are the requirements needed on my machine to
build the documentation.

* Make CI deploy stage fail if there are any Sphinx warnings

* Add 'release' dependency to 'docs' in Makefile

'docs' depends on .zip archives created in the 'release' step

* Declare word embedding evaluation extended_results.ipynb as orphan

Sphinx won't warn about orphan elements not being included in the TOC.

* Fix docstrings causing sphinx warnings

* Install ipython from conda-forge

Workaround ContinuumIO/anaconda-issues#1430
bryanwweber added a commit to bryanwweber/thermostate that referenced this issue Oct 20, 2018
@bryanwweber
Remove IPython lexer workaround
2cb7128 
The issue described in
spatialaudio/nbsphinx#24 seems to be
resolved by ContinuumIO/anaconda-issues#1430
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@fperez @takluyver @ilanschnell @csoja @ResidentMario