#11598: Congruence testing for odd modular subgroups
---------------------------------+------------------------------------------
Reporter: davidloeffler | Owner: craigcitro
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.2
Component: modular forms | Keywords: modular congruence subgroup
Work_issues: | Upstream: N/A
Reviewer: Vincent Delecroix | Author: David Loeffler
Merged: | Dependencies: #11422
---------------------------------+------------------------------------------
Changes (by davidloeffler):
* status: needs_work => needs_review
* work_issues: docstrings =>
Old description:
> The new functionality added in ticket #11422 makes it possible to
> manipulate arbitrary finite index (not necessarily congruence) subgroups
> of SL2Z. This massively extends my old code which only worked for
> subgroups containing -1.
>
> The "is_congruence" method in the new code, though, *defines* a subgroup
> to be congruence if its image modulo -1 is a congruence subgroup of
> PSL2Z. This is not the same as the conventional definition of a
> congruence subgroup of SL2Z. The patch below modifies the algorithm so it
> uses the conventional notion of "congruence". It also adds functionality
> for enumerating all the liftings of a projective modular subgroup.
>
> Part of a series of tickets: #10335 - #11422 - this one - #5048 - #10453
> - #11601.
New description:
The new functionality added in ticket #11422 makes it possible to
manipulate arbitrary finite index (not necessarily congruence) subgroups
of SL2Z. This massively extends my old code which only worked for
subgroups containing -1.
The "is_congruence" method in the new code, though, *defines* a subgroup
to be congruence if its image modulo -1 is a congruence subgroup of PSL2Z.
This is not the same as the conventional definition of a congruence
subgroup of SL2Z. The patch below modifies the algorithm so it uses the
conventional notion of "congruence". It also adds functionality for
enumerating all the liftings of a projective modular subgroup.
Part of a series of tickets: #10335 - #11422 - this one - #5048 - #10453 -
#11601.
'''Apply:'''
1. [attachment:trac_11598-all.patch]
1. [attachment:trac_11598-docfixes.patch]
--
Comment:
Here's a patch which corrects the docstrings as you suggest. If there are
any further documentation fixes, can I suggest we agree to keep them for a
future ticket?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11598#comment:12>
Sage <http://www.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.
