#10096: sha().an() assumes E is minimal.
-------------------------------+--------------------------------------------
Reporter: weigandt | Owner: cremona
Type: defect | Status: new
Priority: major | Milestone: sage-4.6
Component: elliptic curves | Keywords: elliptic curves, sha, real period
Author: | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Changes (by cremona):
* cc: wuthrich (added)
* upstream: Reported upstream. Developers acknowledge bug. => N/A
Comment:
I found the problem here. an() uses the minimal model when it can do the
job itself, which is when the rank is at most 1, but when the rank is at
least 2 it passes the work to an_numerical(), and that did not use the
minimal model.
I am preparing a patch which fixes this. And as I suspect that other
related functions also need the minimal model (notably the p-adic one) I
am now caching the minimal model and changing lots of occurrences of
self.E to self.Emin.
Watch this space...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10096#comment:1>
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.