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: missing API lemmas about the prod instance in Order/CompleteLattice #1369

Closed
wants to merge 2 commits into from

Conversation

mans0954
Copy link
Collaborator

@mans0954 mans0954 commented Jan 6, 2023

This adds lemmas about how fst, snd, and swap interact with supₛ and infₛ.

Manual port of leanprover-community/mathlib#18029 by @eric-wieser

@hrmacbeth hrmacbeth added the mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged label Jan 6, 2023
@eric-wieser
Copy link
Member

Thanks for doing this, but I think it would be best to wait for mathport to run and discard the manual port in favor of the automatic one. There are some ongoing discussions on Zulip about that.

@mans0954
Copy link
Collaborator Author

mans0954 commented Jan 7, 2023

Thanks for doing this, but I think it would be best to wait for mathport to run and discard the manual port in favor of the automatic one. There are some ongoing discussions on Zulip about that.

I take it that's here and also here

@alreadydone alreadydone changed the title feat(order/complete_lattice): missing API lemmas about the prod instance feat: missing API lemmas about the prod instance in Order/CompleteLattice Jan 7, 2023
@eric-wieser eric-wieser self-assigned this Jan 9, 2023
@eric-wieser
Copy link
Member

@Ruben-VandeVelde Ruben-VandeVelde added the awaiting-author A reviewer has asked the author a question or requested changes label Jan 9, 2023
@eric-wieser eric-wieser 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 Jan 13, 2023
Copy link
Member

@kbuzzard kbuzzard left a comment

Choose a reason for hiding this comment

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

LTGM

@fpvandoorn
Copy link
Member

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Jan 25, 2023
bors bot pushed a commit that referenced this pull request Jan 25, 2023
…tice (#1369)

This adds lemmas about how `fst`, `snd`, and `swap` interact with `supₛ` and `infₛ`.

Manual port of a [mathlib PR](leanprover-community/mathlib#18029) by @eric-wieser 

Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
@bors
Copy link

bors bot commented Jan 25, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: missing API lemmas about the prod instance in Order/CompleteLattice [Merged by Bors] - feat: missing API lemmas about the prod instance in Order/CompleteLattice Jan 25, 2023
@bors bors bot closed this Jan 25, 2023
@bors bors bot deleted the mans0954/prod.complete_lattice branch January 25, 2023 11:26
@eric-wieser
Copy link
Member

Whoops, we forgot to update the SHA here.

bors bot pushed a commit that referenced this pull request Mar 5, 2023
#1369 forward-ported these changes, but did not update the SHA.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mathlib3-pair This PR is a forward-port of a mathlib3 PR or part of one, either under review or recently merged ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants