Skip to content

BUG: Fix function signature mismatch for set_clim - #13789

Merged
dstansby merged 1 commit into
matplotlib:masterfrom
ksunden:set_clim
Mar 29, 2019
Merged

dstansby merged 1 commit into
matplotlib:masterfrom
ksunden:set_clim

Conversation

@ksunden

@ksunden ksunden commented Mar 29, 2019

Copy link
Copy Markdown
Member

The dummy parent class introduced in #13234 had a set_clim with a
different signature to the version in ScalarMappable.
This caused code which called it and passed both arguments to fail,
rather than simply no-op.

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

The dummy parent class introduced in matplotlib#13234 had a `set_clim` with a
different signature to the version in `ScalarMappable`.
This caused code which called it and passed both arguments to fail,
rather than simply no-op.
@anntzer anntzer added the Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. label Mar 29, 2019
@anntzer anntzer added this to the v3.1.0 milestone Mar 29, 2019

@dstansby dstansby left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@dstansby
dstansby merged commit fc6ea85 into matplotlib:master Mar 29, 2019
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Mar 29, 2019
dstansby added a commit that referenced this pull request Mar 29, 2019
…789-on-v3.1.x

Backport PR #13789 on branch v3.1.x (BUG: Fix function signature mismatch for set_clim)
@ksunden
ksunden deleted the set_clim branch March 29, 2019 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants