Ticket #11631 (new defect)

Opened 2 years ago

Last modified 20 months ago

L-series attached to cusp forms are broken

Reported by: was Owned by: was
Priority: major Milestone: sage-5.11
Component: number theory Keywords:
Cc: Work issues:
Report Upstream: N/A Reviewers:
Authors: Merged in:
Dependencies: Stopgaps:

Description (last modified by was) (diff)

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).

See also #12015.

Change History

comment:1 Changed 23 months ago by was

I have now written (or am writing) similar (but more powerful) functionality for psage. It'll end up in this directory of psage: psage/lseries/

 http://code.google.com/p/purplesage/source/browse/#hg%2Fpsage%2Flseries

comment:2 Changed 20 months ago by was

  • Description modified (diff)
Note: See TracTickets for help on using tickets.