#9359: Get number field coverage up to 100%
------------------------------+---------------------------------------------
   Reporter:  davidloeffler   |       Owner:  davidloeffler
       Type:  enhancement     |      Status:  needs_review 
   Priority:  major           |   Milestone:  sage-4.6     
  Component:  number fields   |    Keywords:  doctest      
     Author:  David Loeffler  |    Upstream:  N/A          
   Reviewer:                  |      Merged:               
Work_issues:                  |  
------------------------------+---------------------------------------------

Comment(by jdemeyer):

 Also, the hunk
 {{{
 -            sage: I.ideal_below()
 -            Fractional ideal (-b)  # 32-bit
 -            Fractional ideal (b)   # 64-bit
 +            sage: I.ideal_below() == F.ideal(b)
 +            True
 }}}
 conflicts with #9764. So would you mind removing it?

 Apart from that, the patch applies fine and doctests fine, even with #9753
 and #9764 applied.

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

Reply via email to