#15027: Inconsistent primitive_root_of_unity for number fields
-------------------------------------+--------------------------
       Reporter:  cremona            |         Owner:  cremona
           Type:  defect             |        Status:  new
       Priority:  major              |     Milestone:  sage-5.12
      Component:  number fields      |    Resolution:
       Keywords:  number field unit  |     Merged in:
        Authors:  John Cremona       |     Reviewers:
Report Upstream:  N/A                |   Work issues:
         Branch:                     |  Dependencies:
       Stopgaps:                     |
-------------------------------------+--------------------------

Comment (by fwclarke):

 John,

 The patch doesn't seem to eliminate the other inconsistency: between
 `-1/2*a + 1/2` and `1/2*a + 1/2`.  For when `primitive_root_of_unity` is
 called before `unit_group` it selects the primitive with the shortest
 string representation.  On the other hand, `UnitGroup.__init__` just uses
 the element given by Pari's `nfrootsof1`.  There are clearly a variety of
 ways to sort this out.

--
Ticket URL: <http://trac.sagemath.org/ticket/15027#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 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 http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to