#9494: Add a doctest to benchmark.py
---------------------------------------------------------------------+------
Reporter: demosd235 |
Owner: mvngu
Type: defect |
Status: needs_work
Priority: major |
Milestone: sage-4.7.2
Component: documentation |
Keywords: sd32
Work_issues: Fix `mpoly_all()` or mark the doctest `# not tested`. |
Upstream: N/A
Reviewer: Rob Beezer, Benjamin Jones |
Author: Robert Miller
Merged: |
Dependencies:
---------------------------------------------------------------------+------
Changes (by leif):
* status: positive_review => needs_work
* work_issues: => Fix `mpoly_all()` or mark the doctest `# not tested`.
Comment:
Actually, `mpoly_all()` needs work, since it is apparently unable to
interrupt (e.g.) Mathematica a second time.
On redhawk, where `/home` still isn't mounted, Mathematica is installed
but fails to startup (prompting the user), such that I get:
{{{
#!sh
leif@redhawk:/$ time sage -c "from sage.tests.benchmark import mpoly_all;
mpoly_all()"
Compute (x_0 + ... + x_99) * (x_100 + ... + x_200) over Rational Field
(use singular for Sage mult.)
System min avg max trials cpu or
wall
* sage 0.049767 0.049767 0.049767 1
0.049767w
Unable to start magma
computation timed out because alarm was set for 60 seconds
Unable to start macaulay2 because the command 'M2 --no-debug --no-readline
--silent -e 'ZZ#{Standard,Core#"private dictionary"#"InputPrompt"} =
lineno -> "_EGAS_ : ";ZZ#{Standard,Core#"private
dictionary"#"InputContinuationPrompt"} = lineno -> "_EGAS_ : ";printWidth
= 0;lineNumber = 10^9;'' failed.
Compute (x_0 + ... + x_199) * (x_200 + ... + x_400) over Rational Field
(use singular for Sage mult.)
System min avg max trials cpu or
wall
* sage 0.089736 0.089736 0.089736 1
0.089736w
Unable to start magma
Interrupting Mathematica...
}}}
which does not terminate.
What is this doctest supposed to "test" at all?
The output is random, so it doesn't really test anything, except perhaps
which external programs are installed, but that's nothing subject to a
doctest, so IMHO this ticket should either be closed as invalid, or the
"doctest" (i.e., the example) should be marked `# not tested`, at least
until `mpoly_all()` got fixed.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/9494#comment:12>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
