Skip to content

Fix MultiNorm handling and colorbar removal without an Axes - #32605

Open
trygvrad wants to merge 1 commit into
matplotlib:mainfrom
trygvrad:mv-fixes
Open

trygvrad wants to merge 1 commit into
matplotlib:mainfrom
trygvrad:mv-fixes

Conversation

@trygvrad

Copy link
Copy Markdown
Contributor

This PR has a long history. It is part of Bivariate and Multivariate Colormapping and contains changes that were previously part of #31214 which is being separated into multiple parts with the use of AI.

PR summary

  • Colorizer: an existing MultiNorm passed as norm is used as is instead of being wrapped in a new MultiNorm, which raised a ValueError.
  • MultiNorm.autoscale_None only emits the 'changed' signal when the limits of a constituent norm actually changed.
  • Colorbar.remove no longer fails when the mappable has no Axes.

AI Disclosure

Calude has been used to re-structure #31214, resulting in this PR

PR quality check

  • Use an expressive title, e.g. "Fix title font property precedence"
  • New and changed code is tested
  • [N/A] Plotting related features are demonstrated in an example
  • [N/A] New features and API changes have release notes
  • [N/A] Documentation complies with general and docstring guidelines

- Colorizer: an existing MultiNorm passed as *norm* is used as is instead
  of being wrapped in a new MultiNorm, which raised a ValueError.
- MultiNorm.autoscale_None only emits the 'changed' signal when the limits
  of a constituent norm actually changed.
- Colorbar.remove no longer fails when the mappable has no Axes.

Changes implemented with the assistance of AI and manually evaluated

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant