Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Update Colorizer/ColorizingArtist to work with MultiNorm #30511

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update Colorizer/ColorizingArtist to work with MultiNorm #30511

wants to merge 2 commits into from
+478 −76

Conversation

trygvrad
Copy link
Contributor

@trygvrad trygvrad commented Sep 3, 2025

This PR continues the work of #28658 and #28454, #29876, aiming to close #14168. (Feature request: Bivariate colormapping)

This PR allows Colorizer and ColorizingArtist to work with MultiNorm and BivarColormap and MultivarColormap

i.e. this PR will allow:

import matplotlib.pyplot as plt
import numpy as np
import matplotlib as mpl
a = np.arange(12).reshape(3,4)
b = np.arange(12).reshape(4,3).T

norm = mpl.colors.Normalize(vmin=0, vmax=11)
col = mpl.colorizer.Colorizer(cmap='BiOrangeBlue', norm=[norm, norm])

plt.imshow(col.to_rgba((a, b)))
image

Features not included in this PR:

  • Exposes the functionality provided by MultiNorm together with BivarColormap and MultivarColormap to the plotting functions axes.imshow(...), axes.pcolor(...), and axes.pcolormesh(...)`
  • fig.colorbar() equivalents for BivarColormap and MultivarColormap
  • Examples in the docs

PR checklist

  • new and changed code is tested

@trygvrad trygvrad force-pushed the colorizer-multinorm branch from 77bc51a to 9044189 Compare September 3, 2025 17:20
QuLogic
QuLogic reviewed Sep 4, 2025
View reviewed changes
@trygvrad trygvrad force-pushed the colorizer-multinorm branch from 9b9aaf3 to 525314e Compare September 4, 2025 23:35
@trygvrad
Copy link
Contributor Author

trygvrad commented Sep 4, 2025

Thank you for the feedback @QuLogic and apologies for my sloppy mistakes :)

@trygvrad
Update Colorizer/ColorizingArtist to work with MultiNorm
23fb63c 
improved testing for colorizer+multinorm

Apply suggestions from code review

Co-authored-by: Elliott Sales de Andrade <[email protected]>

updates based on feedback from @QuLogic
@trygvrad trygvrad force-pushed the colorizer-multinorm branch from 525314e to 23fb63c Compare September 8, 2025 07:20
This was referenced Sep 8, 2025
Closed
Open
QuLogic
QuLogic reviewed Sep 9, 2025
View reviewed changes
QuLogic
QuLogic approved these changes Sep 11, 2025
View reviewed changes
Copy link
Member

@QuLogic QuLogic left a comment

Choose a reason for hiding this comment

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

As I mentioned on the call, I think this looks okay from a low-level point-of-view, but I am at this point uncertain as to the high-level status. I hope someone more familiar with the current state can confirm.

PS, @trygvrad if you have some time to pop in to the call, it would be good to just re-iterate the current state succinctly, as I'm not sure everyone is up-to-speed right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@QuLogic QuLogic QuLogic approved these changes

Assignees
No one assigned
Labels
topic: color/color & colormaps
Projects
Bivariate and Multivariate Colormapping
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature request: Bivariate colormapping
2 participants
@trygvrad @QuLogic