#11770: Add ability to detect CM number fields and take complex conjugate of 
their
elements
---------------------------------+------------------------------------------
       Reporter:  robharron      |         Owner:  tba           
           Type:  enhancement    |        Status:  needs_review  
       Priority:  minor          |     Milestone:  sage-5.3      
      Component:  number fields  |    Resolution:                
       Keywords:  CM field       |   Work issues:                
Report Upstream:  N/A            |     Reviewers:  Francis Clarke
        Authors:  Robert Harron  |     Merged in:                
   Dependencies:                 |      Stopgaps:                
---------------------------------+------------------------------------------
Changes (by robharron):

  * status:  needs_work => needs_review


Comment:

 I fixed the first two issues as suggested (for the second issue, I used
 that because that's how conjugate *currently* computes its answer for
 cyclotomic fields, but you're right taking the inverse is *significantly*
 quicker for large N). For the third issue, I've amended the code (what I
 had done was quite silly...) to use the list of divisors sorted in reverse
 order. For the final issue, I actually find it more readable the way I did
 it. To each their own I guess!

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