feat(Geometry/Convex/Cone/Pointed): face lattice of pointed cones#33664
feat(Geometry/Convex/Cone/Pointed): face lattice of pointed cones#33664ooovi wants to merge 63 commits intoleanprover-community:masterfrom
Conversation
PR summary 77d929d750Import changes for modified filesNo significant changes to the import graph Import changes for all files
|
YaelDillies
left a comment
There was a problem hiding this comment.
Per the discussion on Zulip, perhaps let's not create this new Pointed folder
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
Co-authored-by: Yaël Dillies <yael.dillies@gmail.com>
| } | ||
|
|
||
| instance : SemilatticeInf (Face C) where | ||
| inf := min |
There was a problem hiding this comment.
Do you really need to give this explicitly?
| inf := min |
There was a problem hiding this comment.
Hmm, weird. I'll have a look later
There was a problem hiding this comment.
Looks like this file has converged. Do you want to move .Lattice to a later PR so that we can move forward here?
There was a problem hiding this comment.
Do you expect much further work on Lattice? I felt we are near the end here, no?
There was a problem hiding this comment.
Maybe, I am simply trying to be helpful and proactively ensure that no part of your PR gets blocked on details in other parts
| constructor <;> intro h | ||
| · exact ⟨h.le, IsFaceOf.mem_of_add_mem h⟩ | ||
| · refine ⟨h.1, fun {x y a} xC yC a0 haxy => ?_⟩ | ||
| have := smul_mem _ (inv_nonneg.mpr (le_of_lt a0)) <| h.2 (smul_mem _ (le_of_lt a0) xC) yC haxy | ||
| simpa [← smul_assoc, inv_mul_cancel₀ (ne_of_gt a0)] using this |
There was a problem hiding this comment.
| constructor <;> intro h | |
| · exact ⟨h.le, IsFaceOf.mem_of_add_mem h⟩ | |
| · refine ⟨h.1, fun {x y a} xC yC a0 haxy => ?_⟩ | |
| have := smul_mem _ (inv_nonneg.mpr (le_of_lt a0)) <| h.2 (smul_mem _ (le_of_lt a0) xC) yC haxy | |
| simpa [← smul_assoc, inv_mul_cancel₀ (ne_of_gt a0)] using this | |
| refine ⟨fun h ↦ ⟨h.le, h.mem_of_add_mem⟩, fun ⟨h₁, h₂⟩ ↦ ⟨h₁, fun hx hy ha haxy ↦ ?_⟩⟩ | |
| simpa [← smul_assoc, inv_mul_cancel₀ (ne_of_gt ha)] using smul_mem _ | |
| (inv_nonneg.mpr (le_of_lt ha)) <| h₂ (smul_mem _ (le_of_lt ha) hx) hy haxy |
Some of the proofs could be made cleaner by making use of refine, this is an example, but the same ideas apply elsewhere too.
|
|
||
| open Submodule | ||
|
|
||
| @[expose] public section |
There was a problem hiding this comment.
Can you put it on the declarations that need it instead?
| @[expose] public section | |
| public section |
There was a problem hiding this comment.
I don't know how to determine who needs it. I somehow thought everything ought to be exposed, given how many files in mathlib are?
PointedCone.IsFaceOf, for a pointed cone being a face of another pointed cone.Faceby bundling theIsFaceOfstructure, and show the complete lattice structure on it.Co-authored-by: Martin Winter