#12662: Improve doctest coverage for sage.rings.qqbar
----------------------------------+-----------------------------------------
Reporter: davidloeffler | Owner: mvngu
Type: defect | Status: needs_info
Priority: major | Milestone: sage-5.0
Component: doctest | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: David Loeffler | Merged in:
Dependencies: | Stopgaps:
----------------------------------+-----------------------------------------
Changes (by davidloeffler):
* status: needs_work => needs_info
* work_issues: doctest failure =>
Comment:
Dear JStarx,
I strongly suspect you applied the patch and ran "sage -t" without running
"sage -b" first, which gives exactly these two errors. (They come from the
two non-docstring changes in the patch: I renamed {{{ late_import }}} to
{{{ _late_import }}} so it wouldn't appear in the reference manual, and
added one line to {{{ AlgebraicGenerator.pari_polynomial }}} so it gave a
more informative error message when called on the rational field.)
Can you run "sage -b" and then do the test again?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12662#comment:8>
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.
