#20028: sorting of number field elements
-------------------------------------+-------------------------------------
       Reporter:  cremona            |        Owner:
           Type:  defect             |       Status:  new
       Priority:  major              |    Milestone:  sage-7.1
      Component:  number fields      |   Resolution:
       Keywords:  sort number field  |    Merged in:
  elements                           |    Reviewers:
        Authors:                     |  Work issues:
Report Upstream:  N/A                |       Commit:
         Branch:  u/cremona/20028    |  ae87261971f6ddc93d1d40c24c3a141be58457bd
   Dependencies:                     |     Stopgaps:
-------------------------------------+-------------------------------------

Comment (by cremona):

 Nils, can you explain the following?  Starting with the most recent commit
 above, if you make one change to sage/structure/factorization.py
 {{{
 @@ -319,10 +319,7 @@ class Factorization(SageObject):
          self.__unit = unit
          self.__cr = cr
          if sort and self.is_commutative():
 -            try:
 -                self.sort()
 -            except NotImplementedError:
 -                pass
 +            self.sort()
 }}}
 then there are many failures, for example
 {{{
 sage: pol26 = hilbert_class_polynomial(-4*26)
 sage: K = NumberField(pol26,'a')
 sage: K.optimized_representation()
 <boom>
 NotImplementedError: root finding for this polynomial not implemented
 }}}

--
Ticket URL: <http://trac.sagemath.org/ticket/20028#comment:28>
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.

Reply via email to