Skip to content
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

Incomplete multivariate factorization #17251

Closed
jdemeyer opened this issue Oct 29, 2014 · 14 comments
Closed

Incomplete multivariate factorization #17251

jdemeyer opened this issue Oct 29, 2014 · 14 comments

Comments

@jdemeyer
Copy link
Contributor

With sage-6.4.beta6, the following polynomial factorization gives a reducible factor (-a^2 + b^2):

sage: R.<z,a,b> = PolynomialRing(QQ)
sage: N = -a^4*z^8 + 2*a^2*b^2*z^8 - b^4*z^8 - 16*a^3*b*z^7 + 16*a*b^3*z^7 + 28*a^4*z^6 - 56*a^2*b^2*z^6 + 28*b^4*z^6 + 112*a^3*b*z^5 - 112*a*b^3*z^5 - 70*a^4*z^4 + 140*a^2*b^2*z^4 - 70*b^4*z^4 - 112*a^3*b*z^3 + 112*a*b^3*z^3 + 28*a^4*z^2 - 56*a^2*b^2*z^2 + 28*b^4*z^2 + 16*a^3*b*z - 16*a*b^3*z - a^4 + 2*a^2*b^2 - b^4
sage: N.factor()
(-1) * (-a^2 + b^2) * (-z^4*a + z^4*b - 4*z^3*a - 4*z^3*b + 6*z^2*a - 6*z^2*b + 4*z*a + 4*z*b - a + b) * (z^4*a + z^4*b - 4*z^3*a + 4*z^3*b - 6*z^2*a - 6*z^2*b + 4*z*a - 4*z*b + a + b)

This is fixed by #17254.

Upstream: Completely fixed; Fix reported upstream

CC: @sagetrac-jakobkroeker

Component: factorization

Keywords: singular

Author: Frédéric Chapoton

Branch/Commit: 8ce76ba

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/17251

@jdemeyer jdemeyer added this to the sage-6.4 milestone Oct 29, 2014
@jdemeyer
Copy link
Contributor Author

Changed keywords from none to singular

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Contributor Author

Upstream: Completely fixed; Fix reported upstream

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Contributor Author

Dependencies: #17254

@sagetrac-jakobkroeker
Copy link
Mannequin

sagetrac-jakobkroeker mannequin commented Mar 4, 2017

comment:5

Next: add a test.

@fchapoton
Copy link
Contributor

New commits:

8ce76batrac 17251 adding a doctest for factorisation of polynomials

@fchapoton
Copy link
Contributor

Commit: 8ce76ba

@fchapoton
Copy link
Contributor

Branch: u/chapoton/17251

@fchapoton
Copy link
Contributor

Changed dependencies from #17254 to none

@fchapoton
Copy link
Contributor

Author: Frédéric Chapoton

@tscrim
Copy link
Collaborator

tscrim commented Mar 10, 2018

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Mar 10, 2018

comment:8

LGTM.

@vbraun
Copy link
Member

vbraun commented Mar 19, 2018

Changed branch from u/chapoton/17251 to 8ce76ba

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants