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

Too many ipyleaflet layers fail to render #543

Open
yudai-nkt opened this issue Feb 15, 2020 · 4 comments
Open

Too many ipyleaflet layers fail to render #543

yudai-nkt opened this issue Feb 15, 2020 · 4 comments

Comments

@yudai-nkt
Copy link

yudai-nkt commented Feb 15, 2020

I noticed that notebooks with too many ipyleaflet layers cannot be properly rendered with Voila. Those notebooks are rendered on mybinder.org, so this could be a problem of my computer. That being said, I'd be happy if notebooks that can be rendered with JupyterLab locally can be rendered with Voila as well, or at least Voila issues a warning saying some widgets are not rendered.

Below is a table of results with a reproducible example and GIFs capturing the results on my local end.

JupyterLab Voila
my end success fail
online (nbviewer or binder) success success

JupyterLab:

lab_imagemagick

Voila

voila_imagemagick

This issue might be a duplicate of #534, but I'm afraid I'm not quite sure. Feel free to close if that's the case.

@davidbrochart
Copy link
Member

davidbrochart commented Feb 24, 2020

I can reproduce locally, and I get a TimeoutError: Cell execution timed out in the terminal. The default timeout value is 30s, but you can change it:

$ voila --help-all
...
--VoilaExecutePreprocessor.timeout=<Int>
    Default: 30
    The time to wait (in seconds) for output from executions. If a cell
    execution takes longer, an exception (TimeoutError on python 3+,
    RuntimeError on python 2) is raised.
    `None` or `-1` will disable the timeout. If `timeout_func` is set, it
    overrides `timeout`.
...

@davidbrochart
Copy link
Member

Or if you want to cancel the execution a cell if it takes too long, use this option:

--VoilaExecutePreprocessor.interrupt_on_timeout=True

@tonywang531
Copy link

I have tried to set the timeout value to 99999 before and it still works (I put some machine learning data that takes like 2 hours to compute). Although it is definitely not user friendly to wait for 2 hours before getting an useful output.

@raybellwaves
Copy link

raybellwaves commented Jan 20, 2022

believe VoilaExecutePreprocessor.interrupt_on_timeout is now named VoilaExecutor.timeout

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

4 participants