#11631: L-series attached to cusp forms are broken
-----------------------------+----------------------------------------------
   Reporter:  was            |          Owner:  was     
       Type:  defect         |         Status:  new     
   Priority:  major          |      Milestone:  sage-4.8
  Component:  number theory  |       Keywords:          
Work_issues:                 |       Upstream:  N/A     
   Reviewer:                 |         Author:          
     Merged:                 |   Dependencies:          
-----------------------------+----------------------------------------------
Description changed by was:

Old description:

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

New description:

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

--

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11631#comment:2>
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.

Reply via email to