New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Implement quo_rem
for Polynomial_sparse_generic
#16544
Comments
Author: Bruno Grenet |
This comment has been minimized.
This comment has been minimized.
Commit: |
comment:2
I've implemented an Euclidean Division algorithm in the class ArithmeticError: Nonunit leading coefficient if the leading coefficient of the divisor is not invertible, as it is done in the class ZeroDivisionError: Division by zero polynomial is raised. The error in the description of the ticket does not occur anymore: sage: R.<x>=PolynomialRing(ZZ,sparse=True)
sage: p=R.random_element(10)
sage: q=R.random_element(5)
sage: p//q
---------------------------------------------------------------------------
ArithmeticError
...
ArithmeticError: Nonunit leading coefficient
sage: q += x^6
sage: p//q
x^4 + 4*x^3 - 14*x^2 + 6*x + 61 Needs review! |
Reviewer: Travis Scrimshaw |
comment:3
I'm somewhat surprised we didn't catch this earlier. LGTM, positive review. |
Changed branch from u/bruno/implement__quo_rem__for__polynomial_sparse_generic_ to |
It is not possible to compute the Euclidean division for generic sparse polynomials.
Component: commutative algebra
Keywords: sparse polynomial, quo_rem
Author: Bruno Grenet
Branch/Commit:
fb1d21c
Reviewer: Travis Scrimshaw
Issue created by migration from https://trac.sagemath.org/ticket/16544
The text was updated successfully, but these errors were encountered: