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

timeout not working with %%capture #712

Open
joamatab opened this issue Feb 22, 2023 · 3 comments
Open

timeout not working with %%capture #712

joamatab opened this issue Feb 22, 2023 · 3 comments

Comments

@joamatab
Copy link

I think there is an error that timeout from conf.py is being ignored when using %%capture magic

gdsfactory/gdsfactory#1330

@mgeier
Copy link
Member

mgeier commented Feb 26, 2023

nbsphinx isn't implementing the timeout itself, but it instead uses nbconvert.preprocessors.ExecutePreprocessor:

pp = nbconvert.preprocessors.ExecutePreprocessor(
kernel_name=self._kernel_name,
extra_arguments=self._execute_arguments,
allow_errors=allow_errors, timeout=timeout)

If you can reproduce the problem directly with that (without involving nbsphinx), you could create an issue at https://github.com/jupyter/nbconvert/issues/, or maybe at https://github.com/jupyter/nbclient/issues/ (which is in turn used by nbconvert).

Once this is solved there, it will automagically also work with nbsphinx (at least I would hope so).

@mgeier
Copy link
Member

mgeier commented Mar 25, 2023

This may or may not be related: jupyter/nbconvert#1964

@nthiery
Copy link

nthiery commented Mar 26, 2023

Thanks @mgeier for the connection; I did some more drilling on jupyter/nbconvert#1964.
You may want to try to downgrade jupyter_client to <8.

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

No branches or pull requests

3 participants