#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):
* status: needs_work => needs_review
Comment:
I have updated the review patch: it used to be that when asking for
order='database' for a curve not in the database, this was caught but the
error message output itself raised a confusing error (on trying to output
%self). Instead the error message now uses %self.E. I added a doctest
which illustrates this.
I give David's original patch a positive review modulo my own reviewer's
patch, so if David (or anyone) thinks my patch is OK, please give the
whole ticket a positive review. I tested this on 5.0.beta14.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12768#comment:12>
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.
