Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork8.1k
MNT: Make useblit a property in Widget classes#30605
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
Conversation
The goal is to skip trying to do anything with blitting whenafter the figure is closed a blitting-capable Canvas instance isswapped out for one that does not support blitting.closesmatplotlib#30575 but in only a partial solution to the bigger issueswith blitting and changing the canvas.
I think this PR is a bit dangerous by itself. It makes sure that the the I've taken a more comprehensive approach in#30591, see in particular#30591 (comment) |
Closing this in favor of#30591. Sorry for the duplicate effort. |
The goal is to skip trying to do anything with blitting when after the figure is closed a blitting-capable Canvas instance is swapped out for one that does not support blitting.
closes#30575 but in only a partial solution to the bigger issues with blitting and changing the canvas.
PR summary
Attempts to make
useblitdynamic (and standardiizes it across all of the widget classes)Not tested with MNE's case and have not sorted out a MWE to add a test.
PR checklist