Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Symmetric square L-function bug for odd chi #23247

Closed
loefflerd mannequin opened this issue Jun 15, 2017 · 14 comments
Closed

Symmetric square L-function bug for odd chi #23247

loefflerd mannequin opened this issue Jun 15, 2017 · 14 comments

Comments

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jun 15, 2017

This is a followup to #19668 which added functionality for computing symmetric square L-functions of modular forms (via the Sage interface to Dokchitser's ComputeL GP/PARI script). Unfortunately, the code has a bug when the twisting character psi has non-real values, as one can see by checking the functional equation.

sage: F = Newforms(1,12)[0]
sage: chi = DirichletGroup(7).0
sage: F.symsquare_lseries(chi^2).check_functional_equation() # even char
-9.06002242118120e-8 - 1.94297172129154e-7*I
sage: F.symsquare_lseries(chi).check_functional_equation() # odd char
8.86658506905467e7 + 1.10964024593305e8*I

Note the huge discrepancy for the odd character. This is because the Gamma and epsilon factor parameters are being computed wrongly. (Thanks to Jim Brown for pointing this out)

Component: modular forms

Keywords: L-functions

Author: David Loeffler

Branch: 390030b

Reviewer: Frédéric Chapoton, Jim Brown

Issue created by migration from https://trac.sagemath.org/ticket/23247

@loefflerd loefflerd mannequin added this to the sage-8.0 milestone Jun 15, 2017
@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 17, 2017

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 17, 2017

comment:2

Here's a patch.


New commits:

9bd5650Trac #23247: fixed gamma and epsilon factors for chi odd

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 17, 2017

Commit: 9bd5650

@loefflerd loefflerd mannequin added the s: needs review label Jun 17, 2017
@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 17, 2017

Author: David Loeffler

@fchapoton
Copy link
Contributor

Changed commit from 9bd5650 to 390030b

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton

@fchapoton
Copy link
Contributor

@fchapoton
Copy link
Contributor

comment:4

Looks good to me. I made a tiny cosmetic change. If you agree with it, you can set to positive on my behalf.


New commits:

390030btrac 23247 some details

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 17, 2017

comment:5

Great, thanks!

@sagetrac-jbrown
Copy link
Mannequin

sagetrac-jbrown mannequin commented Jun 21, 2017

comment:6

I have used the new code and it works correctly now. It is a positive review from me.

@vbraun
Copy link
Member

vbraun commented Jun 22, 2017

Changed branch from public/23247-lfunction to 390030b

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 22, 2017

comment:8

I've added Jim's name as reviewer.

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 22, 2017

Changed commit from 390030b to none

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jun 22, 2017

Changed reviewer from Frédéric Chapoton to Frédéric Chapoton, Jim Brown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants