#5778: [with patch; needs review] Get p-adic doctest coverage to 100% (depends
on
#5105)
--------------------+-------------------------------------------------------
Reporter: roed | Owner: roed
Type: defect | Status: new
Priority: major | Milestone: sage-4.0
Component: padics | Keywords: doctests
--------------------+-------------------------------------------------------
Comment(by robertwb):
I've finished reading everything.
{{{PowComputer_ext._pow_ZZ_tmp_demo}}} is basically a verbatim copy of
{{{PowComputer_class._pow_ZZ_tmp_demo}}}
{{{pAdicZZpXFMElement._teichmuller_set}}} and
{{{PowComputer_ZZ_pX.teichmuller_set_c}}} have nearly the same exact
doctest, as do {{{pAdicCappedRelativeElement._to_gen}}} and
{{{pAdicCappedRelativeElement._pari_}}}. I can see why one would want to
test it in both places, but it would be better to test with distinct
elements. There's probably others, I just get a very deja-vu feeling
reading some of these doctests...
How many more files need to be ReSTified? If its more than a one or two,
lets defer doing this to a later ticket so we can focus on getting this
in.
Your referee patch looks good too, and does address most of my concerns.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5778#comment:43>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
