#11587: update Cremona's tables for Sage
--------------------------------------------------+-------------------------
Reporter: was | Owner: cremona
Type: enhancement | Status: new
Priority: major | Milestone: sage-4.7.2
Component: elliptic curves | Resolution:
Keywords: elliptic curves ellcurve database | Work_issues:
Upstream: N/A | Reviewer: John
Cremona, Tom Boothby
Author: R. Andrew Ohana | Merged:
Dependencies: #11642 |
--------------------------------------------------+-------------------------
Comment(by cremona):
Replying to [comment:41 jdemeyer]:
> This is the cause of a serious speed regression. Message by Dave
Loeffler:
>
> I've found the culprit: this is a side-effect of #11587. If you
> look at [attachment:trac_11587.patch], then it removes a couple of lines
of code
> at lines 710-712 of sage/databases/cremona.py which sets the
> generators to [] if the gens aren't explicitly in the database but the
> rank is known to be 0. The new version doesn't perform this little
> check and so we waste loads of time doing pointless 2-descents.
It has always been rather a nuisance that the small default database
differs from the large one not only in having fewer curves but in having
less information about them (specifically, omits the known rank and
generators). I don't want to delay this ticket longer than necessary (and
apologise for the positive review which missed this glitch) but propose a
follow-up which includes all known data for the small database too.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11587#comment:44>
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.
