#13638: fix adjacency of rays
--------------------------------+-------------------------------------------
Reporter: vbraun | Owner: mhampton
Type: defect | Status: needs_review
Priority: major | Milestone: sage-5.5
Component: geometry | Resolution:
Keywords: | Work issues:
Report Upstream: N/A | Reviewers:
Authors: Volker Braun | Merged in:
Dependencies: | Stopgaps:
--------------------------------+-------------------------------------------
Comment (by dimpase):
Replying to [comment:17 vbraun]:
> The only trailing whitespace policy that we have in effect is to not
care about trailing whitespace. In particular, this might break my other
patches on top of this one for no good reason.
{{{
sage: P = Polyhedron(vertices=[(0, 1), (1, 0), (2, 0), (3, 0), (4, 1)],
rays=[(0,1)])
sage: P.adjacency_matrix?
...
Definition: P.adjacency_matrix(self)
Docstring:
This is an alias for "vertex_adjacency_matrix()"
...
}}}
this seems to indicate that only vertices can be adjacent, no?
Another very confusing thing is
{{{
Two V-representation objects are adjacent if they generate a
975 (1-dimensional) face of the polyhedron. Examples
are two
976 vertices of a polytope that bound an edge, or a
vertex and a
977 ray of a polyhedron that generate a bounding half-
line of the
978 polyhedron.
}}}
in {{{geometry/polyhedron/base.py}}}.
IMHO the whole documentation for the adjacency is a mess. I am unable to
make sense out of it, and the link to PPL docs (which aren't clear to me
at all) is not helping much, either. Please, please, define what is meant
by "ray" somewhere. I just don't get it...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13638#comment:20>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
