FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Backport PR #22002: Fix TkAgg memory leaks and test for memory growth regressions by tacaswell · Pull Request #22935 · matplotlib/matplotlib · GitHub

Repository navigation

Backport PR #22002: Fix TkAgg memory leaks and test for memory growth regressions - #22935

Merged
QuLogic merged 1 commit into
matplotlib:v3.5.xfrom
tacaswell:auto-backport-of-pr-22002-on-v3.5.x
Apr 29, 2022
Merged

QuLogic merged 1 commit into
matplotlib:v3.5.xfrom
tacaswell:auto-backport-of-pr-22002-on-v3.5.x

Conversation

Copy link
Copy Markdown
Member

FIX: TkAgg memory leaks and test for memory growth regressions (#22002)

tkinter variables get cleaned up with normal destroy and gc semantics but tkinter's implementation of trace is effectively global and keeps the callback object alive until the trace is removed.

Additionally extend and clean up the tests.

Closes #20490

Co-authored-by: Elliott Sales de Andrade quantum.analyst@gmail.com
(cherry picked from commit 1a016f0)

tacaswell added this to the v3.5.2 milestone Apr 29, 2022
tacaswell force-pushed the auto-backport-of-pr-22002-on-v3.5.x branch from 69eb739 to ec6fbfb Compare April 29, 2022 00:06
…ory growth regressions

FIX: TkAgg memory leaks and test for memory growth regressions (matplotlib#22002)

tkinter variables get cleaned up with normal `destroy` and `gc` semantics but tkinter's implementation of trace is effectively global and keeps the callback object alive until the trace is removed.

Additionally extend and clean up the tests.

Closes matplotlib#20490

Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
(cherry picked from commit 1a016f0)
QuLogic merged commit 3653fc1 into matplotlib:v3.5.x Apr 29, 2022
tacaswell deleted the auto-backport-of-pr-22002-on-v3.5.x branch April 30, 2022 00:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL