#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:  David Loeffler                                |    Upstream:  N/A 
         
   Reviewer:  Francis Clarke, John Cremona, Jim Stankewicz  |      Merged:      
         
Work_issues:                                                |  
------------------------------------------------------------+---------------
Changes (by fwclarke):

  * status:  needs_review => needs_work


Comment:

 Replying to [comment:11 davidloeffler]:
 > This patch handles the "proof" argument in a slightly different way in
 order to make sure that bnfcertify isn't called unnecessarily.

 I don't think this is quite right.  I think the last two lines of the code
 for `_ideal_class_log` need to read
 {{{
             self.__ideal_class_log[proof] = list(v[0])
             return self.__ideal_class_log[proof]
 }}}

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