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

dimensions of modular forms spaces for Gamma(N) is slow #6860

Closed
williamstein opened this issue Sep 2, 2009 · 9 comments
Closed

dimensions of modular forms spaces for Gamma(N) is slow #6860

williamstein opened this issue Sep 2, 2009 · 9 comments

Comments

@williamstein
Copy link
Contributor

sage: dimension_cusp_forms(Gamma(11))
[hangs forever]

This has now been fixed as a consequence of #10546. Release manager: please close as fixed.

Component: modular forms

Keywords: sd51

Reviewer: David Loeffler, Alex Ghitza

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

@williamstein williamstein added this to the sage-5.11 milestone Sep 2, 2009
@williamstein williamstein self-assigned this Sep 2, 2009
@williamstein
Copy link
Contributor Author

comment:1

Wait, I'm just impatient, since:

sage: dimension_cusp_forms(Gamma(11))
26

So I'm changing this to an enhancement, since I think this should be fast.

@williamstein williamstein changed the title dimensions of modular forms spaces for Gamma(N) is busted / not implemented dimensions of modular forms spaces for Gamma(N) is slow Sep 2, 2009
@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 22, 2013

comment:2

This has now been fastified:

sage: %time dimension_cusp_forms(Gamma(10^28+731))                             
CPU times: user 0.04 s, sys: 0.02 s, total: 0.06 s
Wall time: 1.54 s
36274370885528165103530592700784546931709800073501194539717685491542679896172544001

(and most of that is the time taken to factor the level). I propose closing this as fixed.

@loefflerd loefflerd mannequin removed this from the sage-5.11 milestone Jul 22, 2013
@aghitza
Copy link
Contributor

aghitza commented Jul 22, 2013

Author: David Loeffler

@aghitza
Copy link
Contributor

aghitza commented Jul 22, 2013

comment:3

Agreed.

@aghitza
Copy link
Contributor

aghitza commented Jul 22, 2013

Reviewer: Alex Ghitza

@aghitza
Copy link
Contributor

aghitza commented Jul 22, 2013

Changed keywords from none to sd51

@loefflerd

This comment has been minimized.

@jdemeyer
Copy link
Contributor

jdemeyer commented Aug 9, 2013

Changed reviewer from Alex Ghitza to David Loeffler, Alex Ghitza

@jdemeyer
Copy link
Contributor

jdemeyer commented Aug 9, 2013

Changed author from David Loeffler to none

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

3 participants