search

jupyter-widgets / tutorial

A tutorial for widgets
BSD 3-Clause "New" or "Revised" License
394 stars 164 forks source link

Add missing ipywebrtc dependency #42

Closed martinRenou closed 5 years ago

mwcraig commented 5 years ago

Last year we didn't have participants install ipywebrtc, we just did a demo. @maartenbreddels what do you think for this year?

martinRenou commented 5 years ago

Oh, then maybe we can remove ipysheet/ipytree installation from the README?

I thought those commands were the needed commands for having all the Notebooks working on this repo

mwcraig commented 5 years ago

Oh, then maybe we can remove ipysheet/ipytree installation from the README?

I think these we should keep because they are easy for participants to try out. We left out ipywebrtc last year because we didn't want to try simultaneously streaming ~100 laptops on hotel wifi (to be fair, that hotel's wifi is pretty good).

martinRenou commented 5 years ago

Ahah yes :P It makes sense. Also, I fear that could crash the Notebook if 100 persons join the webrtc room at the same time, that would be too many video streams to display for one webpage

martinRenou commented 5 years ago

Closing it