8000 DOC: CL make pcolormesh calls consistent by jklymak · Pull Request #10849 · matplotlib/matplotlib · GitHub
[go: up one dir, main page]

Skip to content

DOC: CL make pcolormesh calls consistent #10849

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 2 commits into from
Mar 20, 2018

Conversation

jklymak
Copy link
Member
@jklymak jklymak commented Mar 20, 2018

PR Summary

As per discussion in #3644 (comment) with @efiring the constrained layout guide didn't constrain ScalarMappables in different subplots to have the same norm when sharing a colorbar, which was deemed bad form. This fixes that in the most compact way I could think of. I added a note so its not too mysterious to users (I hope).

I'm not 100% convinced this is really a problem. Like if someone shares a colorbar and sets the vmin and vmax in one axes to be inconsistent w/ the colorbar values, I'd hope they notice, but I suppose this is good practice and we should strive to show best pratices in the tutorials.

PR Checklist

  • Documentation is sphinx and numpydoc compliant

@jklymak jklymak added this to the v2.2-doc milestone Mar 20, 2018
Copy link
Member
@efiring efiring left a comment

Choose a reason for hiding this comment

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

One typo. Apart from that, this is a big improvement; I think it is important.

@dopplershift dopplershift merged commit 6af4e4f into matplotlib:master Mar 20, 2018
@jklymak jklymak deleted the doc-improve-CL-tutorial branch March 20, 2018 23:37
QuLogic added a commit that referenced this pull request Mar 21, 2018
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
0