#10236: bug in modular symbols for elliptic curves
-------------------------------+--------------------------------------------
Reporter: wuthrich | Owner: cremona
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.6.2
Component: elliptic curves | Keywords: modular symbols
Author: Chris Wuthrich | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-------------------------------+--------------------------------------------
Changes (by newvalueoldvalue):
* status: new => needs_review
* author: => Chris Wuthrich
Comment:
This second patch, applied after the first one, will solve the both issues
in the ticket description and revert back some errors introduced earlier.
The problem was a "/2" in a formula. I have absolutely no idea why I put
that there. To check that the formula is now correct, I did the
following::
{{{
sage: rs = flatten([[a/m for a in srange(1,m) if gcd(a,m)==1] for m in
srange(2,20)])
sage: for E in cremona_curves([11..100]):
....: m = E.modular_symbol(use_eclib=False)
....: me = E.modular_symbol(use_eclib=True)
....: for r in rs:
....: if m(r) != me(r):
....: print E.label(), r, m(r), me(r)
}}}
which gladly did not yield any output.
So these patches are ready for review.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10236#comment:5>
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.
