Skip to content

math.comb is leaking references #81306

Closed
Closed

Description

@pablogsal
BPO 37125
Nosy @rhettinger, @mdickinson, @serhiy-storchaka, @pablogsal

Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2019-06-01.20:08:35.448>
created_at = <Date 2019-06-01.16:31:17.453>
labels = ['interpreter-core', '3.8', 'tests']
title = 'math.comb is leaking references'
updated_at = <Date 2019-06-01.20:26:09.625>
user = 'https://github.com/pablogsal'

bugs.python.org fields:

activity = <Date 2019-06-01.20:26:09.625>
actor = 'rhettinger'
assignee = 'none'
closed = True
closed_date = <Date 2019-06-01.20:08:35.448>
closer = 'pablogsal'
components = ['Interpreter Core', 'Tests']
creation = <Date 2019-06-01.16:31:17.453>
creator = 'pablogsal'
dependencies = []
files = []
hgrepos = []
issue_num = 37125
keywords = []
message_count = 4.0
messages = ['344185', '344187', '344214', '344220']
nosy_count = 4.0
nosy_names = ['rhettinger', 'mark.dickinson', 'serhiy.storchaka', 'pablogsal']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue37125'
versions = ['Python 3.8']

Activity

  1. pablogsal commented on Jun 1, 2019

    @pablogsal
    MemberAuthor

    ./python Lib/test/bisect_cmd.py test_math -R 2:2

    ...

    WARNING: Running tests with --huntrleaks/-R and less than 3 warmup repetitions can give false positives!
    Run tests sequentially
    0:00:00 load avg: 1.32 [1/1] test_math
    beginning 4 repetitions
    1234
    ....
    test_math leaked [12352, 12352] references, sum=24704
    test_math leaked [1, 1] memory blocks, sum=2
    test_math failed

    == Tests result: FAILURE ==

    1 test failed:
    test_math

    Total duration: 663 ms
    Tests result: FAILURE
    ran 1 tests/2
    exit 2
    Tests failed: continuing with this subtest

    Tests (1):

    • test.test_math.IsCloseTests.testComb

    Bisection completed in 13 iterations and 0:00:11

  2. added
    interpreter-core(Objects, Python, Grammar, and Parser dirs)
    testsTests in the Lib/test dir
    on Jun 1, 2019
  3. pablogsal commented on Jun 1, 2019

    @pablogsal
    MemberAuthor

    It seems that #13725 fixes the leak:

    ❯ ./python -m test test_math -R :
    Run tests sequentially
    0:00:00 load avg: 1.55 [1/1] test_math
    beginning 9 repetitions
    123456789
    .........

    == Tests result: SUCCESS ==

    1 test OK.

    Total duration: 26 sec 283 ms
    Tests result: SUCCESS

  4. pablogsal commented on Jun 1, 2019

    @pablogsal
    MemberAuthor

    Fixed by #13725

  5. rhettinger commented on Jun 1, 2019

    @rhettinger
    Contributor

    Please file this in the tracker issue so that the discussions don't sprawl. Thx.

  6. transferred this issue fromon Apr 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.8 (EOL)end of lifeinterpreter-core(Objects, Python, Grammar, and Parser dirs)testsTests in the Lib/test dir

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions