Skip to content
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

Contour norm scaling #1057

Merged
merged 6 commits into from Aug 11, 2012
Merged

Conversation

efiring
Copy link
Member

@efiring efiring commented Aug 6, 2012

These changes follow #1022 by fixing a baseline image, adding to the CHANGELOG entry, and making the contour handling of norm scaling more consistent with other image-like plotting such as pcolor by adding vmin, vmax kwargs.



Filled contours now use more of the available colormap range than
previously.
This change should have no effect on any plot, but it avoids
warnings that can arise when an inf occurs under a mask.
Previously, although vmin, vmax could be specified manually
in a norm used to initialize a ContourSet, the values would
be used only if both were specified.  Now one can set either
or both of vmin, vmax in the norm, and they will take precedence
over the default.
This changeset also simplifies and clarifies the vmin, vmax
explanation and handling for pcolor and pcolormesh.
@mdboom
Copy link
Member

mdboom commented Aug 6, 2012

Looks good. I think this also fixes #1046.

…ntour

Formerly, colorbar would not place ticks at contour levels outside the
vmin-vmax range, and would not add line contour marks at such levels.
@efiring
Copy link
Member Author

efiring commented Aug 7, 2012

I added some related colorbar changes and a test. That's all I have in mind for this one.

efiring added a commit that referenced this pull request Aug 11, 2012
@efiring efiring merged commit 41822d5 into matplotlib:master Aug 11, 2012
@efiring efiring deleted the contour_norm_scaling branch May 29, 2013 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants