NotificationsYou must be signed in to change notification settings
Fork8.1k
Star22.1k

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.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

story645 merged 4 commits intomatplotlib:mainfromtrygvrad:colorizer-multinorm

Oct 30, 2025

Merged

Update Colorizer/ColorizingArtist to work with MultiNorm#30511

story645 merged 4 commits intomatplotlib:mainfromtrygvrad:colorizer-multinorm

Oct 30, 2025

+492 −77

Conversation

Copy link

Contributor

trygvrad commentedSep 3, 2025•
edited
Loading

EDIT: closes#30511

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

This PR allows Colorizer and ColorizingArtist to work withMultiNorm andBivarColormap andMultivarColormap

i.e. this PR will allow:

importmatplotlib.pyplotaspltimportnumpyasnpimportmatplotlibasmpla=np.arange(12).reshape(3,4)b=np.arange(12).reshape(4,3).Tnorm=mpl.colors.Normalize(vmin=0,vmax=11)col=mpl.colorizer.Colorizer(cmap='BiOrangeBlue',norm=[norm,norm])plt.imshow(col.to_rgba((a,b)))

Features not included in this PR:

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

PR checklist

new and changed code istested

github-actionsbot added the topic: color/color & colormaps label

Sep 3, 2025

trygvrad force-pushed thecolorizer-multinorm branch from77bc51a to9044189Compare

September 3, 2025 17:20

QuLogic added this toBivariate and Multivariate Colormapping

Sep 3, 2025

QuLogic reviewed

Sep 4, 2025

View reviewed changes

lib/matplotlib/cbook.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

lib/matplotlib/tests/test_colors.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

github-actionsbot added the status: needs rebase label

Sep 4, 2025

trygvrad force-pushed thecolorizer-multinorm branch from9b9aaf3 to525314eCompare

September 4, 2025 23:35

Copy link

ContributorAuthor

trygvrad commentedSep 4, 2025

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

Update Colorizer/ColorizingArtist to work with MultiNorm

23fb63c

improved testing for colorizer+multinormApply suggestions from code reviewCo-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>updates based on feedback from@QuLogic

trygvrad force-pushed thecolorizer-multinorm branch from525314e to23fb63cCompare

September 8, 2025 07:20

github-actionsbot removed the status: needs rebase label

Sep 8, 2025

trygvrad moved this toIn Progress inBivariate and Multivariate Colormapping

Sep 8, 2025

This was referencedSep 8, 2025

Preparations for multivariate plotting#29877

Closed

Imshow, pcolor and pcolormesh with Bivariate and Multivariate colormaps#30526

Open

QuLogic reviewed

Sep 9, 2025

View reviewed changes

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

lib/matplotlib/tests/test_colors.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

Updates based on feedback from@QuLogic

d5c43e0

trygvrad force-pushed thecolorizer-multinorm branch fromeb37db8 tod5c43e0Compare

September 11, 2025 07:52

QuLogic approved these changes

Sep 11, 2025

View reviewed changes

Copy link

Member

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.

trygvrad mentioned this pull request

Sep 24, 2025

Multivar imshow#30597

Open

2 tasks

story645 reviewed

Oct 10, 2025

View reviewed changes

Copy link

Member

story645 left a comment

Choose a reason for hiding this comment

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

Sorry it took so long to get to this.

lib/matplotlib/colorizer.py Outdated

		ifnormand (xx.max()>1orxx.min()<0):
		raiseValueError("Floating point image RGB values "
		"must be in the 0..1 range.")
		"must be in the 0..1 range")

Copy link

Member

story645Sep 18, 2025

Choose a reason for hiding this comment

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

Suggested change

	"must be in the0..1 range")
	"must be in the[0,1] range")

teenyist nit but I find interval notation clearer

Copy link

ContributorAuthor

trygvradOct 11, 2025

Choose a reason for hiding this comment

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

FYI: This required a change to an existing testtest_colors.py→test_scalarmappable_nan_to_rgba(bytes) which had a regex for the string '0..1 range'

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.pyShow resolvedHide resolved

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.py

Comment on lines +899 to +902

		elifn_components==2:
		raiseValueError("Invalid data entry for multivariate data. The data"
		" must contain complex numbers, or have a first dimension 2,"
		" or be of a dtype with 2 fields")

Copy link

Member

story645Oct 10, 2025

Choose a reason for hiding this comment

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

I'm not following what 2 components means in this context

Copy link

ContributorAuthor

trygvradOct 11, 2025

Choose a reason for hiding this comment

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

We had a discussion when declaring MultiNorm that we should refer to the different channels/variates as components, so MultiNorm has the propertyn_components which is the number of channels/variates/components.

In the following example it would be 2, because the colormap supports 2 channels, and we provide 2 datasets.

plt.imshow((A, B), cmap='BiOrangeBlue')

Copy link

Member

story645Oct 24, 2025

Choose a reason for hiding this comment

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

Thanks! Sorry for forgetting this. :/

Copy link

ContributorAuthor

trygvrad commentedOct 11, 2025

Sorry it took so long to get to this.

No worries, thank you for getting around to it :)

Apply suggestions from code review

9409d4c

Co-authored-by: hannah <story645@gmail.com>

trygvrad force-pushed thecolorizer-multinorm branch fromcf9296d to9409d4cCompare

October 11, 2025 18:42

story645 approved these changes

Oct 24, 2025

View reviewed changes

Copy link

Member

story645 left a comment

Choose a reason for hiding this comment

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

Comments but definitely not blocking, will leave unmerged for a day but they can also be addressed on followup. Thanks for all your patience and persistence.

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

lib/matplotlib/colorizer.py

Comment on lines +899 to +902

		elifn_components==2:
		raiseValueError("Invalid data entry for multivariate data. The data"
		" must contain complex numbers, or have a first dimension 2,"
		" or be of a dtype with 2 fields")

Copy link

Member

story645Oct 24, 2025

Choose a reason for hiding this comment

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

Thanks! Sorry for forgetting this. :/

Copy link

ContributorAuthor

trygvrad commentedOct 29, 2025

@story645 I added the test we discussed, please take a look at the last commit :)

QuLogic reviewed

Oct 29, 2025

View reviewed changes

lib/matplotlib/colorizer.py OutdatedShow resolvedHide resolved

Added test for cmap.n_variates == 0 in colorizer

6934853

trygvrad force-pushed thecolorizer-multinorm branch from8745b81 to6934853Compare

October 29, 2025 20:41

story645 added this to thev3.11.0 milestone

Oct 30, 2025

story645 merged commit78617c1 intomatplotlib:main

Oct 30, 2025

35 of 39 checks passed

github-project-automationbot moved this fromIn Progress toDone inBivariate and Multivariate Colormapping

Oct 30, 2025

Copy link

Member

story645 commentedOct 30, 2025

doc build failures seem to be intersphinx related

Labels

topic: color/color & colormaps

Movatterモバイル変換

Uh oh!

Update Colorizer/ColorizingArtist to work with MultiNorm#30511

Update Colorizer/ColorizingArtist to work with MultiNorm#30511

Conversation

trygvrad commentedSep 3, 2025• editedLoading Uh oh!There was an error while loading.Please reload this page.

Uh oh!

PR checklist

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

trygvrad commentedSep 4, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

QuLogic left a comment

Choose a reason for hiding this comment

Uh oh!

story645 left a comment

Choose a reason for hiding this comment

Uh oh!

story645Sep 18, 2025

Choose a reason for hiding this comment

Uh oh!

trygvradOct 11, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

story645Oct 10, 2025

Choose a reason for hiding this comment

Uh oh!

trygvradOct 11, 2025

Choose a reason for hiding this comment

Uh oh!

story645Oct 24, 2025

Choose a reason for hiding this comment

Uh oh!

trygvrad commentedOct 11, 2025

Uh oh!

story645 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

story645Oct 24, 2025

Choose a reason for hiding this comment

Uh oh!

trygvrad commentedOct 29, 2025

Uh oh!

Uh oh!

Uh oh!

story645 commentedOct 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

trygvrad commentedSep 3, 2025•
edited
Loading