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

xeus-kernels fail #80

Closed
DerThorsten opened this issue May 14, 2022 · 7 comments · Fixed by jupyterlite/xeus-sqlite-kernel#11
Closed

xeus-kernels fail #80

DerThorsten opened this issue May 14, 2022 · 7 comments · Fixed by jupyterlite/xeus-sqlite-kernel#11
Labels
bug Something isn't working

Comments

@DerThorsten
Copy link
Contributor

DerThorsten commented May 14, 2022

all xeus kernels fail with the same error as jupyterlite/xeus-sqlite-kernel#10.

Interestingly I can locally install all of them via pip and they work without that issue

@DerThorsten DerThorsten added the bug Something isn't working label May 14, 2022
@DerThorsten
Copy link
Contributor Author

@jtpio when I install all extensions locally (as in the requirements) I can reproduce the error locally...so...lets figure out which one is the bad one

@DerThorsten
Copy link
Contributor Author

it sems that the xeus kernels atm dont work when ipywidgets is installed

@jtpio
Copy link
Member

jtpio commented May 16, 2022

@DerThorsten the SQLite kernel seems to be working (when opening https://jupyterlite.github.io/demo/lab/index.html in a private window):

image

But after a kernel restart.

it sems that the xeus kernels atm dont work when ipywidgets is installed

You mean when the ipywidgets Python package is installed as part of the deployment? Or when the xeus kernels are compiled with widgets.

@martinRenou
Copy link
Member

This issue is coming from xeus which complains about a non-existing comm target, and lamentably fails to show the error message. We fixed it very recently in https://github.com/jupyter-xeus/xeus/pull/327/files. Maybe we are not using that fix yet

@martinRenou
Copy link
Member

Just to give more context about this error, https://github.com/jupyter-xeus/xeus/pull/327/files#diff-af1c39110d53dd864ae63b152634851fb7b89dbb43a93584ecd368db8414e9c2L90 is trying to concatenate an std::string with a fixed_string<64> resulting in a fixed_string<64> that is more than 64 characters, which makes a static assert fail.

@martinRenou
Copy link
Member

I think the reason why you are not able to reproduce locally @DerThorsten is because you don't have the ipywidgets labextension in the page, so the front-end is not requesting the kernel for comm targets.

@martinRenou
Copy link
Member

This should fix it jupyterlite/xeus-sqlite-kernel#11

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

Successfully merging a pull request may close this issue.

3 participants