-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
changes for MEP12/sphinx-gallery compliance #8466
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
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.
More minor stuff (altough markup needs to be correct). Also squash if you agree?
|
||
This shows 4 possible projections using subplot. | ||
Matplotlib also supports | ||
`Basemaps Toolkit <http://matplotlib.org/basemap/`_ and |
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.
wrong markup (and below)
also reflow the paragraph, perhaps
|
||
This example illustrates how to read an image (of an MRI) into a NumPy array, | ||
and display it in greyscale using `imshow`. | ||
|
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.
remove empty line
Going to close this since it's a very tedious rebase, and the original is still at #8209 |
This is a rebase of #8209 . I've marked as MRG+1 since I think this only needs one more pair of eyes to be merged.