#20214: Type inconsistencies in polynomial factorization
-------------------------------------+-------------------------------------
Reporter: vdelecroix | Owner:
Type: defect | Status: needs_work
Priority: major | Milestone: sage-7.1
Component: algebra | Resolution:
Keywords: | Merged in:
Authors: Benjamin Hackl | Reviewers:
Report Upstream: N/A | Work issues:
Branch: | Commit:
u/vdelecroix/20214 | d1583d8cedac98c38e0fdad3e4dc083161cf0242
Dependencies: | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by tscrim):
The biggest issue seems to be that the matrix group code was relying on
the current behavior for its factorization. Although I am somewhat more
inclined to think the current behavior is the correct way to go because
the units do not have to live in the base ring in general. Also from
looking at the code, I don't see this fixing things if you work over
`QQ[]`.
BTW, you should use `.one()`. It is usually cached, making construction of
units in more non-trivial rings much faster and doesn't have to use the
coercion framework.
--
Ticket URL: <http://trac.sagemath.org/ticket/20214#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 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.
