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

CI: re-consider how we test GUI memory leaks #22988

Open
@tacaswell

Description

@tacaswell

#22002 fixed some (but not all) memory leaks it tkagg as well as added tests to put an upper bound on the amount of memory that lost when making GUI windows in a tight Python loop and only periodically running the event loop.

However after merging we found that these tests were found that the memory consumed had significant variability on OSX on both the OSX and tkagg backends (bug report:#22959 patches:#22987 ,#22961 ). While pragmatic, bumping the threshold is giving a reminder of when our image comparisons had some tolerance in them and we ended up using the wrong glyph for a while even though we had an explicit test (it passed with in tolerance!). It is possible I have over-learned this lesson and should chill.

When talking about this on the dev call this week we had at least one wrong idea about the source of the variability and we still do not understand it. Due to the variability it is easily possible that a small leak will not be robustly detectable under the noise and hence the test would not actually test what we want to test. We could get around that my running the tests for much longer (like we do inhttps://github.com/matplotlib/matplotlib/blob/main/tools/memleak.py) , however that would result in a test that runs for 10s or greater which is very much not ideal for a test suite (either localy or on CI).

To this end we had a couple of ideas:

  • push the memory leakage testing into its own CI job (and only run it on merge / on a clock)?
  • use ASV to run these tests
    • maybe on macstadium, maybe on ARM test box we have, maybe on digital ocean

In the very short term I think we should

  1. mergeCI: bump test limit from tkagg on osx #22987
  2. if 1. fails, skip these tests on osx
  3. If we think 2. too much just hiding the bad case under the carpet, pull these tests out and get one of the above ideas done ASAP.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


      [8]ページ先頭

      ©2009-2025 Movatter.jp