Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(group_theory/free_group): norm of elements #15503

Closed
wants to merge 31 commits into from

Conversation

kisonecat
Copy link
Collaborator

This PR adds to a new free_group.metric namespace some useful lemmas related to the word metric on the free group. This will be useful for defining the word metric for an arbitrary (marked) group.


Open in Gitpod

kisonecat and others added 2 commits July 18, 2022 20:15
Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
@kisonecat kisonecat added the awaiting-review The author would like community review of the PR label Jul 19, 2022
@fpvandoorn fpvandoorn added the awaiting-CI The author would like to see what CI has to say before doing more work. label Jul 19, 2022
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Jul 19, 2022
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
@robertylewis robertylewis added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jul 20, 2022
kisonecat and others added 5 commits July 19, 2022 21:47
Co-authored-by: Rob Lewis <Rob.y.lewis@gmail.com>
Co-authored-by: Rob Lewis <Rob.y.lewis@gmail.com>
Co-authored-by: Rob Lewis <Rob.y.lewis@gmail.com>
Co-authored-by: Rob Lewis <Rob.y.lewis@gmail.com>
Co-authored-by: Rob Lewis <Rob.y.lewis@gmail.com>
@kisonecat
Copy link
Collaborator Author

Thank you @robertylewis and @urkud for your feedback on this.

@kisonecat kisonecat added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 20, 2022
Copy link
Member

@urkud urkud left a comment

Choose a reason for hiding this comment

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

I have a few more comments. Thank you for working on this!

src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
@urkud urkud added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jul 20, 2022
@kisonecat kisonecat added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jul 21, 2022
@urkud
Copy link
Member

urkud commented Jul 24, 2022

I'm going to cleanup the code locally, then push changes.

@urkud
Copy link
Member

urkud commented Jul 24, 2022

I golfed some proofs but now we need a review from another maintainer.

Copy link
Collaborator

@ocfnash ocfnash left a comment

Choose a reason for hiding this comment

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

Thanks!

src/group_theory/free_group.lean Outdated Show resolved Hide resolved
src/group_theory/free_group.lean Outdated Show resolved Hide resolved
@ocfnash
Copy link
Collaborator

ocfnash commented Aug 22, 2022

Thanks again to both of you.

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 Aug 22, 2022
bors bot pushed a commit that referenced this pull request Aug 22, 2022
This PR adds to a new `free_group.metric` namespace some useful lemmas related to the word metric on the free group.  This will be useful for defining the word metric for an arbitrary (marked) group.



Co-authored-by: Jim Fowler <fowler@math.osu.edu>
Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
Co-authored-by: Oliver Nash <github@olivernash.org>
@bors
Copy link

bors bot commented Aug 22, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(group_theory/free_group): norm of elements [Merged by Bors] - feat(group_theory/free_group): norm of elements Aug 22, 2022
@bors bors bot closed this Aug 22, 2022
@bors bors bot deleted the free_group_metric branch August 22, 2022 12:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

6 participants