#5778: [with patch; needs review] Get p-adic doctest coverage to 100%
---------------------------+------------------------------------------------
Reporter: roed | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: number theory | Keywords: doctests
---------------------------+------------------------------------------------
Comment(by mabshoff):
The last patch fail against 3.4.1:
{{{
patching file sage/rings/padics/padic_generic_element.pyx
Hunk #13 FAILED at 737.
Hunk #14 succeeded at 869 (offset -3 lines).
Hunk #15 succeeded at 898 (offset -3 lines).
Hunk #16 succeeded at 930 (offset -3 lines).
Hunk #17 succeeded at 956 (offset -3 lines).
Hunk #18 FAILED at 1006.
2 out of 18 hunks FAILED -- saving rejects to file
sage/rings/padics/padic_generic_element.pyx.rej
}}}
There are some more issues with 3.4.2.alpha0, but I can revert the changes
from there and just patch them back in after merging padics instead of
rebasing the patch :)
One more thing: You credit Genya Zaytman for writing doctests, but AFAIK
the person has never been credited for contributing to Sage.
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5778#comment:21>
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
-~----------~----~----~----~------~----~------~--~---
