Opened 7 years ago
Closed 6 years ago
#13156 closed defect (fixed)
The is_old method from sage.modular.modform.element.ModularFormElement doesn't work correctly
Reported by: | Snark | Owned by: | craigcitro |
---|---|---|---|
Priority: | major | Milestone: | sage-5.11 |
Component: | modular forms | Keywords: | sd51 |
Cc: | Merged in: | sage-5.11.rc0 | |
Authors: | David Loeffler | Reviewers: | Alex Ghitza |
Report Upstream: | N/A | Work issues: | |
Branch: | Commit: | ||
Dependencies: | Stopgaps: |
Description
sage: E=ModularForms(3,12).cuspidal_subspace() sage: f=E.gens()[0] sage: g=f-f sage: g.is_old() False
This is definitely a case where sage returns a wrong answer.
Attachments (1)
Change History (6)
comment:1 Changed 6 years ago by
Version 0, edited 6 years ago
by
(next)
comment:2 Changed 6 years ago by
- Keywords sd51 added
Test case for the underlying bug:
sage: R.<q> = QQ[[]] sage: ModularForms(2, 8)(q^20 + q^120) --------------------------------------------------------------------------- TypeError Traceback (most recent call last) <ipython-input-101-f1640b98a1fd> in <module>() ----> 1 ModularForms(Integer(2), Integer(8))(q**Integer(20) + q**Integer(120)) /storage/masiao/sage-5.11.beta3/local/lib/python2.7/site-packages/sage/modular/modform/space.pyc in __call__(self, x, check) 1147 x_potential = self.free_module().linear_combination_of_basis(x_potential) 1148 x_potential = element.ModularFormElement(self, x_potential) -> 1149 for i in range(int(W.degree()), x.prec()): 1150 if x_potential[i] != x[i]: 1151 raise ValueError, "q-expansion does not correspond to a form in self" TypeError: range() integer end argument expected, got PlusInfinity.
comment:3 Changed 6 years ago by
- Status changed from new to needs_review
comment:4 Changed 6 years ago by
- Milestone set to sage-5.11
- Reviewers set to Alex Ghitza
- Status changed from needs_review to positive_review
comment:5 Changed 6 years ago by
- Merged in set to sage-5.11.rc0
- Resolution set to fixed
- Status changed from positive_review to closed
Note: See
TracTickets for help on using
tickets.
I traced this back to a problem with the
__contains__
method for modular forms subspaces. The problem is that it expects imprecise q-expansions, and fails badly when given the zero q-expansion as argument.