#5995: Membership testing for modular forms subspaces is hopeless
---------------------------+------------------------------------------------
Reporter: davidloeffler | Owner: craigcitro
Type: defect | Status: new
Priority: major | Milestone: sage-4.0
Component: modular forms | Keywords:
---------------------------+------------------------------------------------
This is pretty poor, IMHO:
{{{
sage: M = ModularForms(17, 4)
sage: S = M.cuspidal_submodule()
sage: M.0 == S.0
True
sage: M.0 in S
False
}}}
As far as I can tell at a glance this is happening because
{{{S.__call__(x)}}} tests whether or not the parent of x has a canonical
inclusion map to S; it should probably be testing whether the parent of x
has a canonical inclusion map to the *ambient space* of S.
Once the above is fixed we should also have a method {{{is_cuspidal()}}}
for modular forms objects, which would be secretly just {{{self in
self.parent().cuspidal_submodule()}}}. A corresponding
{{{is_eisenstein()}}} would be good, too.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5995>
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
-~----------~----~----~----~------~----~------~--~---