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

Jacobi sums incorrect when exactly one character is trivial #6393

Closed
wdjoyner opened this issue Jun 24, 2009 · 9 comments
Closed

Jacobi sums incorrect when exactly one character is trivial #6393

wdjoyner opened this issue Jun 24, 2009 · 9 comments

Comments

@wdjoyner
Copy link

Example:

sage: G=DirichletGroup(5); X=G.list(); Y=X[0]; Z=X[1] 
sage: # Y is trivial and Z is quartic
sage: sum([Y(x)*Z(1-x) for x in IntegerModRing(5)])
 -1
sage: # The value -1 above is the correct value of the Jacobi sum J(Y, Z).
sage: Y.jacobi_sum(Z);    Z.jacobi_sum(Y)
0
0
sage: #The 0 values above are incorrect values of J(Y, Z).

Component: modular forms

Author: David Joyner

Reviewer: David Loeffler

Merged: sage-4.1.1.alpha0

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

@wdjoyner
Copy link
Author

Attachment: trac_6393-jacobi-sum.patch.gz

applies to 4.0.2.rc3 passes sage -testall

@wdjoyner wdjoyner changed the title Jacobi sums incorrect when exactly one chacater is trivial Jacobi sums incorrect when exactly one character is trivial Jun 24, 2009
@wdjoyner wdjoyner added this to the sage-4.1 milestone Jun 24, 2009
@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 14, 2009

Reviewer: David Loeffler

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 14, 2009

Author: David Joyner

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 14, 2009

comment:2

Looks good to me, and doctests pass. I'm surprised this went so long without being spotted.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 14, 2009

comment:3

BTW, in the course of reviewing this I've realised that our Jacobi sum algorithm is absurdly slow, vastly slower than calculating the sums directly! I've just opened #6534 for this.

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 14, 2009

comment:4

I'm about to upload a patch at #6534 which totally changes the way we calculate Jacobi sums, thus superseding this patch somewhat. I've made sure that it takes into account the issue you raised here.

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 16, 2009

comment:5

Just to let people know, this has been merged in sage-4.1.1-alpha0. I can't close this ticket because I don't have the privilege to do so. Sorry, folks :-(

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 16, 2009

Merged: sage-4.1.1.alpha0

@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed Jul 16, 2009
@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 24, 2009

comment:7

See #6613 for a follow up to this ticket.

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