FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

BUG: Fix function signature mismatch for set_clim by ksunden · Pull Request #13789 · matplotlib/matplotlib · GitHub

Repository navigation

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 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 added the Release critical For bugs that make the library unusable (segfaults, incorrect plots, etc) and major regressions. label Mar 29, 2019
anntzer added this to the v3.1.0 milestone Mar 29, 2019

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks!

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 deleted the set_clim branch March 29, 2019 14:31
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL