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
sage6.4
tosage9.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 1dimensional 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 Vrepresentation as no Vrepresentation given at all. 24 25 The attached branch fixes the Hrepresentation part, but already that comes at the cost of failing doctests. The real deal is fixing the Vrepresentation (which then should fix the doctests as well).