kendryte / nncase

Open deep learning compiler stack for Kendryte AI accelerators ✨
Apache License 2.0
747 stars 181 forks source link

Fix memory leak: Avoid disposing python stream in finalizer thread #1142

Closed sunnycase closed 10 months ago

github-actions[bot] commented 10 months ago

Test Results

5 859 tests  ±0   5 859 :heavy_check_mark: ±0   7m 35s :stopwatch: -18s        6 suites ±0          0 :zzz: ±0         6 files   ±0          0 :x: ±0 

Results for commit c71f9f10. ± Comparison against base commit c4f7b5fb.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (c4f7b5f) 73.23% compared to head (c71f9f1) 73.23%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1142 +/- ## ========================================== - Coverage 73.23% 73.23% -0.01% ========================================== Files 707 707 Lines 25801 25804 +3 Branches 4207 4207 ========================================== + Hits 18896 18898 +2 - Misses 6264 6266 +2 + Partials 641 640 -1 ``` | [Flag](https://app.codecov.io/gh/kendryte/nncase/pull/1142/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kendryte) | Coverage Δ | | |---|---|---| | [integration](https://app.codecov.io/gh/kendryte/nncase/pull/1142/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kendryte) | `39.12% <100.00%> (+<0.01%)` | :arrow_up: | | [unit](https://app.codecov.io/gh/kendryte/nncase/pull/1142/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kendryte) | `65.08% <0.00%> (-0.01%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=kendryte#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.