Allow for per notebook/cell execution timeout settings

[chrisjsewell]

Stored in metadata, that is implemented by the executor

Also relates to #5

[choldgraf]

Hmm - I suspect that the nbformat issue will take some time to resolve, so I wonder if we should just go with @AakashGfude 's proposed metadata spec, and if Jupyter adopts an "official" one then we can support that as well. Some folks have reported this bug in Jupyter Book, so I think we should just go with a short-term solution. WDYT?

[chrisjsewell]

Yeh go for it. Just make sure that an issue is still open for the "long-term" solution, so we don't forget

[choldgraf]

cc @AakashGfude , in case you have a moment to wrap up your PR on this one (see above comment for context)

[AakashGfude]

ooh. cool. I will wrap this up as soon as I can.

[choldgraf]

@AakashGfude awesome, thanks...we've had like 3 bug reports opened about this particular issue already 😆

[AakashGfude]

oops. sorry I missed this one. The PRs in Myst-NB and jupyter-cache were ready to be merged from my side. Additionally, I will try and add a test for this in Myst-NB.

[AakashGfude]

@choldgraf have added test-cases. Note that the Tests in Myst-NB will fail, until the PR in jupyter-cache is merged.
If you have time, will you be able to add the necessary additions for execution_timeout in jupyter-book? Else I can do it some time during the weekend.

[choldgraf]

@AakashGfude for sure, I'll figure out how to get it into jupyter book

[poldrack]

any update on this? I'm running into the same timeout issue as #613

[choldgraf]

I believe that if you add notebook-level metadata to a page then it should work w/ the latest version of jupyter-cache: https://jupyter-cache.readthedocs.io/en/latest/using/api.html?highlight=timeout#timeout

however we have not yet documented this in Jupyter Book! @poldrack if this works for you, can you let us know and we can try to add docs for it in JB?