[with patch, positive review] Add a more efficient implementation of index for Gamma(N).
Gamma(N).index used the default implementation which was slow. Attached is a new implementation which works for the specific subgroup.
Change History (7)
Reviewers: 
→ John Cremona

Summary: 
[with patch; needs review] Add a more efficient implementation of index for Gamma(N). →
[with patch; needs work] Add a more efficient implementation of index for Gamma(N).

Summary: 
[with patch; needs work] Add a more efficient implementation of index for Gamma(N). →
[with patch; needs review] Add a more efficient implementation of index for Gamma(N).

Summary: 
[with patch; needs review] Add a more efficient implementation of index for Gamma(N). →
[with patch; positive review] Add a more efficient implementation of index for Gamma(N).

Authors: 
simon →
Simon Morris

Merged in: 
→ Sage 4.1.1.alpha1

Resolution: 
→ fixed

Reviewers: 
John Cremona →
John Cremona, David Roed

Status: 
new →
closed

Summary: 
[with patch; positive review] Add a more efficient implementation of index for Gamma(N). →
[with patch, positive review] Add a more efficient implementation of index for Gamma(N).

The code loooked (though I would have written p(3*e2)*(p*p1) ) but after applying to 4.1:
Looks like someone forgot to run sage t before submitting the patch...