#17840: Factorization of multivariate polynomials over the integers
-------------------------------------+-------------------------------------
Reporter: bruno | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-6.6
Component: factorization | Resolution:
Keywords: multivariate | Merged in:
integer polynomial | Reviewers: Jeroen Demeyer
Authors: Bruno Grenet | Work issues:
Report Upstream: N/A | Commit:
Branch: | 0e773f480e56b1b6496ab99f0a9d34623c364998
u/bruno/factorization_of_multivariate_polynomials_over_the_integers|
Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Changes (by jdemeyer):
* status: needs_review => needs_work
* reviewer: => Jeroen Demeyer
Comment:
Some comments:
1. Replace `raise Exception, "Message."` by `raise Exception("message")`.
2. Is the check for integral domain really needed? If it's not an integral
domain, it will be caught by the branch below.
3. Why did you add the check for symbolic ring? If it's needed, there
should also be a doctest added for this case.
4. The message `Factorization of multivariate polynomials over Symbolic
Ring is not implemented. Consider using multivariate polynomial rings
instead.` is confusing, it essentially says "factorization of multivariate
polynomials is not supported, use multivariate polynomials instead".
--
Ticket URL: <http://trac.sagemath.org/ticket/17840#comment:15>
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/d/optout.
