Skip to content

[Merged by Bors] - feat: port Order.RelClasses#560

Closed
pechersky wants to merge 16 commits intomasterfrom
pechersky/port-order-rel-classes
Closed

[Merged by Bors] - feat: port Order.RelClasses#560
pechersky wants to merge 16 commits intomasterfrom
pechersky/port-order-rel-classes

Conversation

@pechersky
Copy link
Copy Markdown
Contributor

@pechersky pechersky commented Nov 8, 2022

mathlib3 hash: fd47bdf09e90f553519c712378e651975fe8c829


checked against mathlib3port using:

leanf=Order/RelClasses.lean; diff Mathlib/$leanf <(rg -v "^#print" ../mathlib3port/Mathbin/$leanf) -d -y --suppress-common-lines | rg -w "(inductive|structure|class|lemma|theorem|def|instance)"

@pechersky pechersky added blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) awaiting-review help-wanted The author needs attention to resolve issues labels Nov 8, 2022
@digama0 digama0 added the mathlib-port This is a port of a theory file from mathlib. label Nov 13, 2022
@kim-em kim-em removed the blocked-by-other-PR This PR depends on another PR (this label is automatically managed by a bot) label Nov 13, 2022
Comment on lines +457 to +458
-- TODO: How to align this?
-- it was `⟨measure_wf f⟩` before
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.

Just don't align?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Basically, the question is, in places where measure_wf f is used, now one would use (measure \.).wf. Ideally, that transformation is something that mathport can handle.

@pechersky pechersky removed help-wanted The author needs attention to resolve issues needs-mathlib-SHA labels Nov 18, 2022
@kim-em
Copy link
Copy Markdown
Contributor

kim-em commented Nov 18, 2022

bors merge

bors bot pushed a commit that referenced this pull request Nov 18, 2022
mathlib3 hash: fd47bdf09e90f553519c712378e651975fe8c829

- [x] depends on: #556



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

bors bot commented Nov 18, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Order.RelClasses [Merged by Bors] - feat: port Order.RelClasses Nov 18, 2022
@bors bors bot closed this Nov 18, 2022
@bors bors bot deleted the pechersky/port-order-rel-classes branch November 18, 2022 03:44
bors bot pushed a commit that referenced this pull request Nov 18, 2022
mathlib3 hash: fd47bdf09e90f553519c712378e651975fe8c829

- [x] depends on: #567 
- [x] depends on: #560 
- [x] depends on: #569 



Co-authored-by: Yakov Pechersky <ypechersky@treeline.bio>
Co-authored-by: Vierkantor <vierkantor@vierkantor.com>
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
bors bot pushed a commit that referenced this pull request Nov 23, 2022
This note was added in #560. Since then, `@[mk_iff]` was implemented in #561, and was added to `IsWellFounded` in #663.
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.

5 participants