#9244: Number field class group improvements
-----------------------------+----------------------------------------------
   Reporter:  davidloeffler  |       Owner:  davidloeffler
       Type:  defect         |      Status:  needs_work   
   Priority:  major          |   Milestone:  sage-4.5     
  Component:  number fields  |    Keywords:               
     Author:                 |    Upstream:  N/A          
   Reviewer:                 |      Merged:               
Work_issues:                 |  
-----------------------------+----------------------------------------------
Changes (by fwclarke):

  * status:  needs_review => needs_work


Comment:

 These are definite improvements, well implemented, and doctests pass.
 Just one remark:

 The eventuality envisaged in the comment at line 95 of the
 patched `class_group.py` (which should be referring to ideal classes
 rather ideals) ought to have its own doctest, such as:

 {{{
 sage: K.<a> = QuadraticField(-23)
 sage: L.<b> = K.extension(x^2 - 2)
 sage: CK = K.class_group()
 sage: CL = L.class_group()
 sage: [CL(I).list() for I in CK]
 [[0], [2], [4]]
 }}}

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