#9332: S_class_group() should return a group
--------------------------------------------------------------------------------------------+
   Reporter:  stankewicz                                                        
            |       Owner:  davidloeffler
       Type:  enhancement                                                       
            |      Status:  needs_review 
   Priority:  major                                                             
            |   Milestone:               
  Component:  number fields                                                     
            |    Keywords:               
     Author:  Jim Stankewicz, Erin Beyerstedt, Anna Haensch, Robert Miller, 
David Loeffler  |    Upstream:  N/A          
   Reviewer:  John Cremona                                                      
            |      Merged:               
Work_issues:                                                                    
            |  
--------------------------------------------------------------------------------------------+
Changes (by cremona):

  * reviewer:  => John Cremona


Comment:

 Two trivial doctest failures
 {{{
 sage -t -long sage/rings/polynomial/polynomial_quotient_ring.py
 **********************************************************************
 File "/home/jec/sage-4.6.1.alpha2/devel/sage-
 main/sage/rings/polynomial/polynomial_quotient_ring.py", line 737:
     sage: K.class_group()
 Expected:
     Class group of order 1 with structure  of Number Field in a with
 defining polynomial x^2 + 3
 Got:
     Class group of order 1 of Number Field in a with defining polynomial
 x^2 + 3
 }}}
 and
 {{{
 sage -t -long sage/rings/number_field/number_field_ideal.py
 **********************************************************************
 File "/home/jec/sage-4.6.1.alpha2/devel/sage-
 main/sage/rings/number_field/number_field_ideal.py", line 1051:
     sage: I._S_ideal_class_log([])
 Expected:
     [3]
 Got:
     [1]
 }}}
 where in both cases the "Expected " does not look right anyway!

 I am leaving this as needs review since I have not looked closely again at
 the code, but I can assert that apart from the above two things all (long)
 tests pass with 4.6.1.alpha2.

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