Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Nov 2, 2022

... instead of triplicating it on the caller side.

PR Summary

PR Checklist

Tests and Styling

  • Has pytest style unit tests (and pytest passes).
  • Is Flake 8 compliant (install flake8-docstrings and run flake8 --docstring-convention=all).

Documentation

  • 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: geometry manager LayoutEngine, Constrained layout, Tight layout Maintenance labels Nov 2, 2022
"compatible with tight_layout, so results "
"might be incorrect.")

from ._tight_layout import get_subplotspec_list
Copy link
Member

Choose a reason for hiding this comment

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

We already have _tight_layout imported, and this call would fit in a line...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

indeed.

... instead of triplicating it on the caller side.
@ksunden ksunden added this to the v3.6.3 milestone Nov 3, 2022
@ksunden ksunden merged commit 6c09a56 into matplotlib:main Nov 3, 2022
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Nov 3, 2022
@anntzer anntzer deleted the tlw branch November 3, 2022 16:44
tacaswell added a commit that referenced this pull request Nov 3, 2022
…343-on-v3.6.x

Backport PR #24343 on branch v3.6.x (Emit "axes not compatible with tight_layout" in a single place.)
@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
Maintenance topic: geometry manager LayoutEngine, Constrained layout, Tight layout
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants