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

Problem with Hecke eigenvalues with the same index as the level #15201

Closed
fredstro opened this issue Sep 16, 2013 · 8 comments
Closed

Problem with Hecke eigenvalues with the same index as the level #15201

fredstro opened this issue Sep 16, 2013 · 8 comments

Comments

@fredstro
Copy link

It seems to be a problem to compute Hecke eigenvalues for Modular Symbol subspaces at integers dividing the level N unless the eigenvalue for the Hecke operator at N is computed first.
I have verified this problem for sage versions between 5.8 and 5.12.beta.

Example

sage: M=ModularSymbols(5,6,sign=1)
sage: S=M.decomposition()
sage: f=S[0]
sage: f.eigenvalue(10)
IndexError                                Traceback (most recent call last)
...
IndexError: n must be a positive integer
sage: f.eigenvalue(5)
25
sage: f.eigenvalue(10)
50

Component: modular forms

Author: Alex Ghitza

Branch/Commit: 56341a3

Reviewer: Peter Bruin

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

@fredstro fredstro added this to the sage-6.1 milestone Sep 16, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@aghitza

This comment has been minimized.

@aghitza
Copy link

aghitza commented Apr 24, 2014

comment:3

The code used the recursive formula for the Hecke eigenvalues of p-power index without implementing the base case. This is fixed in the attached branch.

@aghitza
Copy link

aghitza commented Apr 24, 2014

Branch: u/AlexGhitza/ticket/15201

@aghitza
Copy link

aghitza commented Apr 24, 2014

Author: Alex Ghitza

@aghitza
Copy link

aghitza commented Apr 24, 2014

Commit: 56341a3

@aghitza
Copy link

aghitza commented Apr 24, 2014

New commits:

56341a3fix Hecke eigenvalues recurrence

@pjbruin
Copy link
Contributor

pjbruin commented Apr 24, 2014

Reviewer: Peter Bruin

@vbraun
Copy link
Member

vbraun commented Apr 29, 2014

Changed branch from u/AlexGhitza/ticket/15201 to 56341a3

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

4 participants