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

[Bug]: tripcolor ignores clim #22726

Closed
anntzer opened this issue Mar 29, 2022 · 2 comments · Fixed by #22734
Closed

[Bug]: tripcolor ignores clim #22726

anntzer opened this issue Mar 29, 2022 · 2 comments · Fixed by #22734
Milestone

Comments

@anntzer
Copy link
Contributor

anntzer commented Mar 29, 2022

Bug summary

from pylab import *
tripcolor(np.random.rand(100), np.random.rand(100), np.random.rand(100), clim=(0, 0.5))
colorbar()
show()

shows that tripcolor ignores clim.

Code for reproduction

See above.

Actual outcome

Colorbar/colormapping goes from nearly 0 to nearly 1.

Expected outcome

Colorbar/colormapping goes from 0 to 0.5.

Additional information

This is the same issue as #21146/#21525: kwargs should be handled a bit further down in the implementation of tripcolor() (just before calling _scale_norm).

Operating system

arch linux

Matplotlib Version

3.6.0.dev1920+gdfd83c2c5d

Matplotlib Backend

mplcairo

Python version

310

Jupyter version

ENOSUCHLIB

Installation

git checkout

@andrew-fennell
Copy link
Contributor

andrew-fennell commented Mar 30, 2022

I want to make sure I understand the issue correctly. Could you look at my "correct output"?

Test code

from pylab import *
tripcolor(np.random.rand(100), np.random.rand(100), np.random.rand(100), clim=(0, 0.5))
colorbar()
show()

Incorrect output

image

Correct output (???)

image

@anntzer
Copy link
Contributor Author

anntzer commented Mar 30, 2022

Yes, this would be correct.

@QuLogic QuLogic added this to the v3.6.0 milestone Mar 31, 2022
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 a pull request may close this issue.

3 participants