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

[Merged by Bors] - feat(number_theory/modular_forms/congruence_subgroups): Add definition of congruence subgroups. #15159

Closed
wants to merge 22 commits into from

Conversation

CBirkbeck
Copy link
Collaborator

@CBirkbeck CBirkbeck commented Jul 6, 2022

This contains the definition of a congruence subgroup of SL(2,Z) and defines Gamma1, Gamma0 and full level subgroups. It also contains some basic results about them.


Open in Gitpod

…n of congruence subgroups and basic examples.
@CBirkbeck CBirkbeck changed the title feat(number_theory/modular_forms/congruence_subgroups): Add definitio… feat(number_theory/modular_forms/congruence_subgroups): Add definition of congruence subgroups. Jul 6, 2022
@CBirkbeck CBirkbeck added the WIP Work in progress label Jul 6, 2022
@CBirkbeck CBirkbeck marked this pull request as ready for review July 15, 2022 08:26
@CBirkbeck CBirkbeck added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Jul 15, 2022
@CBirkbeck CBirkbeck added WIP Work in progress and removed awaiting-review The author would like community review of the PR labels Jul 15, 2022
@CBirkbeck CBirkbeck added awaiting-review The author would like community review of the PR and removed WIP Work in progress labels Jul 27, 2022
@riccardobrasca riccardobrasca self-assigned this Aug 17, 2022
src/data/matrix/basic.lean Outdated Show resolved Hide resolved
src/group_theory/subgroup/pointwise.lean Outdated Show resolved Hide resolved
@@ -209,6 +209,29 @@ subtype.ext $ (@ring_hom.map_matrix n _ _ _ _ _ _ (int.cast_ring_hom R)).map_neg

end has_neg

section special_cases

lemma SL2_inv_det_expl (A : SL(2,R)) : det ![![A.1 1 1, -A.1 0 1], ![-A.1 1 0 , A.1 0 0]] = 1 :=
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand the name...

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok so I think what I meant to call this is SL2_inv_expl_det for the det of the explicit inverse. But its not a great name, I'm happy to change it to something else. I guess maybe the SL isn't needed...

src/number_theory/modular_forms/congruence_subgroups.lean Outdated Show resolved Hide resolved
src/number_theory/modular_forms/congruence_subgroups.lean Outdated Show resolved Hide resolved
src/number_theory/modular_forms/congruence_subgroups.lean Outdated Show resolved Hide resolved
src/number_theory/modular_forms/congruence_subgroups.lean Outdated Show resolved Hide resolved
Copy link
Member

@riccardobrasca riccardobrasca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's ready to go, after taking into account the remaining (cosmetic) comments.

src/group_theory/subgroup/pointwise.lean Show resolved Hide resolved
@CBirkbeck
Copy link
Collaborator Author

I think it's ready to go, after taking into account the remaining (cosmetic) comments.

done!

@riccardobrasca
Copy link
Member

Thanks!

bors merge

@github-actions github-actions bot added ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) and removed awaiting-review The author would like community review of the PR labels Sep 8, 2022
bors bot pushed a commit that referenced this pull request Sep 8, 2022
…n of congruence subgroups. (#15159)

This contains the definition of a congruence subgroup of `SL(2,Z)` and defines Gamma1, Gamma0 and full level subgroups. It also contains some basic results about them.
@bors
Copy link

bors bot commented Sep 8, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(number_theory/modular_forms/congruence_subgroups): Add definition of congruence subgroups. [Merged by Bors] - feat(number_theory/modular_forms/congruence_subgroups): Add definition of congruence subgroups. Sep 8, 2022
@bors bors bot closed this Sep 8, 2022
@bors bors bot deleted the congruence_subgroups branch September 8, 2022 13:17
bottine pushed a commit that referenced this pull request Sep 13, 2022
…n of congruence subgroups. (#15159)

This contains the definition of a congruence subgroup of `SL(2,Z)` and defines Gamma1, Gamma0 and full level subgroups. It also contains some basic results about them.
b-mehta pushed a commit that referenced this pull request Sep 21, 2022
…n of congruence subgroups. (#15159)

This contains the definition of a congruence subgroup of `SL(2,Z)` and defines Gamma1, Gamma0 and full level subgroups. It also contains some basic results about them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants