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

feat: representatives for the FixedDetMatrices under SL action #16160

Open
wants to merge 34 commits into
base: master
Choose a base branch
from

Conversation

CBirkbeck
Copy link
Collaborator

@CBirkbeck CBirkbeck commented Aug 26, 2024

Define the set of representatives under the action of SL(2, ZZ) and give some reduction lemmas that are useful for later proving that SL(2, ZZ) is generated by S, T.


Copy link

github-actions bot commented Aug 26, 2024

PR summary a8e364cb34

Import changes for modified files

Dependency changes

File Base Count Head Count Change
Mathlib.LinearAlgebra.Matrix.FixedDetMatrices 1255 1256 +1 (+0.08%)
Import changes for all files
Files Import difference
Mathlib.LinearAlgebra.Matrix.FixedDetMatrices 1

Declarations diff

+ S_smul_four
+ T_S_rel
+ induction_on
+ reduce
+ reduce_eqn1
+ reduce_eqn2
+ reduce_eqn3
+ reduce_mem_reps
+ reduce_rec
+ reduce_step
+ reps
+ reps_entries_le_m'
+ reps_fintype
+ reps_zero_empty

You can run this locally as follows
## summary with just the declaration names:
./scripts/declarations_diff.sh <optional_commit>

## more verbose report:
./scripts/declarations_diff.sh long <optional_commit>

The doc-module for script/declarations_diff.sh contains some details about this script.

@CBirkbeck CBirkbeck mentioned this pull request Aug 26, 2024
2 tasks
@CBirkbeck CBirkbeck marked this pull request as ready for review August 26, 2024 16:51
@CBirkbeck CBirkbeck added the blocked-by-other-PR This PR depends on another PR to Mathlib (this label is automatically managed by a bot) label Aug 26, 2024
@grunweg grunweg changed the title Representatives for the FixedDetMatrices under SL action. feat: representatives for the FixedDetMatrices under SL action Aug 27, 2024
@grunweg grunweg added the t-algebra Algebra (groups, rings, fields, etc) label Aug 27, 2024
@CBirkbeck CBirkbeck added the WIP Work in progress label Oct 20, 2024
@leanprover-community-bot-assistant leanprover-community-bot-assistant added the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Oct 28, 2024
@mathlib4-dependent-issues-bot mathlib4-dependent-issues-bot removed the blocked-by-other-PR This PR depends on another PR to Mathlib (this label is automatically managed by a bot) label Oct 28, 2024
@mathlib4-dependent-issues-bot
Copy link
Collaborator

This PR/issue depends on:

@leanprover-community-bot-assistant leanprover-community-bot-assistant removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. (this label is managed by a bot) label Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t-algebra Algebra (groups, rings, fields, etc) WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants