#6606: [with patch; needs work] Add a more efficient implementation of index for
Gamma(N).
---------------------------+------------------------------------------------
Reporter: simon | Owner: craigcitro
Type: enhancement | Status: new
Priority: minor | Milestone: sage-4.1.1
Component: modular forms | Keywords:
Reviewer: John Cremona | Author: simon
Merged: |
---------------------------+------------------------------------------------
Changes (by cremona):
* reviewer: => John Cremona
Comment:
The code loooked (though I would have written p**(3*e-2)*(p*p-1) ) but
after applying to 4.1:
{{{
sage: Gamma(19).index()
---------------------------------------------------------------------------
NameError Traceback (most recent call
last)
/home/john/.sage/temp/ubuntu/25083/_home_john__sage_init_sage_0.py in
<module>()
/home/john/sage-4.1/local/lib/python2.6/site-
packages/sage/modular/arithgroup/congroup_gamma.pyc in index(self)
105 32893086819240
106 """
--> 107 return prod([p**(3*e) - p**(3*e-2) for (p,e) in
self.level().factor()])
108
}}}
Looks like someone forgot to run sage -t before submitting the patch...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6606#comment:1>
Sage <http://sagemath.org/>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---
