Skip to content

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Dec 31, 2019

PR Summary

Regression from #14579. As a result setting properties Axis.v_interval and Axis.d_interval did not work.

Comments suggest testing or deprecating. But I don't want to spend more time on this:

  • Re testing: More tests are always nice, but here it's obvious that the fix just fixes a copy-paste bug.
  • Re deprecating: Might make sense. However, additional properties are not that much of an API problem. They just came in via Fix inversion of 3d axis. #14579 when these attibutes were turned into properties.

Therefore I'd like to push this PR as minimal as it is. If anybody want to take further action, feel free.

@timhoffm timhoffm added this to the v3.1.3 milestone Dec 31, 2019
@timhoffm timhoffm mentioned this pull request Dec 31, 2019
@Kojoley Kojoley merged commit 3218b11 into matplotlib:master Jan 7, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 7, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 7, 2020
@timhoffm timhoffm deleted the fix-v_interval_setter branch January 7, 2020 21:25
timhoffm added a commit that referenced this pull request Jan 7, 2020
…053-on-v3.2.x

Backport PR #16053 on branch v3.2.x (Fix v_interval setter)
timhoffm added a commit that referenced this pull request Jan 7, 2020
…053-on-v3.1.x

Backport PR #16053 on branch v3.1.x (Fix v_interval setter)
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.

3 participants