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(order/closure): closure of unions and bUnions #7361
Conversation
I've added two new constructors |
Okay so I've been trying my two new constructors and they seem to work great for what I want. Specifically, mk2 and mk3 (if you think of better names...) do the same, but you can extract a sufficient property for being closed from mk3 (and not from mk2). This allows you for example to get |
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, modulo Eric's and my suggestions.
I've added a bunch of simp declarations. To this end, some lemmas have been turned around. I've hesitated a bit for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🎉
bors merge
prove closure_closure_union and similar
This PR was included in a batch that was canceled, it will be automatically retried |
prove closure_closure_union and similar
Build failed (retrying...): |
prove closure_closure_union and similar
prove closure_closure_union and similar
Pull request successfully merged into master. Build succeeded: |
prove closure_closure_union and similar