Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generic congruence subgroups #11601

Closed
loefflerd mannequin opened this issue Jul 15, 2011 · 18 comments
Closed

Generic congruence subgroups #11601

loefflerd mannequin opened this issue Jul 15, 2011 · 18 comments

Comments

@loefflerd
Copy link
Mannequin

loefflerd mannequin commented Jul 15, 2011

This ticket adds functionality to work with arbitrary congruence subgroups of the modular group (specified by a level N and a subgroup of the finite group SL(2, Z / N Z)). These support all the basic functionality one would expect: computation of index, genus, elliptic points, cusps, etc. There is also a facility to compute the congruence closure of a (possibly noncongruence) subgroup.

Some "rationalisation" is also included: e.g. one can no longer create Gamma1(1) or GammaH(11, [2]), which previously existed as less-functional duplicates of SL2Z and Gamma0(11). (They previously played a role in "remembering" where certain degeneracy maps would go, but they didn't do this very well -- which was the cause of the issue at #10453 -- and the new more robust approach introduced at #10453 means we can get rid of them at last.)

Part of a series of tickets: #10335 - #11422 - #11598 - #10453 - #5048 - this one - #10546.

Apply attachment: 11601.4.patch on top of #5048. Once this is merged, #12001 can be closed as fixed.

Depends on #10335
Depends on #11422
Depends on #11598
Depends on #10453
Depends on #5048

Component: modular forms

Keywords: modular subgroup congruence sd35

Author: David Loeffler

Reviewer: Johan Bosman

Merged: sage-5.0.beta1

Issue created by migration from https://trac.sagemath.org/ticket/11601

@loefflerd loefflerd mannequin added this to the sage-4.8 milestone Jul 15, 2011
@loefflerd loefflerd mannequin assigned craigcitro Jul 15, 2011
@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jul 15, 2011

Attachment: trac_11601-congroups.patch.gz

Patch against 4.7.1.alpha4 + dependencies as in ticket description

@loefflerd loefflerd mannequin added the s: needs review label Jul 15, 2011
@loefflerd

This comment has been minimized.

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Sep 29, 2011

New version rebased for 4.7.2.alpha3

@loefflerd

This comment has been minimized.

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Sep 29, 2011

comment:3

Attachment: trac_11601-congroups.2.patch.gz

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Sep 29, 2011

Changed dependencies from #10335, #11422, #11598, #5048, #10453 to #10335, #11422, #11598, #10453, #5048

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

Work Issues: rebase

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

comment:4

I tried to apply trac_11601-congroups.2.patch to 4.8.alpha4 + #5048, but it failed.

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

Changed work issues from rebase to docbuild

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

comment:5

Attachment: 11601.3.patch.gz

I'm getting a warning when building the documentation, in is_CongruenceSubgroup in congroup_generic.py:

    Note that this may be False even if `x` really is a congruence subgroup --
    it tests whether `x` is "obviously" congruence, i.e.~whether it has a
    congruence subgroup datatype. To test whether or not an arithmetic subgroup
    of `SL(2, \ZZ)` is congruence, use the ``is_congruence()` method instead.

I guess a quote is missing at the end of is_congruence().

@sagetrac-johanbosman

This comment has been minimized.

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

comment:6

Attachment: 11601.4.patch.gz

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

Changed work issues from docbuild to none

@sagetrac-johanbosman
Copy link
Mannequin

sagetrac-johanbosman mannequin commented Dec 17, 2011

Reviewer: Johan Bosman

@jdemeyer jdemeyer modified the milestones: sage-4.8, sage-5.0 Dec 22, 2011
@loefflerd

This comment has been minimized.

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Jan 5, 2012

Changed keywords from modular subgroup congruence to modular subgroup congruence sd35

@jdemeyer
Copy link

Merged: sage-5.0.beta1

@loefflerd
Copy link
Mannequin Author

loefflerd mannequin commented Mar 10, 2012

comment:13

Apply 11601.4.patch

(for the patchbot, so it knows the prerequisites for 11709)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants