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: port remaining missing Algebra.GroupWithZero.Defs #563

Closed
wants to merge 23 commits into from

Conversation

pechersky
Copy link
Collaborator

@pechersky pechersky commented Nov 9, 2022

Tracking mathlib3 sha: 39af7d3bf61a98e928812dbc3e16f4ea8b795ca3

The file had been missing some classes, added those.
Apart from adding missing classes, put in docstrings for the module, as well as classes and lemmas that were missing them.
Turns out this file doesn't rely on the Logic.Nontrivial import.
This file still has AddMonoidWithOne at the bottom of the file -- this is in dat.nat.cast.defs in mathlib3 -- I left it in because it is likely in this file to assist with downstream tactic usage.

Along with module, class, and some lemma docstrings
I kept the additional later-file defns near the bottom
Turns out this file doesn't rely on the Logic.Nontrivial import
@semorrison
Copy link
Contributor

I don't understand what you mean by

Along with module, class, and some lemma docstrings
I kept the additional later-file defns near the bottom

@pechersky
Copy link
Collaborator Author

I tried to clarify the PR description.

@semorrison
Copy link
Contributor

Hmm, I don't like this. When it's time to port this file, we should be completely replacing its contents with a one-to-one port of the mathlib3 file.

If there's "junk" in here from previous ad-hoc porting, that is still necessary on a temporary basis, lets move it out to a new file, possibly with some alarming name like Mathlib/Temporary/Algebra/GroupWithZero/Defs.lean...?

@pechersky
Copy link
Collaborator Author

Makes sense to me. Can we just port data.nat.cast.defs to not have to do a three-body problem?

@digama0 digama0 added the mathlib-port This is a port of a theory file from mathlib. label Nov 13, 2022
@j-loreaux
Copy link
Collaborator

j-loreaux commented Nov 18, 2022

I have created #641 to port Mathlib.Data.Nat.Cast.Defs. That PR moves the AddMonoidWithOne and related lemmas out of this file (but leaves AddGroupWithOne for now because that belongs in Mathlib.Data.Int.Cast.Defs). The changes were significant enough that I thought it should have it's own PR instead of incorporating the changes here. So you may want to make this depends on that.

@j-loreaux
Copy link
Collaborator

It looks like data.int.cast.defs is only waiting on data.nat.cast.defs, so I'll just do that now and see if I can make it into one PR.

@semorrison
Copy link
Contributor

@pechersky, could you work out how to merge master on this one?

@semorrison
Copy link
Contributor

There was a bunch more that still needed porting, so I added that. Fortunately it was trivial.

@semorrison
Copy link
Contributor

bors merge

bors bot pushed a commit that referenced this pull request Nov 20, 2022
Tracking mathlib3 sha: 39af7d3bf61a98e928812dbc3e16f4ea8b795ca3

The file had been missing some classes, added those.
Apart from adding missing classes, put in docstrings for the module, as well as classes and lemmas that were missing them.
Turns out this file doesn't rely on the Logic.Nontrivial import.
This file still has `AddMonoidWithOne` at the bottom of the file -- this is in `dat.nat.cast.defs` in mathlib3 -- I left it in because it is likely in this file to assist with downstream tactic usage.

Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@bors
Copy link

bors bot commented Nov 20, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port remaining missing Algebra.GroupWithZero.Defs [Merged by Bors] - feat: port remaining missing Algebra.GroupWithZero.Defs Nov 20, 2022
@bors bors bot closed this Nov 20, 2022
@bors bors bot deleted the pechersky/port-group-with-zero-defs branch November 20, 2022 07:14
bors bot pushed a commit that referenced this pull request Dec 2, 2022
mathlib3 SHA: 8c53048add6ffacdda0b36c4917bfe37e209b0ba

- [x] depends on #563
- [x] depends on leanprover-community/mathlib#17733
- [x] depends on leanprover/lean4#1901
- [x] depends on leanprover/lean4#1907

Co-authored-by: Winston Yin <winstonyin@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Co-authored-by: Jireh Loreaux <loreaujy@gmail.com>
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mathlib-port This is a port of a theory file from mathlib.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants