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

Kernel misses autoreload IPython extension #62

Closed
SchraivogelS opened this issue Jul 8, 2021 · 5 comments
Closed

Kernel misses autoreload IPython extension #62

SchraivogelS opened this issue Jul 8, 2021 · 5 comments

Comments

@SchraivogelS
Copy link

Hi there,
first of all thanks for the jupyter notebook extension for 3DSlicer.
It allows for streamlined data processing and reproducible workflows.

Slicer kernel seems to miss autoreload extension, which is a handy feature within jupyter notebooks.

%load_ext autoreload
causes AttributeError: 'kernel.XInteractiveShell' object has no attribute 'events'

@lassoan
Copy link
Contributor

lassoan commented Jul 8, 2021

We use xeus-python kernel, which might have not implemented this yet - see the discussion here:

jupyter-xeus/xeus-python#63 (comment)

Could you ask for clarification either in that issue or creating a new one and report here what you find out?

If it is fixed/implemented recently then we may just need to update the xeus-python kernel version in SlicerJupyter.

@SchraivogelS
Copy link
Author

They added a bunch of magics at the beginning of this year:
https://github.com/jupyter-xeus/xeus-python/blob/fc09e25d882e62bf72cc3ae6644b29cadd174fb8/notebooks/xeus-magics.ipynb

Just installed xeus-python 0.12.5 in a fresh conda env along with python 3.9.6 and autoreload extension is available.
Is an update feasible?

@lassoan
Copy link
Contributor

lassoan commented Jul 9, 2021

I've tried to update but recent xeus-python versions require IPython>=7.21, but the last IPython version for Python-3.6 is IPython-7.15. We need to wait until Slicer core is updated to at least Python-3.7 - see Slicer/Slicer#5014.

@SchraivogelS
Copy link
Author

SchraivogelS commented Jul 9, 2021 via email

@lassoan
Copy link
Contributor

lassoan commented Jun 26, 2022

We have updated Slicer's Python some time ago and can use the latest xeus-python, so hopefully the problem is now fixed.

@lassoan lassoan closed this as completed Jun 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants