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

HyperellipticCurve.change_ring(R) may reduce genus #25249

Open
edgarcosta opened this issue Apr 25, 2018 · 0 comments
Open

HyperellipticCurve.change_ring(R) may reduce genus #25249

edgarcosta opened this issue Apr 25, 2018 · 0 comments

Comments

@edgarcosta
Copy link
Member

At the moment, change_ring checks implicitly for smoothness, by creating a new instance of HyperellipticCurve, with the defining polynomials is the new ring, in short, it returns:

HyperellipticCurve(f.change_ring(R), h.change_ring(R), "%s,%s"%(x,y))

However, we never really check if we are obtaining a proper base change.
For example, if we take R = GF(p), we can reduce the degree of f and h and get a smooth model for a hyperelliptic curve of a smaller genus.
For example:

sage: C = HyperellipticCurve(-6*x^6 - 4*x^5 - 4*x^4 - 5*x^3 - 2*x^2 - x - 1, 1)
....: print C.genus()
....: print C.change_ring(GF(2)).genus()
....:
2
1

Should we add a check for the genus?

Component: algebraic geometry

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

@edgarcosta edgarcosta added this to the sage-8.2 milestone Apr 25, 2018
@mkoeppe mkoeppe removed this from the sage-8.2 milestone Dec 29, 2022
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

2 participants