#4637: [with-patch; needs review] bug/stupid design of padic_printing.sep print
mode stuff
---------------------------+------------------------------------------------
 Reporter:  was            |       Owner:  was       
     Type:  defect         |      Status:  new       
 Priority:  major          |   Milestone:  sage-3.4.2
Component:  number theory  |    Keywords:            
---------------------------+------------------------------------------------

Comment(by ncalexan):

 First comment: your choice of ... for eliding terms in printing conflicts
 with the doctest framework; could that be .. or something different?  For
 example, in factory.py around line 150,

 {{{
         sage: T = Qp(5, print_mode='series', print_max_terms=4); b =
 R(-70700); repr(b)
         '2*5^2 + 4*5^3 + 5^4 + 2*5^5 + ... + O(5^22)'
 }}}

 is *not* testing what you think it is -- the doctester matches ... inside
 of strings!  (You've typoed, and you meant {{{T(-70700)}}}.)

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4637#comment:4>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to