Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Errors running notebooks with nbconvert 5.4? (after pip install nbsite) #84

Closed
ceball opened this issue Oct 11, 2018 · 2 comments · Fixed by #155
Closed

Errors running notebooks with nbconvert 5.4? (after pip install nbsite) #84

ceball opened this issue Oct 11, 2018 · 2 comments · Fixed by #155

Comments

@ceball
Copy link
Contributor

ceball commented Oct 11, 2018

See e.g. https://travis-ci.org/ioam/param/jobs/440195588#L658

nbsite uses nbconvert to execute notebooks. Although I recently tried to update nbsite to work with nbconvert 5.4 (released about a month ago), it seems that I was not actually successful.

nbsite contains some hacks to execute code at notebook startup. We should really revisit this mechanism, because it's a complete hack. nbconvert 5.4 (the first release in over a year) appears to have made various changes to the execute preprocessor, probably breaking our hack:

Overall the ExecutePreprocessor has been reworked to make it easier to use. Future releases will continue this trend to make this section of the code more inheritable and reusable by others. We encourage you read the source code for this version if you’re interested in the detailed improvements.

(https://nbconvert.readthedocs.io/en/latest/changelog.html)

Default startup code (can be overridden by a project, but probably nobody does that): https://github.com/pyviz/nbsite/blob/master/nbsite/ipystartup.py

Hack to execute it: https://github.com/pyviz/nbsite/blob/master/nbsite/nbbuild.py#L55

Surely there's a better way to achieve that?

Should also verify that the problem in the travis link above is definitely from nbconvert 5.4; most of this post is just speculation.

@ceball
Copy link
Contributor Author

ceball commented Oct 11, 2018

holoviz/param#293 indicates that there is some problem with nbsite and nbconvert 5.4, because installing <5.4 fixes the problem. If true, we should fix nbsite or otherwise pin nbconvert.

@jbednar
Copy link
Contributor

jbednar commented Oct 11, 2018

For now please pin nbconvert here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants