-
Notifications
You must be signed in to change notification settings - Fork 164
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
[BUG] Added xarray to dependecies #116
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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: Velocity
can't be executed on binder.IMHO adding
netcdf4
to the normal dependencies and making the download code for thewind-global.nc
dataset a code cell instead of an instruction in a markdown cell would be an improvement.