#5138: [with patch; with review, needs work] implement computing manin constants
of elliptic curves
---------------------------+------------------------------------------------
Reporter: was | Owner: was
Type: enhancement | Status: new
Priority: major | Milestone: sage-3.3
Component: number theory | Resolution:
Keywords: |
---------------------------+------------------------------------------------
Changes (by cremona):
* summary: [with patch; needs review] implement computing manin
constants of elliptic curves => [with patch;
with review, needs work] implement computing
manin constants of elliptic curves
Comment:
Patch applies cleanly to 3.3.alpha2 + #5139. Tests pass BUT:
{{{
sage: Ellsage: EllipticCurve('990a1').optimal_curve()
---------------------------------------------------------------------------
RuntimeError
}}}
since on line 3099 you set the number to 3 when N=990 for all isogeny
classes, not just class h.
Somewhere in the database code I think we have utilities for splitting the
label into its 3 components, by the way, which might be more transparent
than (e.g.) stripping off the last character to get the number.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5138#comment:8>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
