#11631: L-series attached to cusp forms are broken
-----------------------------+----------------------------------------------
Reporter: was | Owner: was
Type: defect | Status: new
Priority: major | Milestone: sage-4.7.2
Component: number theory | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
-----------------------------+----------------------------------------------
{{{
sage: C = CuspForms(43,2)
sage: N = C.newforms('a')
sage: f = N[1]
sage: L = f.cuspform_lseries()
Boom!
}}}
There are two issues:
(1) the code doesn't work at all when the degree of the form is > 1,
which is the main interesting case,
(2) the name "cuspform_lseries" is bad, since f is already a cuspform,
and we use the name "lseries" in all other places (e.g., elliptic curves,
abelian varieties), so it is hard to find. In fact, I didn't even think
to look for cuspform_lseries, instead only finding this via lots of
grepping and reading source code.
So to fix this issue, I think (1) the bug should get fixed, and (2) the
name should be changed (actually *deprecate* the old name as explained in
the developers guide and introduce the name lseries).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11631>
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.