#9400: modify the NumberField constructor to pass in optional integer B such 
that
all the internal pari routines will replace the discriminant by its gcd
with B, making some things massively faster.
-----------------------------+----------------------------------------------
   Reporter:  was            |       Owner:  davidloeffler
       Type:  enhancement    |      Status:  needs_review 
   Priority:  major          |   Milestone:  sage-4.5.3   
  Component:  number fields  |    Keywords:               
     Author:                 |    Upstream:  N/A          
   Reviewer:                 |      Merged:               
Work_issues:                 |  
-----------------------------+----------------------------------------------

Comment(by was):

 > Note how the integer is printed first in the first case but last in the
 second case (and personally I find it clearer when the integer is put
 first).  Maybe the sorting and uniqueing should be done in the
 NumberFieldIdeal constructor instead of when the ideal is printed?

 Would that make any difference?  The difference above is that in one case
 the number field has a very small discriminant (-1323), and in the other
 it doesn't (-2700540027).   When the discriminant is small, then reduced
 generators are used for printing.   A solution could be to apply the
 sorting and "uniquing" in both cases before printing -- right now it is
 only applied in the case of large discriminant.

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