#4357: [with patch, with positive review after minor changes] modular forms --
new
subspace used to work and now broken
---------------------------+------------------------------------------------
Reporter: was | Owner: davidloeffler
Type: enhancement | Status: assigned
Priority: major | Milestone: sage-4.0
Component: modular forms | Keywords:
---------------------------+------------------------------------------------
Comment(by davidloeffler):
As for the t argument thing: for modular symbols, where the raising
matrices are complicated trace-like operators, the argument t is not used,
since the degeneracy raising matrix for all values of t is calculated from
the one for t = 1 by composing with a Hecke operator. But for modular
forms, the degeneracy raising matrices are pretty easy (since we're
working in a "cohomological" setup rather than a "homological" one) so it
is easier to calculate them directly for each t.
Hence if _compute_degeneracy_raising matrix is getting called on a modular
symbols space with t != 1, that's a runtime error, but on a modular forms
space that's not the case. That should be explained in docstrings
somewhere; I'll do a follow-up patch.
David
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/4357#comment:6>
Sage <http://sagemath.org/>
Sage - Open Source Mathematical Software: Building the Car Instead of
Reinventing the Wheel
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
