-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Possibly excessive transform creation #12031
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
Comments
When creating composite transforms, would it work if we checked parents and returned the one that already existed and contained the two transforms? I think these are immutable, and there's already some shortcuts to avoid adding redundant |
This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help! |
@anntzer You have worked on transforms quite recently; do you know whether this issue is now obsolete? If it isn't, I'm tempted to reopen it. |
I don't think it's obsolete. |
This issue has been marked "inactive" because it has been 365 days since the last comment. If this issue is still present in recent Matplotlib releases, or the feature request is still wanted, please leave a comment and this label will be removed. If there are no updates in another 30 days, this issue will be automatically closed, but you are free to re-open or create a new issue if needed. We value issue reports, and this procedure is meant to help us resurface and prioritize issues that have not been addressed yet, not make them disappear. Thanks for your help! |
Follow-up to #11972, which solves a memory leak, but leaves the question of whether we can improve performance by reducing the number of transform instantiations.
It was pointed out that simply moving the cursor in an otherwise static plot is enough to cause calls to
Transform.set_children
, which occurs when transforms are chained or blended.The text was updated successfully, but these errors were encountered: