Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix configure subplots#18306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

richardsheridan
Copy link
Contributor

PR Summary

Fixes#18299. Draft for the moment because I still need to see if we can keep#17025 from haunting us. For the moment the reference to the subplot_tool is kept on the navbar; I think that makes more sense than an artificial cycle, as the navbar is responsible for spawning the tool anyway.

PR Checklist

  • Has Pytest style unit tests
  • Code isFlake 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/next_api_changes/* if API changed in a backward-incompatible way

@richardsheridan
Copy link
ContributorAuthor

Found a different way to fix#17025. I don't really trust CI on this, can I get someone else to help manually check nbagg is working after5f6a785?

@richardsheridanrichardsheridan marked this pull request as ready for reviewAugust 20, 2020 13:08
Copy link
Member

@QuLogicQuLogic left a comment

Choose a reason for hiding this comment

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

Can confirm that nbAgg is not broken.

@QuLogicQuLogic added this to thev3.4.0 milestoneAug 22, 2020
@dopplershift
Copy link
Contributor

Needs a rebase.

richardsheridan reacted with thumbs up emoji

@jklymakjklymak merged commit0e87f48 intomatplotlib:masterAug 24, 2020
@richardsheridanrichardsheridan deleted the fix_configure_subplots branchAugust 24, 2020 16:07
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@dopplershiftdopplershiftdopplershift approved these changes

@QuLogicQuLogicQuLogic approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.4.0
Development

Successfully merging this pull request may close these issues.

New configure_subplots behaves badly on TkAgg backend
4 participants
@richardsheridan@dopplershift@QuLogic@jklymak

[8]ページ先頭

©2009-2025 Movatter.jp