#11930: disallow non-smooth hyperelliptic curves, and let hyperelliptic curves
know
they are not singular
-------------------------------------------+--------------------------------
Reporter: dkrenn | Owner: cremona
Type: defect | Status: needs_work
Priority: major | Milestone: sage-5.0
Component: elliptic curves | Keywords: hyperelliptic
curve singular smooth sd35
Work_issues: | Upstream: N/A
Reviewer: Marco Streng, Damiano Testa | Author: Daniel Krenn,
Marco Streng, Damiano Testa
Merged: | Dependencies:
-------------------------------------------+--------------------------------
Comment(by mstreng):
Replying to [comment:19 davidloeffler]:
> I'm not happy with this:
...
> I don't think we can merge this if we know or suspect that it gives
incorrect results! We should either raise an error in these cases, or
check that the results that are returned are correct.
Hi David,
This "NOTE" is just a warning that we added about the code as it was. It
is not about a change that we are making. We decided to leave support for
conics and elliptic curves intact, as people may be running a loop over
general double covers, or not care too much.
Removing support for conics and elliptic curves would be something for a
discussion on sage-nt, and then a new ticket. I can remove this note if
you'd like, and leave the rest of the patch as it is. Would that be an
improvement?
Marco
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11930#comment:20>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
