Redo xarray with MetPy tutorial for 1.0 #1605
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
jthielen
added
Area: Docs
|
This pull request introduces 1 alert when merging 4581a27 into f991506 - view on LGTM.com new alerts:
|
dopplershift
requested changes
Dec 9, 2020
dopplershift
reviewed
Dec 9, 2020
|
This pull request introduces 1 alert when merging 4f5d697 into 495061a - view on LGTM.com new alerts:
|
|
If you rebase on master, that should clean up Code Climate. |
jthielen
force-pushed
the
xarray-guide-1.0
branch
from
4f5d697
to
83e5a21
Compare
|
Remaining comments should now have been addressed, and I've rebased on master. |
dopplershift
previously approved these changes
Dec 10, 2020
dopplershift
previously approved these changes
Dec 10, 2020
dopplershift
force-pushed
the
xarray-guide-1.0
branch
from
20f3a6e
to
d3cfd73
Compare
dopplershift
force-pushed
the
xarray-guide-1.0
branch
from
d3cfd73
to
32e2090
Compare
dopplershift
approved these changes
Dec 16, 2020
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description Of Changes
Overhaul the xarray with MetPy tutorial given the substantial changes in xarray support for the 1.0 release. Part of the remaining effort towards #1385. This makes extensive use of HTML reprs for xarray and Pint, so it is nice that we've gotten that working seemingly well with the new theme.
Checklist
ClosesTests added