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

modular form times scalar = boom #7548

Closed
aghitza opened this issue Nov 28, 2009 · 6 comments
Closed

modular form times scalar = boom #7548

aghitza opened this issue Nov 28, 2009 · 6 comments

Comments

@aghitza
Copy link

aghitza commented Nov 28, 2009

----------------------------------------------------------------------
| Sage Version 4.2.1, Release Date: 2009-11-14                       |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: M = CuspForms(Gamma0(5*3^2), 2)
sage: f = M.basis()[0]
sage: f
q - q^4 + O(q^6)
sage: 2*f
2*q - 2*q^4 + O(q^6)
sage: f*2
---------------------------------------------------------------------------
NameError                                 Traceback (most recent call last)

/home/ghitza/.sage/temp/artin/19582/_home_ghitza__sage_init_sage_0.py in <module>()

/home/ghitza/sage-devel/local/lib/python2.6/site-packages/sage/modular/modform/element.pyc in __mul__(self, other)
    967         # boring case: scalar multiplication

    968         if not isinstance(other, ModularFormElement):
--> 969             return HeckeModuleElement.__mul__(self, other)
    970 
    971         # first ensure the levels are equal


NameError: global name 'HeckeModuleElement' is not defined

Component: modular forms

Author: Alex Ghitza

Reviewer: William Stein

Merged: sage-4.3.1.alpha0

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

@aghitza
Copy link
Author

aghitza commented Jan 3, 2010

comment:1

Well, the fix was embarrassingly trivial. Patch is attached.

@aghitza
Copy link
Author

aghitza commented Jan 3, 2010

Author: Alex Ghitza

@aghitza
Copy link
Author

aghitza commented Jan 3, 2010

Attachment: trac_7548.patch.gz

@williamstein
Copy link
Contributor

comment:2

Looks good.

@mwhansen
Copy link
Contributor

mwhansen commented Jan 3, 2010

Reviewer: William Stein

@mwhansen
Copy link
Contributor

mwhansen commented Jan 3, 2010

Merged: sage-4.3.1.alpha0

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