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] - refactor(group_theory/order_of_element): use minimal_period for the definition #7082

Closed
wants to merge 104 commits into from

Conversation

Julian-Kuelshammer
Copy link
Collaborator

@Julian-Kuelshammer Julian-Kuelshammer commented Apr 7, 2021

This PR redefines order_of_element in terms of function.minimal_period. It furthermore introduces a predicate on elements of a finite group to be of finite order.

Co-authored by: Damiano Testa adomani@gmail.com


Open in Gitpod

@Julian-Kuelshammer Julian-Kuelshammer added the WIP Work in progress label Apr 7, 2021
Julian-Kuelshammer and others added 2 commits April 7, 2021 18:02
Co-authored-by: damiano <adomani@gmail.com>
src/group_theory/order_of_element.lean Outdated Show resolved Hide resolved
src/group_theory/order_of_element.lean Outdated Show resolved Hide resolved
@Julian-Kuelshammer Julian-Kuelshammer added the RFC Request for comment label Apr 7, 2021
@github-actions github-actions bot added the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Apr 21, 2021
@github-actions github-actions bot added merge-conflict Please `git merge origin/master` then a bot will remove this label. and removed merge-conflict Please `git merge origin/master` then a bot will remove this label. labels Apr 23, 2021
Merge branch 'master' into minimal_period
@github-actions github-actions bot added merge-conflict Please `git merge origin/master` then a bot will remove this label. and removed merge-conflict Please `git merge origin/master` then a bot will remove this label. labels Apr 27, 2021
@github-actions github-actions bot removed the merge-conflict Please `git merge origin/master` then a bot will remove this label. label Apr 29, 2021
Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks a lot for this refactor! And sorry that I didn't look at it sooner.
This is looking very good. Lots of proof golfing 🐙 🎉

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 Apr 30, 2021
bors bot pushed a commit that referenced this pull request Apr 30, 2021
…efinition (#7082)

This PR redefines `order_of_element` in terms of `function.minimal_period`. It furthermore introduces a predicate on elements of a finite group to be of finite order.

Co-authored by: Damiano Testa adomani@gmail.com



Co-authored-by: Julian-Kuelshammer <68201724+Julian-Kuelshammer@users.noreply.github.com>
Co-authored-by: adomani <adomani@gmail.com>
@bors
Copy link

bors bot commented Apr 30, 2021

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request May 1, 2021
…efinition (#7082)

This PR redefines `order_of_element` in terms of `function.minimal_period`. It furthermore introduces a predicate on elements of a finite group to be of finite order.

Co-authored by: Damiano Testa adomani@gmail.com



Co-authored-by: Julian-Kuelshammer <68201724+Julian-Kuelshammer@users.noreply.github.com>
Co-authored-by: adomani <adomani@gmail.com>
@bors
Copy link

bors bot commented May 1, 2021

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title refactor(group_theory/order_of_element): use minimal_period for the definition [Merged by Bors] - refactor(group_theory/order_of_element): use minimal_period for the definition May 1, 2021
@bors bors bot closed this May 1, 2021
@bors bors bot deleted the minimal_period branch May 1, 2021 05:11
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

6 participants