memray icon indicating copy to clipboard operation
memray copied to clipboard

Update test_exercise_3.py

Open jcarnaxide opened this issue 7 months ago • 1 comments

#743 testing in Python version 3.13, the solution uses a bit more memory than previously in Python version 3.12.

I saw an increase up to about 170MB of memory used for my run.

The provided example which is meant to use too much memory uses about 740MB.

So I think a new threshold of 200MB should be sufficient. This will also be backwards compatible.

jcarnaxide avatar Apr 14 '25 17:04 jcarnaxide

Codecov Report

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

Project coverage is 92.82%. Comparing base (9884c9d) to head (061e0c9).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #754      +/-   ##
==========================================
- Coverage   92.94%   92.82%   -0.12%     
==========================================
  Files          95       95              
  Lines       11686    11686              
  Branches      407      407              
==========================================
- Hits        10861    10848      -13     
- Misses        825      838      +13     
Flag Coverage Δ
cpp 92.82% <ø> (-0.12%) :arrow_down:
python_and_cython 92.82% <ø> (-0.12%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Apr 14 '25 17:04 codecov-commenter