Changes between Initial Version and Version 15 of Ticket #17339
- Timestamp:
- 12/11/19 15:02:15 (2 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #17339
-
Property
Status
changed from
new
toneeds_review
-
Property
Authors
changed from
to
Darij Grinberg, Jonathan Kliem
- Property Cc mkoeppe tscrim added
-
Property
Branch
changed from
public/polytopes/0
topublic/17339
-
Property
Milestone
changed from
sage-6.4
tosage-9.0
-
Property
Commit
changed from
f7642d6e2d2502e3df95c76ef2ffa7e9319a99aa
toc5bf04100b1b1bbd32e7c85e0096f69934948811
-
Property
Status
changed from
-
Ticket #17339 – Description
initial v15 12 12 13 13 {{{ 14 sage: Polyhedron(ambient_dim=2, vertices=[], rays=[(2,2)], lines=[], base_ring=QQ)15 A 1-dimensional polyhedron in QQ^2 defined as the convex hull of 1 vertex and 1 ray16 }}}17 This should be the empty polyhedron, since there are no vertices for the ray to emanate from.18 19 {{{20 14 sage: Polyhedron(ambient_dim=2, vertices=[], rays=[], lines=[], base_ring=QQ) 21 15 The empty polyhedron in QQ^2 22 16 }}} 23 17 This is correct, but only because the code misunderstands the empty V-representation as no V-representation given at all. 24 25 The attached branch fixes the H-representation part, but already that comes at the cost of failing doctests. The real deal is fixing the V-representation (which then should fix the doctests as well).