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

StackOverflowError when computing roots over Puiseux series field #4034

Open
YueRen opened this issue Aug 21, 2024 · 2 comments
Open

StackOverflowError when computing roots over Puiseux series field #4034

YueRen opened this issue Aug 21, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@YueRen
Copy link
Member

YueRen commented Aug 21, 2024

This is related to #4007, and found together with @VictoriaSchleis @APMS04

julia> K,t = puiseux_series_field(QQ,10,"t")
(Puiseux series field in t over QQ, t + O(t^11))

julia> R,x = K["x"]
(Univariate polynomial ring in x over K, x)

julia> roots(x-1)
ERROR: StackOverflowError:
Stacktrace:
     [1] factor(a::AbstractAlgebra.Generic.Poly{AbstractAlgebra.Generic.PuiseuxSeriesFieldElem{QQFieldElem}})
       @ Hecke ~/.julia/packages/Hecke/x2Cxg/src/Misc/Integer.jl:179
     [2] factor(x::AbstractAlgebra.Generic.Poly{AbstractAlgebra.Generic.PuiseuxSeriesFieldElem{QQFieldElem}})
       @ Hecke ~/.julia/packages/Hecke/x2Cxg/src/Hecke.jl:202
     [3] factor(a::AbstractAlgebra.Generic.Poly{AbstractAlgebra.Generic.PuiseuxSeriesFieldElem{QQFieldElem}}; b::@Kwargs{})
       @ Hecke ~/.julia/packages/Hecke/x2Cxg/src/Misc/Integer.jl:179
--- the last 2 lines are repeated 26659 more times ---
 [53322] factor(x::AbstractAlgebra.Generic.Poly{AbstractAlgebra.Generic.PuiseuxSeriesFieldElem{QQFieldElem}})
       @ Hecke ~/.julia/packages/Hecke/x2Cxg/src/Hecke.jl:202

@thofma Is computing roots over puiseux_series_field currently supported? Roots over PadicFields seem to work.

@YueRen YueRen added the bug Something isn't working label Aug 21, 2024
@joschmitt joschmitt changed the title StackOverflowError when computing roots of a polynomial over ``(QQ) StackOverflowError when computing roots over Puiseux series field Aug 21, 2024
@thofma
Copy link
Collaborator

thofma commented Aug 21, 2024

No, not possible. Even the roots over PadicFields have some issues.

I will change the error messages to "Not implemented yet" or something like that.

@lgoettgens
Copy link
Member

I think this stack overflow is here instead of a MethodError: no method matching factor(::AbstractAlgebra.Generic.PuiseuxSeriesFieldElem{QQFieldElem}) due to thofma/Hecke.jl#1394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants