widgetti / ipyvolume

3d plotting for Python in the Jupyter notebook based on IPython widgets using WebGL
MIT License
1.95k stars 236 forks source link

FEATURE REQUEST: a no dependencies option for embed_html with offline=True #199

Open GenevieveBuckley opened 6 years ago

GenevieveBuckley commented 6 years ago

Feature request for a no dependencies option for embed_html with offline=True.

As discussed here: https://github.com/maartenbreddels/ipyvolume/issues/192#issuecomment-434214895

I was wondering if it would be of general use to have a no dependencies option for the offline html? It'd be useful in situations where you write many html files to the same location, and care about how fast you can write them to disk. (This is something we're doing at the moment, overwriting a single file with new data, and I think it'd speed up the responsiveness quite a bit if it didn't re-download every dependency each time.)

@maartenbreddels commented Yes, a cache would be nice, I think though it should (optionally) check the CDN for the right version, and if not it should update it. My guess is that the request package can do this. I also hate that it downloads everything every time. It can also do a hardlink to the cached file (for non-windows users).

maartenbreddels commented 6 years ago

Hi Genevieve,

That is what offline=True should do right, or do you mean with the no dependencies option to use the cache? Do you think using the cache should be op in?

cheers,

Maarten

GenevieveBuckley commented 6 years ago

Sorry, I think I misunderstand. From the code in embed.py it doesn't look like it checks whether the necessary files already exist, it just writes them every time to a folder called js next to the output html. I do think using cached/existing dependencies should be the default.