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

bug in finite fields of degree 1 #522

Open
tthsqe12 opened this issue Nov 16, 2021 · 2 comments
Open

bug in finite fields of degree 1 #522

tthsqe12 opened this issue Nov 16, 2021 · 2 comments
Labels

Comments

@tthsqe12
Copy link
Contributor

tthsqe12 commented Nov 16, 2021

LibNormal.normal can still return the wrong base_ring. Two steps to reproduce:

  1. Change

    to
    if false && n == 1

  2. include this file multiple times in a row from the repl. It should eventually give an assertion error:

using Singular

   FiniteField(7, 1, "x", cached=false)

   F, (Fa,) = FunctionField(Fp(7), ["a"])
   K, a = AlgebraicExtensionField(F, Fa^2 + 1)
@show K
@show K.ptr.cpp_object
   R, (x, y, z) = PolynomialRing(K, ["x", "y", "z"])
   I = Ideal(R, a*z-x^4, z-y^6)
   l = Singular.LibNormal.normal(I)
   S = l[1][1][1]
@show K
@show K.ptr.cpp_object
@show base_ring(S)
   @assert base_ring(S) == K
@tthsqe12 tthsqe12 added the bug label Nov 16, 2021
@fingolfin
Copy link
Member

Is this issue still around? The linked code comments suggests this is due to a bug in Singular, so perhaps @hannes14 needs to take a look ?

@tthsqe12
Copy link
Contributor Author

tthsqe12 commented Nov 3, 2022

Well, with

  1. the above change to NumberTypes.jl
  2. libSingular_julia at oscar-system/libsingular-julia@85d0dec
  3. Singular at Singular/Singular@2ea97f9

things have improved. The above code just segfaults right away instead of making me include it multiple times!

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

No branches or pull requests

2 participants