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

roots() method of polynomials over QQ misses roots in p-adic extension fields #31040

Closed
pjbruin opened this issue Dec 11, 2020 · 7 comments
Closed

Comments

@pjbruin
Copy link
Contributor

pjbruin commented Dec 11, 2020

In SageMath 9.3.beta3:

sage: R.<x> = QQ[]
sage: K.<a> = Qq(3).extension(x^2 + 1)
sage: (x^2 + 1).base_extend(K).roots()
[(a + O(3^20), 1),
 (2*a + 2*a*3 + 2*a*3^2 + 2*a*3^3 + 2*a*3^4 + 2*a*3^5 + 2*a*3^6 + 2*a*3^7 + 2*a*3^8 + 2*a*3^9 + 2*a*3^10 + 2*a*3^11 + 2*a*3^12 + 2*a*3^13 + 2*a*3^14 + 2*a*3^15 + 2*a*3^16 + 2*a*3^17 + 2*a*3^18 + 2*a*3^19 + O(3^20),
  1)]
sage: (x^2 + 1).roots(K)  # should return the same as above
[]

This can be fixed by only using the factor_padic() method when we are looking for roots in Zp or Qp.

Component: algebra

Author: Peter Bruin

Branch/Commit: 3dfbe69

Reviewer: Marc Mezzarobba

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

@pjbruin pjbruin added this to the sage-9.3 milestone Dec 11, 2020
@pjbruin
Copy link
Contributor Author

pjbruin commented Dec 14, 2020

@pjbruin
Copy link
Contributor Author

pjbruin commented Dec 14, 2020

Author: Peter Bruin

@pjbruin
Copy link
Contributor Author

pjbruin commented Dec 14, 2020

Commit: 3dfbe69

@pjbruin

This comment has been minimized.

@mezzarobba
Copy link
Member

Reviewer: Marc Mezzarobba

@mezzarobba
Copy link
Member

comment:2

lgtm!

@vbraun
Copy link
Member

vbraun commented Jan 17, 2021

Changed branch from u/pbruin/31040-roots_padic_extension_fields to 3dfbe69

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