#5778: [with patch; needs review] More p-adic doctests
---------------------------+------------------------------------------------
Reporter: roed | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-3.4.2
Component: number theory | Keywords: doctests
---------------------------+------------------------------------------------
Comment(by mabshoff):
The patch is not ready to be build with ReST, David just started changing
the doctests as a step in the right direction. About 2/3 of the patch are
indentation changes, the other 1/3 adds new doctests. I have taken a look
a the patch via a local diff viewer since as you point out the patch is
too large for trac.
Given your reservations I would like someone else to take another look, so
I am setting this to "needs review" again.
David is also working on subsequent patches to add more doctests and his
eventual goal here is to get all of padics 100% tested and in the
reference manual. It might be a good idea to stay below 256kb/patch in the
future ;)
Cheers,
Michael
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5778#comment:14>
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
-~----------~----~----~----~------~----~------~--~---
