#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 |
--------------------------------------------------+-------------------------
Changes (by jdemeyer):
* cc: davidloeffler (added)
* status: closed => new
* resolution: fixed =>
* merged: sage-4.7.2.alpha3 =>
Comment:
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.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11587#comment:41>
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.
