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

sage --notebook=export does not start #24198

Closed
egourgoulhon opened this issue Nov 11, 2017 · 12 comments
Closed

sage --notebook=export does not start #24198

egourgoulhon opened this issue Nov 11, 2017 · 12 comments

Comments

@egourgoulhon
Copy link
Member

In Sage 8.1.rc0, the command sage --notebook=export (default for sage --notebook) fails with

[C 14:16:56.066 NotebookApp] Bad config encountered during initialization:
[C 14:16:56.066 NotebookApp] The 'nbserver_extensions' trait of a NotebookApp instance must be a dict, but a value of type 'list' (i.e. ['sagenb_export.nbextension']) was specified.

(see attached file for full log).

Both sage --notebook=jupyter and sage --notebook=sagenb are OK though.

This is on Ubuntu 16.04 and ahs been reported at
https://groups.google.com/d/msg/sage-release/7-eKF-xN0p0/z3T5zeMhAQAJ

NB: everything is OK with Sage 8.1.beta8, so the issue is most certainly due to a change introduced in Sage 8.1.beta9 or 8.1.rc0.

CC: @jdemeyer @kiwifb

Component: notebook

Author: Jeroen Demeyer

Branch/Commit: f8eef4e

Reviewer: Eric Gourgoulhon

Issue created by migration from https://trac.sagemath.org/ticket/24198

@egourgoulhon
Copy link
Member Author

Log of "sage --notebook" command

@egourgoulhon

This comment has been minimized.

@egourgoulhon
Copy link
Member Author

comment:1

Attachment: sage_notebook.log

@egourgoulhon
Copy link
Member Author

comment:2

It seems that the issue is related to #24064, which has been merged in Sage 8.1.beta9: in the discussion of that ticket, there is mention of nbserver_extensions, which appears in the error message.

@jdemeyer
Copy link

comment:3

Sorry for messing this up...

I usually run ./sage -n jupyter, so that is why I did not notice this.

@jdemeyer
Copy link

Author: Jeroen Demeyer

@jdemeyer
Copy link

@jdemeyer
Copy link

Commit: f8eef4e

@jdemeyer
Copy link

comment:5

The new format is a dict {module: enabled}.


New commits:

f8eef4eUse correct format for nbserver_extensions

@egourgoulhon
Copy link
Member Author

Reviewer: Eric Gourgoulhon

@egourgoulhon
Copy link
Member Author

comment:6

Thanks for the fix! (I've just checked it on my computer).

@vbraun
Copy link
Member

vbraun commented Nov 15, 2017

Changed branch from u/jdemeyer/sage___notebook_export_does_not_start to f8eef4e

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

No branches or pull requests

3 participants