Skip to content

Update backend_qt.py: parent not passed to __init__ on subplottool #29182

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

proximalf
Copy link
Contributor

This PR passes the parent of an initialised SubplotToolQt to its inherited __init__.

It is missed here, but not on other classes.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for opening your first PR into Matplotlib!

If you have not heard from us in a week or so, please leave a new comment below and that should bring it to our attention. Most of our reviewers are volunteers and sometimes things fall through the cracks.

You can also join us on gitter for real-time discussion.

For details on testing, writing docs, and our review process, please see the developer guide

We strive to be a welcoming and open project. Please follow our Code of Conduct.

@ksunden ksunden added this to the v3.10.0 milestone Nov 27, 2024
@ksunden
Copy link
Member

ksunden commented Nov 27, 2024

Failure on azure is known flaky test, Failure on 3.13t is upstream numpy importing (and trying to do docstring things)

@ksunden ksunden merged commit a5c0b86 into matplotlib:main Nov 27, 2024
39 of 42 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Nov 27, 2024
rcomer added a commit that referenced this pull request Nov 30, 2024
…182-on-v3.10.x

Backport PR #29182 on branch v3.10.x (Update backend_qt.py: parent not passed to __init__ on subplottool)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

3 participants