[BUG] Added xarray to dependecies#116
[BUG] Added xarray to dependecies#116s-weigand wants to merge 4 commits intojupyter-widgets:mainfrom
Conversation
mwcraig
left a comment
There was a problem hiding this comment.
Thanks, @s-weigand. Any chance you could also make the change you mentioned to the notebook? I think the reason we had it that way was, in part, that the hotel WiFi at SciPy, while good, crumbled under too much simultaneous load.
Not an issue when the conference is distributed!
Other tutorial folks, if this looks good to you too please go ahead and merge.
|
Yeah WiFi at conferences ... 😆 |
|
Let's go with the download code; some day we'll be able to be in person again and we don't want the repo to get too big. |
and removed now obsolete download instructions. Also added 'wind-global.nc' to .gitignore, so it won't be uploaded accidentally.
so it is show in the output cell as well as in the Sidecar.
|
I made the velocity map the last statement of the input cell, so it is shown in the output cell as well as in the |
|
Btw. I use |
The notebook 07.01-ipyleaflet.ipynb uses
xarray, which isn't part of the dependencies and thusAdvanced example 1: Velocitycan't be executed on binder.IMHO adding
netcdf4to the normal dependencies and making the download code for thewind-global.ncdataset a code cell instead of an instruction in a markdown cell would be an improvement.