#12768: Better plotting for isogeny graphs of elliptic curves, and handling of
LMFDB labels
-----------------------------------+----------------------------------------
Reporter: roed | Owner: cremona
Type: enhancement | Status: needs_review
Priority: minor | Milestone: sage-5.0
Component: elliptic curves | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers: John Cremona
Authors: David Roe | Merged in:
Dependencies: #12769 | Stopgaps:
-----------------------------------+----------------------------------------
Changes (by cremona):
* reviewer: => John Cremona
Old description:
> There are only 17 different possible isogeny graphs for elliptic curves
> over Q. It would be nice if the isogeny graph was laid out in the same
> way each time, and if the labels corresponded to the Cremona labels of
> the curves in the isogeny class.
>
> The second major topic handled by this ticket is to implement handling of
> LMFDB labels for elliptic curves as well as Cremona labels, and the
> conversions between these.
New description:
There are only 17 different possible isogeny graphs for elliptic curves
over Q. It would be nice if the isogeny graph was laid out in the same
way each time, and if the labels corresponded to the Cremona labels of the
curves in the isogeny class.
The second major topic handled by this ticket is to implement handling of
LMFDB labels for elliptic curves as well as Cremona labels, and the
conversions between these. the third is a new class for isogeny classes
of elliptic curves over Q.
Apply: 12768.patch, 12768-review.patch
--
Comment:
Review: This patch implements three different things: (1) A new class
for isogeny classes of elliptic curves (currently only for curves over Q);
(2) implementation of the new (March 2012) LMFDB labels for elliptic
curves over Q, with utilities for comverting to and from Cremona labels;
(3) better layouts for isogeny graphs of elliptic curves over Q.
I have been through the code in some detail and only found some minor
things to fix, as in the reviewer's patch. One of these was a typo
causing doctest failure. Nothing more than minor. If David R is happy
with these, I'll give the ticket a positive review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12768#comment:8>
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.
