#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 leif):
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.
I was fearing something like that.
Why not open a follow-up (blocker) ticket for fixing that?
alpha3 is already out...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11587#comment:42>
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.
