#12463: Polyhedron intersection and equality inconsistencies
------------------------+---------------------------------------------------
Reporter: lftabera | Owner: mhampton
Type: defect | Status: new
Priority: major | Milestone: sage-5.0
Component: geometry | Keywords: polyhedron, equality, intersection
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
------------------------+---------------------------------------------------
Description changed by novoselt:
Old description:
> First of all, polyhedron __eq__ documentation is wrong. It states
>
> """
> def __eq__(self, other):
> """
> Test whether ``self`` is a strict sub-polyhedron of ``other``.
> """
>
> This does not correspond with the actual code nor the test.
>
> Morevover, intersection of polyhedra produces output with wrong internal
> representation.
>
> {{{
> sage: P=Polyhedron([[0,0]], rays=[[1,1]])
> sage: Q1=Polyhedron([[0,0],[1,1]])
> sage: Q2=Polyhedron([[1,0],[0,1]])
> sage: Q=Q1.intersection(Q2)
> sage: Q==P
> True
> sage: Q < P
> False
> }}}
New description:
First of all, polyhedron __eq__ documentation is wrong. It states
{{{
def __eq__(self, other):
"""
Test whether ``self`` is a strict sub-polyhedron of ``other``.
}}}
This does not correspond with the actual code nor the test.
Morevover, intersection of polyhedra produces output with wrong internal
representation.
{{{
sage: P=Polyhedron([[0,0]], rays=[[1,1]])
sage: Q1=Polyhedron([[0,0],[1,1]])
sage: Q2=Polyhedron([[1,0],[0,1]])
sage: Q=Q1.intersection(Q2)
sage: Q==P
True
sage: Q < P
False
}}}
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12463#comment:1>
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.
