Skip to content

Commit 6fdd6da

Browse files
author
Release Manager
committed
sagemathgh-40472: src/sage/symbolic/random_tests.py: fewer random test repetitions We currently run ``` sage: check_symbolic_expression_order(200) sage: check_symbolic_expression_order(10000) # long time ``` in succession, for a total of 10200 total repetitions of this random test. The second one is slow and is raising warnings in the CI: ``` 2025-07-24T02:19:27.8495248Z ##[warning]slow doctest: 2025-07-24T02:19:27.8496718Z check_symbolic_expression_order(10000) 2025-07-24T02:19:27.8497325Z Test ran for 34.32s cpu, 34.36s wall 2025-07-24T02:19:27.8498929Z Check ran for 0.00s cpu, 0.00s wall ``` This commit combines the two lines into a single test, ``` sage: check_symbolic_expression_order(500) # long time ``` to eliminate the warning. We run these tests on every pull request, so if there is a problem, we will catch it eventually, even with fewer repetitions per run. And anyone messing with the ordering on SR can of course run the tests with as many repetitions as he likes. URL: sagemath#40472 Reported by: Michael Orlitzky Reviewer(s):
2 parents 1bd6ba7 + 4fc99c0 commit 6fdd6da

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/sage/symbolic/random_tests.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -430,8 +430,7 @@ def check_symbolic_expression_order(repetitions=100):
430430
EXAMPLES::
431431
432432
sage: from sage.symbolic.random_tests import check_symbolic_expression_order
433-
sage: check_symbolic_expression_order(200)
434-
sage: check_symbolic_expression_order(10000) # long time
433+
sage: check_symbolic_expression_order(500) # long time
435434
"""
436435
rnd_length = 50
437436
nvars = 10

0 commit comments

Comments
 (0)