Opened 10 years ago
Last modified 8 years ago
#12768 closed enhancement
Better plotting for isogeny graphs of elliptic curves, and handling of LMFDB labels — at Version 3
Reported by: | roed | Owned by: | cremona |
---|---|---|---|
Priority: | minor | Milestone: | sage-5.3 |
Component: | elliptic curves | Keywords: | |
Cc: | cremona, kedlaya, vbraun | Merged in: | |
Authors: | David Roe | Reviewers: | |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description (last modified by )
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.
Change History (4)
comment:1 Changed 10 years ago by
- Cc cremona kedlaya added
comment:2 Changed 10 years ago by
- Status changed from new to needs_review
The tests in sage/schemes/elliptic_curves
pass for me. Let's see what patchbot says.
comment:3 Changed 10 years ago by
- Description modified (diff)
- Summary changed from Better plotting for isogeny graphs of elliptic curves to Better plotting for isogeny graphs of elliptic curves, and handling of LMFDB labels
Note: See
TracTickets for help on using
tickets.
I still need to finish doctesting this, but I wanted to post it now since I'm not going to be able to work on it tomorrow. It adds the capability to make curves from LMFDB labels, to reorder isogeny classes, and makes the plot for the graph of an isogeny class deterministic. I have a corresponding patch to the LMFDB code that fixes all the problems I've observed with ordering and plotting.