#14284: Sampling in unit tests
-------------------------------+--------------------------------------------
Reporter: roed | Owner: jason
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.9
Component: misc | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: Julian Rueth
Authors: David Roe | Merged in:
Dependencies: #14285 | Stopgaps:
-------------------------------+--------------------------------------------
Comment (by nthiery):
Replying to [comment:10 roed]:
> Sounds okay to me: I'll make a new version. There will be a couple
> tests that can't use this idiom (`_test_eq_symmetric` for example),
> but they can certainly use the current approach.
Thanks!
By the way: there are quite a few spots where _test_associativity is
disabled, precisely because they are too expensive (either due to too
many elements, but also in some cases because of high degree
calculations). See:
grep -r _test_associativity . | grep skip
You might want to play around with those and maybe fix a couple if
things work better now.
> Since Sage sets the random seed at the beginning of each run, tests
> that use randomness should be reproducible....
Unless some edit elsewhere in the file changes the order in which
things are run. Or if one wants to reproduce an error in the terminal
(hopefully not so critical with the upcoming doctest framework if one
can finally explore a failing test with the debugger).
But I see the advantages too.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14284#comment:11>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.
