Opened 12 years ago

Closed 12 years ago

#7548 closed defect (fixed)

modular form times scalar = boom

Reported by: AlexGhitza Owned by: craigcitro
Priority: major Milestone: sage-4.3.1
Component: modular forms Keywords:
Cc: Merged in: sage-4.3.1.alpha0
Authors: Alex Ghitza Reviewers: William Stein
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


| 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/ 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)
    971         # first ensure the levels are equal

NameError: global name 'HeckeModuleElement' is not defined

Attachments (1)

trac_7548.patch (1.3 KB) - added by AlexGhitza 12 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 12 years ago by AlexGhitza

  • Authors set to Alex Ghitza
  • Status changed from new to needs_review

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

Changed 12 years ago by AlexGhitza

comment:2 Changed 12 years ago by was

  • Status changed from needs_review to positive_review

Looks good.

comment:3 Changed 12 years ago by mhansen

  • Merged in set to sage-4.3.1.alpha0
  • Resolution set to fixed
  • Reviewers set to William Stein
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.