Changes between Version 2 and Version 3 of Ticket #29683, comment 38


Ignore:
Timestamp:
05/18/21 05:37:28 (5 months ago)
Author:
yzh
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #29683, comment 38

    v2 v3  
    7878Using 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.
    7979
    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.
     80Similarly, 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.
    8181
    8282Perhaps, 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.