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

[Merged by Bors] - chore(data/set/semiring): add an idem_semiring instance#18449

Closed
eric-wieser wants to merge 6 commits intomasterfrom
eric-wieser/set-semiring-kleene
Closed

[Merged by Bors] - chore(data/set/semiring): add an idem_semiring instance#18449
eric-wieser wants to merge 6 commits intomasterfrom
eric-wieser/set-semiring-kleene

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser commented Feb 15, 2023

This typeclass is reasonably new, so some obvious instances are missing.

Also adds some basic API lemmas that were missing about the casting functions set.up and set_semiring.down.

forward-ported as leanprover-community/mathlib4#2518


Open in Gitpod

@eric-wieser eric-wieser added easy < 20s of review time. See the lifecycle page for guidelines. 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. modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. labels Feb 15, 2023
@ghost ghost added blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. A bot manages this label via PR comment. labels Feb 16, 2023
@ghost
Copy link
Copy Markdown

ghost commented Feb 16, 2023

@eric-wieser eric-wieser force-pushed the eric-wieser/set-semiring-kleene branch from 319e1f8 to 8785dea Compare February 16, 2023 20:03
@github-actions github-actions bot removed the awaiting-CI The author would like to see what CI has to say before doing more work. label Feb 17, 2023
Copy link
Copy Markdown
Collaborator

@YaelDillies YaelDillies left a comment

Choose a reason for hiding this comment

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

Very nice!

maintainer merge

@github-actions
Copy link
Copy Markdown

🚀 Pull request has been placed on the maintainer queue by YaelDillies.

eric-wieser added a commit to leanprover-community/mathlib4 that referenced this pull request Feb 27, 2023
Copy link
Copy Markdown
Collaborator

@Vierkantor Vierkantor left a comment

Choose a reason for hiding this comment

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

bors d+

@bors
Copy link
Copy Markdown

bors bot commented Feb 28, 2023

✌️ eric-wieser can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@leanprover-community-bot-assistant leanprover-community-bot-assistant added delegated The PR author may merge after reviewing final suggestions. and removed awaiting-review The author would like community review of the PR labels Feb 28, 2023
Co-authored-by: Anne Baanen <Vierkantor@users.noreply.github.com>
@eric-wieser
Copy link
Copy Markdown
Member Author

bors merge

@github-actions github-actions bot added the ready-to-merge All that is left is for bors to build and merge this PR. (Remember you need to say `bors r+`.) label Feb 28, 2023
bors bot pushed a commit that referenced this pull request Feb 28, 2023
This typeclass is reasonably new, so some obvious instances are missing.

Also adds some basic API lemmas that were missing about the casting functions `set.up` and `set_semiring.down`.

forward-ported as leanprover-community/mathlib4#2518
@bors
Copy link
Copy Markdown

bors bot commented Feb 28, 2023

Build failed:

@eric-wieser
Copy link
Copy Markdown
Member Author

bors merge

bors bot pushed a commit that referenced this pull request Feb 28, 2023
This typeclass is reasonably new, so some obvious instances are missing.

Also adds some basic API lemmas that were missing about the casting functions `set.up` and `set_semiring.down`.

forward-ported as leanprover-community/mathlib4#2518
@bors
Copy link
Copy Markdown

bors bot commented Feb 28, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title chore(data/set/semiring): add an idem_semiring instance [Merged by Bors] - chore(data/set/semiring): add an idem_semiring instance Feb 28, 2023
@bors bors bot closed this Feb 28, 2023
@bors bors bot deleted the eric-wieser/set-semiring-kleene branch February 28, 2023 13:19
bors bot pushed a commit that referenced this pull request Mar 2, 2023
#18539)

Previously this was abusing the defeq of the types, resulting in lemmas stated in weird ways.

This also fixes a type in #18449, and adds three missing lemmas about `image_hom`.

Forward port of `set_semiring` will be included in leanprover-community/mathlib4#2518
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

delegated The PR author may merge after reviewing final suggestions. easy < 20s of review time. See the lifecycle page for guidelines. modifies-synchronized-file This PR touches a files that has already been ported to mathlib4, and may need a synchronization PR. 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