Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

[Merged by Bors] - feat(data/quot): make trunc a quotient#18320

Closed
astrainfinita wants to merge 4 commits intomasterfrom
FR_trunc
Closed

[Merged by Bors] - feat(data/quot): make trunc a quotient#18320
astrainfinita wants to merge 4 commits intomasterfrom
FR_trunc

Conversation

@astrainfinita
Copy link
Copy Markdown
Collaborator

@astrainfinita astrainfinita commented Jan 28, 2023

This allows us to use quotient lemmas for trunc.

mathlib4 PR: leanprover-community/mathlib4#1924


Open in Gitpod

This allows us to use `quotient` lemmas for `trunc`.
@astrainfinita astrainfinita added awaiting-review The author would like community review of the PR awaiting-CI The author would like to see what CI has to say before doing more work. labels Jan 28, 2023
@astrainfinita astrainfinita changed the title feat(data/quot): make trunc a quotient feat(data/quot): make trunc a quotient Jan 28, 2023
@astrainfinita astrainfinita removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Jan 28, 2023
@fpvandoorn
Copy link
Copy Markdown
Member

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 Jan 31, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Jan 31, 2023
bors bot pushed a commit that referenced this pull request Jan 31, 2023
This allows us to use `quotient` lemmas for `trunc`.

mathlib4 PR: leanprover-community/mathlib4#1924
Comment on lines +371 to +373
/-- Always-true relation as a `setoid`. -/
def true_setoid : setoid α :=
⟨_, true_equivalence⟩
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser Jan 31, 2023

Choose a reason for hiding this comment

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

This already exists, it's ⊤ : setoid α.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's not available in data.quot.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Since this fell off the bors batch, I pushed a comment remarking that ⊤ : setoid α is another spelling of the same thing.

@bors
Copy link
Copy Markdown

bors bot commented Jan 31, 2023

Build failed (retrying...):

@urkud
Copy link
Copy Markdown
Member

urkud commented Jan 31, 2023

bors merge

@bors
Copy link
Copy Markdown

bors bot commented Jan 31, 2023

Canceled.

@eric-wieser
Copy link
Copy Markdown
Member

bors merge

bors bot pushed a commit that referenced this pull request Jan 31, 2023
This allows us to use `quotient` lemmas for `trunc`.

mathlib4 PR: leanprover-community/mathlib4#1924



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

bors bot commented Jan 31, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat(data/quot): make trunc a quotient [Merged by Bors] - feat(data/quot): make trunc a quotient Jan 31, 2023
@bors bors bot closed this Jan 31, 2023
@bors bors bot deleted the FR_trunc branch January 31, 2023 19:36
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Feb 3, 2023
This forward-ports the last commits in leanprover-community/mathlib3#18320, which was prematurely forward-ported as #1924 before it was merged in mathlib3.
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 18, 2023
bors bot pushed a commit to leanprover-community/mathlib4 that referenced this pull request Mar 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

4 participants