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

Systematic Approach of adding notebooks #335

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

kolibril13
Copy link
Contributor

No description provided.

@kolibril13
Copy link
Contributor Author

@zivy :
I do not understand the error at readthedocs:
After the commit "added more notebooks" everything is fine:
https://readthedocs.org/projects/simpleitk-notebooks/builds/15235578/
but after the commit "adding 11" readthedocs is failing:
https://readthedocs.org/projects/simpleitk-notebooks/builds/15235764/

Notebook error:
DeadKernelError in Python/03_Image_Details.ipynb:
Kernel died

Do you have an idea why the kernel died here?
It is strange, because this notebook was not altered from the previous commit, so it seems the kernel died randomly.
The only thing that got changed was that I added the notebook Python/11_Progress.ipynb in "adding 11"
I hope I am missing something here.

@zivy
Copy link
Member

zivy commented Nov 11, 2021

@kolibril13,

Not sure what the issue is. The 03_Image_Details notebook has multiple cells that have exceptions. These are either intentional or due to missing Fiji. If the conversion is run with ignore exceptions it should just work fine. Apparently it didn't but I have no clue why. Locally I have never had a problem with this notebook. I tried it via binder and it is fine there too, just many exceptions but these are all expected.

Is there a way to ssh into a RTD environment to try to debug?

@kolibril13
Copy link
Contributor Author

Interesting, after the commit change nbsphinx_timeout value readthedocs does not fail.

@kolibril13
Copy link
Contributor Author

Still not failing, maybe the nbsphinx_timeout parameter did the trick. Adding even more notebooks

@kolibril13
Copy link
Contributor Author

And kernel died again at Python/11_Progress.ipynb
Only removing that one

@zivy
Copy link
Member

zivy commented Nov 15, 2021

Notebook 11 uses javascript and is not amenable to conversion (see testing).

@kolibril13
Copy link
Contributor Author

Now 55 is dying:

Notebook error:
DeadKernelError in Python/55_VH_Resample.ipynb:
Kernel died

@kolibril13
Copy link
Contributor Author

Now, there is a timeout error

@kolibril13
Copy link
Contributor Author

added more notebooks

@kolibril13
Copy link
Contributor Author

Now here:
Notebook error:
DeadKernelError in Python/03_Image_Details.ipynb:
Kernel died

Just randomly.

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.

2 participants