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

Action fails to run notebook #13

Closed
MathewBiddle opened this issue Feb 1, 2024 · 5 comments · Fixed by #14
Closed

Action fails to run notebook #13

MathewBiddle opened this issue Feb 1, 2024 · 5 comments · Fixed by #14

Comments

@MathewBiddle
Copy link
Contributor

https://github.com/marinebon/map-of-activities/actions/runs/7736796590/job/21094706544

Problem with:

jupyter nbconvert --to notebook --execute map-of-activities.ipynb --output=map-of-activities-output.ipynb

Error messages:

[NbConvertApp] ERROR | Kernel died while waiting for execute reply.
nbclient.exceptions.DeadKernelError: Kernel died
@MathewBiddle
Copy link
Contributor Author

seems to run fine locally. Trying to rerun in actions now.

@MathewBiddle
Copy link
Contributor Author

Still failing. https://github.com/marinebon/map-of-activities/actions/runs/7743934360

@ocefpaf any ideas?

We could reorganize this repo by moving the notebook code to a python script and running that. Would eliminate the need for jupyter nbconvert, which is what seems to be causing the problem. Would probably install and run faster too?

@ocefpaf
Copy link
Contributor

ocefpaf commented Feb 1, 2024

which is what seems to be causing the problem.

Not sure...

Would probably install and run faster too?

That is true. You don't need/store the outputs of the notebook, right? I can try that in a PR right now. Hold on...

@MathewBiddle
Copy link
Contributor Author

You don't need/store the outputs of the notebook, right?

We do. We write out index.html to https://github.com/marinebon/map-of-activities/tree/main/docs

Outputs inside the notebook? No.

@ocefpaf
Copy link
Contributor

ocefpaf commented Feb 1, 2024

We can keep that with the script if we don't care about opening the notebook. Working on it.


Edit: you can run that b/c you have an old env. Some updated dependency is causing a segfault. Fixing that and locking the env to avoid future regressions.

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 a pull request may close this issue.

2 participants