-
Notifications
You must be signed in to change notification settings - Fork 234
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: port Analysis.Convex.Body #3431
Conversation
urkud
commented
Apr 14, 2023
Mathbin -> Mathlib fix certain import statements move "by" to end of line add import to Mathlib.lean
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 d+
Mathlib/Analysis/Convex/Body.lean
Outdated
@[simp] -- porting note: todo: add `norm_cast` | ||
theorem coe_add (K L : ConvexBody V) : (↑(K + L) : Set V) = (K : Set V) + L := |
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.
I think it would be good to make these porting notes more explicit. I guess the idea is that norm_cast
is not present in mathlib3 but should have been, but during the port it's better to stick to mathlib3 behavior and fix it later -- right? In this case how about "porting note: add norm_cast
; we leave it out for now to reproduce mathlib3 behavior."
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.
If` someone wants this PR to be merged before Wednesday, April 19: feel free to update the porting notes as suggested & merge. I have a few deadlines in a row right now... I'll fix it in a few minutes.
✌️ urkud can now approve this pull request. To approve and merge a pull request, simply reply with |
bors merge |
Pull request successfully merged into master. Build succeeded: |