-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
Polyhedron.relative_interior_manifold, as_manifold_subset #31660
Comments
Author: Matthias Koeppe |
Last 10 new commits:
|
Commit: |
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
This comment has been minimized.
This comment has been minimized.
comment:20
With #31916, instead of introducing |
comment:21
Likewise with #31963 for |
Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
|
We extend the
ManifoldSubsetPullback
constructor to handle relatively open convex sets.In this case, we
ConvexSet_base.affine_hull_projection
(ConvexSet_base: Add affine_hull, affine_hull_projection, an_affine_basis #31993), generalized fromPolyhedron.affine_hull_manifold
(Polyhedron.affine_hull_manifold #31659)We add
ConvexSet_base
methodsas_manifold_subset
.relative_interior_manifold
Next step:
Polyhedron.face_manifold_poset
, top element of which is the polyhedron as a union of its relative interior and its facets. Using #31732, the intersection of the relative interior and each facet is declared empty.Depends on #31688
Depends on #31963
Depends on #31993
CC: @mjungmath @kliem @yuan-zhou @jplab @tscrim @egourgoulhon
Component: manifolds
Work Issues: Refactor through #31688, #31916, #31963, #31993
Author: Matthias Koeppe
Branch/Commit: u/mkoeppe/polyhedron_relative_interior_manifold @
1f4bd68
Issue created by migration from https://trac.sagemath.org/ticket/31660
The text was updated successfully, but these errors were encountered: