Skip to content

Minor tweaks to image docs. #29184

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

Merged
merged 1 commit into from
Nov 25, 2024
Merged

Minor tweaks to image docs. #29184

merged 1 commit into from
Nov 25, 2024

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Nov 25, 2024

PR summary

PR checklist

Copy link
Member

@timhoffm timhoffm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Take or leave the suggested change to "Parameters". You can self-merge then.

Comment on lines +237 to 247
*interpolation* and *cmap* default to their rc settings.

cmap is a colors.Colormap instance
norm is a colors.Normalize instance to map luminance to 0-1
*cmap* is a `.colors.Colormap` instance.
*norm* is a `.colors.Normalize` instance to map luminance to 0-1.

extent is data axes (left, right, bottom, top) for making image plots
registered with data plots. Default is to label the pixel
centers with the zero-based row and column indices.
*extent* is a ``(left, right, bottom, top)`` tuple in data coordinates, for
making image plots registered with data plots; the default is to label the
pixel centers with the zero-based row and column indices.

Additional kwargs are matplotlib.artist properties
Additional kwargs are `.Artist` properties.
"""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is an improvement, so ok. But actually, this should be a formal "Parameters" section.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know, but I didn't really want to go through rewriting the whole thing, so I'll self-merge.

@anntzer anntzer merged commit 96df509 into matplotlib:main Nov 25, 2024
47 of 50 checks passed
@anntzer anntzer deleted the id branch November 25, 2024 10:47
@QuLogic QuLogic added this to the v3.11.0 milestone Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants