-
Notifications
You must be signed in to change notification settings - Fork 68
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
Update Spatial_selection.ipynb notebook #278
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
I think we want to move this example into "Tutorials". What do you think @hrsdawson?
I also noticed that at the top there is a broken link to the map tutorial; could we fix that in this PR?
View / edit / reply to this conversation on ReviewNB navidcy commented on 2023-09-04T01:30:54Z Line #8. #from pint import application_registry as ureg Line #9. #import cf_xarray.units delete commented lines? |
View / edit / reply to this conversation on ReviewNB julia-neme commented on 2023-09-04T04:13:53Z Link to tutorial on working with projections not working! |
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.
as per comments
View / edit / reply to this conversation on ReviewNB julia-neme commented on 2023-09-04T04:21:00Z I think the colormap could be prettier! |
Nice! I moved it to Tutorials. |
This pull request has been mentioned on ACCESS Hive Community Forum. There might be relevant details there: https://forum.access-hive.org.au/t/cosima-hackathon-v3-0-monday-september-4th-2023/1186/4 |
Closes #272