Changes between Version 2 and Version 3 of Ticket #29683, comment 38
- Timestamp:
- 05/18/21 05:37:28 (12 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #29683, comment 38
v2 v3 78 78 Using this order with inequalities first, there will be no strange offset treatment needed for `meet_of_facets`. We just require that the indices of `CP.meet_of_facets(*indices)` cannot exceed `CP.n_facets()`. Raise error otherwise. 79 79 80 Similarly, in `CP.Vrepresentation()`, the order is vertices+rays+lines (as in the code right now). Thus, for `CP.join_of_Vrep(*indices) , we just require that indices cannot exceed `CP.n_vertices() + CP.n_rays()`. Raise error otherwise.80 Similarly, in `CP.Vrepresentation()`, the order is vertices+rays+lines (as in the code right now). Thus, for `CP.join_of_Vrep(*indices)`, we just require that indices cannot exceed `CP.n_vertices() + CP.n_rays()`. Raise error otherwise. 81 81 82 82 Perhaps, it would be useful to have `CP.rays/lines/inequalities/equations` and `cf.vertices/rays/lines/inequalities/equations` just for output? Not sure about this, but at least `CP.n_rays()` seems needed.