-
Notifications
You must be signed in to change notification settings - Fork 298
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] - chore(analysis/convex/strict_convex_space): generalize/add strict_convex_space.of_*
lemmas
#17206
Conversation
urkud
commented
Oct 27, 2022
•
edited by github-actions
bot
edited by github-actions
bot
- depends on: [Merged by Bors] - refactor(analysis/convex/strict): rename, golf, generalize #17464
It suffices to verify `∥x + y∥ = ∥x∥ + ∥y∥ → same_ray ℝ x y` for nonzero vectors.
src/analysis/convex/segment.lean
Outdated
lemma open_segment_subset_union (c : 𝕜) (x y : E) : | ||
open_segment 𝕜 x y ⊆ insert (line_map x y c) (open_segment 𝕜 x (line_map x y c) ∪ | ||
open_segment 𝕜 (line_map x y c) y) := |
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.
This statement could do with a docstring explanation.
EDIT: If you modify it according to my next comment, maybe not.
rcases lt_trichotomy a c with hac|rfl|hca, | ||
{ right, left, | ||
have hc : 0 < c, from h₀.trans hac, | ||
refine ⟨a / c, ⟨div_pos h₀ hc, (div_lt_one hc).2 hac⟩, _⟩, |
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.
refine ⟨a / c, ⟨div_pos h₀ hc, (div_lt_one hc).2 hac⟩, _⟩, | |
refine ⟨a / c, ⟨by positivity, (div_lt_one hc).2 hac⟩, _⟩, |
and in other places.
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.
What's wrong with short *_pos
proofs?
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.
Not much, but I thought I would showcase positivity
!
src/analysis/convex/topology.lean
Outdated
/-- A convex set `s` is strictly convex provided that for any two distinct points of | ||
`s \ interior s`, the line passing through these points has nonempty intersection with | ||
`interior s`. -/ | ||
lemma convex.to_strict_convex' {s : set E} (hs : convex 𝕜 s) |
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.
lemma convex.to_strict_convex' {s : set E} (hs : convex 𝕜 s) | |
lemma convex.strict_convex' {s : set E} (hs : convex 𝕜 s) |
The to_
convention is reserved for instance names.
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.
We already had convex.strict_convex
. I renamed the old lemma to convex.strict_convex_of_open
and golfed lemmas about strict_convex_Ixx
.
This PR/issue depends on: |
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.
bors merge
🔒 Permission denied Existing reviewers: click here to make YaelDillies a reviewer |
@YaelDillies Did you mean |
Uh yeah sorry. maintainer merge |
🚀 Pull request has been placed on the maintainer queue by YaelDillies. |
bors merge |
…nvex_space.of_*` lemmas (#17206)
Pull request successfully merged into master. Build succeeded: |
strict_convex_space.of_*
lemmasstrict_convex_space.of_*
lemmas