Skip to content

Deprecate AxisArtistHelpers with inconsistent loc/nth_coord.#24904

Merged
oscargus merged 1 commit into
matplotlib:mainfrom
anntzer:aaf
Jan 9, 2023
Merged

Deprecate AxisArtistHelpers with inconsistent loc/nth_coord.#24904
oscargus merged 1 commit into
matplotlib:mainfrom
anntzer:aaf

Conversation

@anntzer

@anntzer anntzer commented Jan 7, 2023

Copy link
Copy Markdown
Contributor

A "top y-axis" or a "left x-axis" doesn't make sense.

See #22314 (comment).

PR Summary

PR Checklist

Documentation and Tests

  • Has pytest style unit tests (and pytest passes)
  • Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • New plotting related features are documented with examples.

Release Notes

  • New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

@anntzer anntzer added topic: mpl_toolkit API: argument checking Validation of acceptable argument values labels Jan 7, 2023
A "top y-axis" or a "left x-axis" doesn't make sense.
@oscargus oscargus added this to the v3.7.0 milestone Jan 9, 2023
@oscargus oscargus merged commit 7bc69f2 into matplotlib:main Jan 9, 2023
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Jan 9, 2023
@oscargus oscargus modified the milestones: v3.7.0, v3.8.0 Jan 9, 2023
@anntzer anntzer deleted the aaf branch January 9, 2023 08:31
@QuLogic QuLogic modified the milestones: v3.8.0, v3.7.0 Jan 10, 2023
ksunden added a commit that referenced this pull request Jan 10, 2023
…904-on-v3.7.x

Backport PR #24904 on branch v3.7.x (Deprecate AxisArtistHelpers with inconsistent loc/nth_coord.)
@ksunden ksunden mentioned this pull request Feb 20, 2023
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API: argument checking Validation of acceptable argument values topic: mpl_toolkit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants