-
Notifications
You must be signed in to change notification settings - Fork 26
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
Send command: restart kernel and run all? #48
Comments
We have separately implemented the Let me know if this solves your issue! |
On Fri Jan 20, 2023 at 01:07, Zack Polizzi wrote:
We have separately implemented the `restart` and `execute_all` commands, you can execute the requests with the files in this folder: https://github.com/untitled-ai/jupyter_ascending/tree/main/jupyter_ascending/requests. You should be able to chain these together from wherever you're calling it from.
Let me know if this solves your issue!
Thanks! Almost, but not quite.
If you attempt something like this:
```
python -m jupyter_ascending.requests.restart --filename 20221221.sync.py && python -m jupyter_ascending.requests.execute_all --filename 20221221.sync.py
```
You'll get an error:
```
jsonrpcclient.exceptions.ReceivedErrorResponseError: Server error
```
So a "restart and run all", like what you have in the jupyter notebook dropdown
menu needs to be implemented differently. I'm guessing jupyter should already
expose something like this that ascending could directly call?
|
Ah that's a good point, the server will go down after the restart command and probably fail to handle the run all command properly. One quick fix would just be to add like a 10 second delay between the two. But yeah likely there's a way to implement a unified command - I'd be happy to review a PR if anyone wants to try to get this working. |
I would like to be able to send a command that restarts the kernel and runs the entire notebook. Is this currently possible?
The text was updated successfully, but these errors were encountered: