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

Genus outputs -1 sometimes #28336

Open
sagetrac-bthompson mannequin opened this issue Aug 9, 2019 · 6 comments
Open

Genus outputs -1 sometimes #28336

sagetrac-bthompson mannequin opened this issue Aug 9, 2019 · 6 comments

Comments

@sagetrac-bthompson
Copy link
Mannequin

sagetrac-bthompson mannequin commented Aug 9, 2019

Sage outputs -1 as the genus of a curve, it should probably throw an error instead. The genus function doesn't seem to check if the function is irreducible, and it just attempts to calculate the genus anyway.

sage: FF.<c>  =QQ[] 
sage: R.<x> = FF[]
sage: K = QQ[c, x]
sage: poly2 = K(x^2 - c^4)
sage: C2 = Curve(poly2)
sage: C2.genus()
-1

CC: @bhutz @slel

Component: elliptic curves

Keywords: sagedays103, dynamics, curves

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

@sagetrac-bthompson sagetrac-bthompson mannequin added this to the sage-8.9 milestone Aug 9, 2019
@sagetrac-bthompson

This comment has been minimized.

@sagetrac-bthompson
Copy link
Mannequin Author

sagetrac-bthompson mannequin commented Aug 9, 2019

Changed keywords from sagedays103, dynamics to sagedays103, dynamics, curves

@embray
Copy link
Contributor

embray commented Dec 30, 2019

comment:3

Ticket retargeted after milestone closed

@embray embray modified the milestones: sage-8.9, sage-9.1 Dec 30, 2019
@mkoeppe
Copy link
Member

mkoeppe commented May 1, 2020

comment:4

Moving tickets to milestone sage-9.2 based on a review of last modification date, branch status, and severity.

@mkoeppe mkoeppe modified the milestones: sage-9.1, sage-9.2 May 1, 2020
@mkoeppe mkoeppe modified the milestones: sage-9.2, sage-9.3 Oct 24, 2020
@slel

This comment has been minimized.

@slel slel changed the title Genus outputs -1 sometimes. Genus outputs -1 sometimes Mar 27, 2021
@mkoeppe
Copy link
Member

mkoeppe commented May 10, 2021

comment:7

Moving to 9.4, as 9.3 has been released.

@mkoeppe mkoeppe modified the milestones: sage-9.3, sage-9.4 May 10, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.4, sage-9.5 Aug 22, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.5, sage-9.6 Dec 18, 2021
@mkoeppe mkoeppe modified the milestones: sage-9.6, sage-9.7 May 3, 2022
@mkoeppe mkoeppe modified the milestones: sage-9.7, sage-9.8 Sep 19, 2022
@mkoeppe mkoeppe removed this from the sage-9.8 milestone Jan 29, 2023
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