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

multivariate factorization over non-prime finite fields hangs #11838

Closed
zimmermann6 opened this issue Sep 23, 2011 · 9 comments
Closed

multivariate factorization over non-prime finite fields hangs #11838

zimmermann6 opened this issue Sep 23, 2011 · 9 comments

Comments

@zimmermann6
Copy link
Contributor

In Sage 4.7.1:

sage: K = GF(4,'a')
sage: a = K.gens()[0]
sage: R.<x,y> = K[]
sage: p=x^8*y^3 + x^2*y^9 + a*x^9 + a*x*y^4
sage: q=y^11 + (a)*y^10 + (a + 1)*x*y^3
sage: p.factor(proof=False)
x * (x^7*y^3 + x*y^9 + (a)*x^8 + (a)*y^4)
sage: q.factor(proof=False)
y^3 * (y^8 + (a)*y^7 + (a + 1)*x)
sage: f = p*q
sage: f.factor(proof=False)

The last command seems to hang forever.

CC: @malb @simon-king-jena

Component: factorization

Keywords: sd35

Author: Paul Zimmermann

Reviewer: Bogdan Banu

Merged: sage-4.8.alpha6

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

@zimmermann6
Copy link
Contributor Author

Changed keywords from none to sd34

@a-andre
Copy link
Contributor

a-andre commented Nov 4, 2011

comment:2

I can confirm this in sage 4.7.1. But it don't happen in sage 4.7.2 anymore.
I immediately get

x * y^3 * (y^8 + (a)*y^7 + (a + 1)*x) * (x^7*y^3 + x*y^9 + (a)*x^8 + (a)*y^4)

@zimmermann6
Copy link
Contributor Author

comment:3

I confirm it works with sage 4.7.2. I will add a non-regression test during the SageFlint days.

Paul

@zimmermann6
Copy link
Contributor Author

Author: Paul Zimmermann

@zimmermann6
Copy link
Contributor Author

comment:4

Attachment: trac_11838.patch.gz

the attached patch adds a non-regression test. Please review it.
Paul

@zimmermann6
Copy link
Contributor Author

Changed keywords from sd34 to sd35

@sagetrac-bbanu
Copy link
Mannequin

sagetrac-bbanu mannequin commented Dec 18, 2011

comment:5

Looks good. Bogdan

@sagetrac-bbanu
Copy link
Mannequin

sagetrac-bbanu mannequin commented Dec 18, 2011

Reviewer: Bogdan Banu

@jdemeyer
Copy link
Contributor

Merged: sage-4.8.alpha6

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

3 participants