[Merged by Bors] - feat(GroupTheory/SpecificGroups/Alternating/Centralizer): compute the centralizer of a permutation in the alternating group#17047
Conversation
ocfnash
left a comment
There was a problem hiding this comment.
Thanks, these are nice results.
I have left some superficial golfing suggestions and two suggestions for relocating lemmas.
I will do a more thorough review once you've looked at these suggestions.
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Co-authored-by: Oliver Nash <7734364+ocfnash@users.noreply.github.com>
ocfnash
left a comment
There was a problem hiding this comment.
Another round of superficial remarks. Unfortunately I'm out of time till tomorrow but I should be able to finish then.
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Co-authored-by: Oliver Nash <7734364+ocfnash@users.noreply.github.com>
ocfnash
left a comment
There was a problem hiding this comment.
A few more minor comments. Sorry this is such a fragmented review experience (my time is currently a bit fragmented).
I'm confident I'll run out of nitpicks on my final review.
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
ocfnash
left a comment
There was a problem hiding this comment.
Thanks so much for all of this!
I have a final round of minor suggestions but otherwise this looks great to me.
bors d+
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
Mathlib/GroupTheory/SpecificGroups/Alternating/Centralizer.lean
Outdated
Show resolved
Hide resolved
|
✌️ AntoineChambert-Loir can now approve this pull request. To approve and merge a pull request, simply reply with |
Co-authored-by: Oliver Nash <7734364+ocfnash@users.noreply.github.com>
|
bors r+ |
… centralizer of a permutation in the alternating group (#17047) Let α : Type with Fintype α (and DecidableEq α). The main goal of this file is to compute the cardinality of conjugacy classes in `alternatingGroup α`. Every g : Equiv.Perm α has a cycleType α : Multiset ℕ. By Equiv.Perm.isConj_iff_cycleType_eq, two permutations are conjugate in Equiv.Perm α iff their cycle types are equal. To compute the cardinality of the conjugacy classes, we could use a purely combinatorial approach and compute the number of permutations with given cycle type but we resorted to a more algebraic approach. This PR builds on the case of the full permutation group which is treated in #17522 Co-authored-by: leanprover-community-mathlib4-bot <leanprover-community-mathlib4-bot@users.noreply.github.com>
|
Pull request successfully merged into master. Build succeeded: |
Let α : Type with Fintype α (and DecidableEq α).
The main goal of this file is to compute the cardinality of
conjugacy classes in
alternatingGroup α.Every g : Equiv.Perm α has a cycleType α : Multiset ℕ.
By Equiv.Perm.isConj_iff_cycleType_eq,
two permutations are conjugate in Equiv.Perm α iff
their cycle types are equal.
To compute the cardinality of the conjugacy classes, we could use
a purely combinatorial approach and compute the number of permutations
with given cycle type but we resorted to a more algebraic approach.
This PR builds on the case of the full permutation group which is treated in #17522