#15808: Remove genus2reduction
-------------------------------------+-------------------------------------
Reporter: jdemeyer | Owner:
Type: enhancement | Status: positive_review
Priority: major | Milestone: sage-6.4
Component: packages: | Resolution:
standard | Merged in:
Keywords: | Reviewers: Peter Bruin
Authors: Jeroen Demeyer | Work issues:
Report Upstream: N/A | Commit:
Branch: | f35c94473e316c01c968f2a43027868c517815a2
u/jdemeyer/ticket/15808 | Stopgaps:
Dependencies: #15767 |
-------------------------------------+-------------------------------------
Changes (by pbruin):
* status: needs_review => positive_review
* reviewer: => Peter Bruin
Comment:
Replying to [comment:19 jdemeyer]:
> Replying to [comment:16 pbruin]:
> > However, your formula seems to mean that the 2-adic valuation of the
minimal discriminant is bounded on the set of all genus 2 curves, which I
can hardly believe, since it isn't even true for elliptic curves.
> After thinking about this some more, perhaps my code is correct but it
depends on the interpretation of "Minimal Discriminant (away from 2)". If
you interpret it as "smallest positive integer which occurs as
discriminant of a model of the given curve over Z[1/2]", then my code
might be correct.
Possibly, but I would be surprised if this definition coincides with the
one used by the original `genus2reduction` program. Anyway, I don't think
it makes sense to spend too much time on finding a nice definition for
"minimal discriminant away from 2", so any power of 2 in the "minimal
discriminant away from 2" is fine with me.
--
Ticket URL: <http://trac.sagemath.org/ticket/15808#comment:22>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
