Skip to content

[Merged by Bors] - port logic.embedding.basic#700

Closed
kbuzzard wants to merge 15 commits intomasterfrom
kbuzzard-logic-embedding-basic
Closed

[Merged by Bors] - port logic.embedding.basic#700
kbuzzard wants to merge 15 commits intomasterfrom
kbuzzard-logic-embedding-basic

Conversation

@kbuzzard
Copy link
Copy Markdown
Member

@kbuzzard kbuzzard commented Nov 23, 2022

From mathlib3port: e50b8c261b0a000b806ec0e1356b41945eda61f7

@kim-em kim-em added the WIP Work in progress label Nov 24, 2022
@urkud
Copy link
Copy Markdown
Member

urkud commented Nov 24, 2022

@kbuzzard Instructions page says to make a commit with mathlib3port version first so that one can easily see the diff between auto conversion and final version.

@urkud urkud added awaiting-review mathlib-port This is a port of a theory file from mathlib. and removed WIP Work in progress labels Nov 24, 2022
@kbuzzard
Copy link
Copy Markdown
Member Author

Thanks Yury for working on this when I was asleep! I've added the commit from the mathlib3port README. This has been marked "awaiting review" but I left several TODOs in the text which are still there.

@urkud
Copy link
Copy Markdown
Member

urkud commented Nov 24, 2022

@kbuzzard I see 1 TODO that is still there (about trans). I hope that someone will tell us what to do about it.

@kbuzzard
Copy link
Copy Markdown
Member Author

I've added the Trans instance. This was what I was told was the appropriate replacement for the @[trans] attribute when I ported logic.relation. I think this is ready for review now.

/-- Convert an `α ≃ β` to `α ↪ β`.

This is also available as a coercion `equiv.coe_embedding`.
The explicit `equiv.to_embedding` version is preferred though, since the coercion can have issues
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Case needs fixing (twice)

@kim-em kim-em added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Nov 25, 2022
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@kim-em kim-em added awaiting-author A reviewer has asked the author a question or requested changes. and removed awaiting-review labels Nov 27, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 28, 2022

bros merge

@bors
Copy link
Copy Markdown

bors bot commented Nov 28, 2022

👊

@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 28, 2022

Um, okay. :-)

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-author A reviewer has asked the author a question or requested changes. labels Nov 28, 2022
bors bot pushed a commit that referenced this pull request Nov 28, 2022
From mathlib3port: e50b8c261b0a000b806ec0e1356b41945eda61f7

Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
Co-authored-by: Scott Morrison <scott@tqft.net>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link
Copy Markdown

bors bot commented Nov 28, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title port logic.embedding.basic [Merged by Bors] - port logic.embedding.basic Nov 28, 2022
@bors bors bot closed this Nov 28, 2022
@bors bors bot deleted the kbuzzard-logic-embedding-basic branch November 28, 2022 02:06
bors bot pushed a commit that referenced this pull request Nov 28, 2022
I believe these were accidentally added by @semorrison in 06f1a77 as part of #700
bors bot pushed a commit that referenced this pull request Nov 28, 2022
I believe these were accidentally added by @semorrison in 06f1a77 as part of #700
bors bot pushed a commit to leanprover-community/mathlib3 that referenced this pull request Dec 1, 2022
Regenerated from the [port status wiki page](https://github.com/leanprover-community/mathlib/wiki/mathlib4-port-status).
Relates to the following PRs:
* `algebra.field.defs`: leanprover-community/mathlib4#668
* `algebra.group.commute`: leanprover-community/mathlib4#750
* `algebra.group_with_zero.commute`: leanprover-community/mathlib4#762
* `algebra.group_with_zero.semiconj`: leanprover-community/mathlib4#757
* `algebra.group_with_zero.units.basic`: leanprover-community/mathlib4#735
* `algebra.hom.embedding`: leanprover-community/mathlib4#764
* `algebra.order.monoid.cancel.defs`: leanprover-community/mathlib4#774
* `algebra.order.monoid.canonical.defs`: leanprover-community/mathlib4#778
* `algebra.order.monoid.defs`: leanprover-community/mathlib4#771
* `algebra.order.monoid.min_max`: leanprover-community/mathlib4#763
* `algebra.order.monoid.order_dual`: leanprover-community/mathlib4#786
* `algebra.order.sub.defs`: leanprover-community/mathlib4#732
* `algebra.regular.basic`: leanprover-community/mathlib4#758
* `algebra.ring.commute`: leanprover-community/mathlib4#759
* `algebra.ring.regular`: leanprover-community/mathlib4#795
* `algebra.ring.semiconj`: leanprover-community/mathlib4#751
* `control.applicative`: leanprover-community/mathlib4#798
* `control.functor`: leanprover-community/mathlib4#612
* `control.monad.basic`: leanprover-community/mathlib4#752
* `data.countable.defs`: leanprover-community/mathlib4#736
* `data.int.units`: leanprover-community/mathlib4#807
* `data.nat.basic`: leanprover-community/mathlib4#729
* `data.nat.psub`: leanprover-community/mathlib4#806
* `data.nat.units`: leanprover-community/mathlib4#805
* `data.pi.algebra`: leanprover-community/mathlib4#564
* `data.prod.lex`: leanprover-community/mathlib4#783
* `logic.embedding.basic`: leanprover-community/mathlib4#700
* `logic.equiv.option`: leanprover-community/mathlib4#674
* `order.bounded_order`: leanprover-community/mathlib4#697
* `order.with_bot`: leanprover-community/mathlib4#776
* `order.disjoint`: leanprover-community/mathlib4#773
* `order.min_max`: leanprover-community/mathlib4#728
* `order.prop_instances`: leanprover-community/mathlib4#792
* `order.rel_iso.basic`: leanprover-community/mathlib4#772
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. ready-to-merge This PR has been sent to bors.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants