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

ENH: Update the napari notebook #220

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

ntatsisk
Copy link
Collaborator

Updates ITK_Example14_WorldCoordinatesExplanation_Napari.ipynb to address #201. I also removed the qt version pins because they were forcing a very old napari version to be installed.

When I test the notebook with pytest in an ubuntu machine via SSH, it passes normally. However, I cannot find how to make napari with jupuyter work in the CI. Some help would be appreciated.

Related to https://forum.image.sc/t/napari-jupyter-notebook-issue/32640/15?

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@ntatsisk ntatsisk changed the title Update napari notebook ENH: Update the napari notebook May 19, 2023
@thewtex
Copy link
Member

thewtex commented May 19, 2023

@ntatsisk thanks!

Yes, napari in CI is more difficult -- there needs to be a graphical context on the server. Some of the napari testing code or smart folks like @tlambert03 @jni @kne42 may provide hints.

@jni
Copy link

jni commented Jun 10, 2023

👋 hello, sorry for the delay! 😅

Some resources: we use https://github.com/aganders3/headless-gui and https://github.com/tlambert03/setup-qt-libs to set up our CI.

To run napari in a remote notebook, you need jupyter-desktop to have a remote desktop on which to open the napari window. This repo should be the most up-to-date for getting that working: https://github.com/napari/napari-workshop-template/ (see the binder/ folder).

@ntatsisk ntatsisk force-pushed the update-napari-notebook branch 6 times, most recently from 6dcf777 to 77f62b8 Compare June 20, 2023 02:15
@ntatsisk ntatsisk force-pushed the update-napari-notebook branch 2 times, most recently from 1e2e6e8 to a793967 Compare July 15, 2023 17:43
@ntatsisk
Copy link
Collaborator Author

Hi @jni, thank you for your reply and your direct suggestions during the SciPy 2023 sprints, much appreciated! :)

Unfortunately, still I cannot make it work. I am getting this error on that particular notebook that includes napari:

------------------------------ Captured log call -------------------------------
ERROR traitlets:client.py:863 Kernel died while waiting for execute reply.
=========================== short test summary info ============================
FAILED examples/ITK_Example14_WorldCoordinatesExplanation_Napari.ipynb:: - NBMAKE INTERNAL ERROR
================== 1 failed, 27 passed in 1594.12s (0:26:34) ===================
Error: Process completed with exit code 1.

Any further ideas? Thanks!

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

Successfully merging this pull request may close these issues.

None yet

3 participants