#6991: lattice polytope nfacets method broken for non-reflexive polytopes
----------------------+-----------------------------------------------------
Reporter: mhampton | Owner: mhampton
Type: defect | Status: new
Priority: minor | Milestone: sage-4.1.2
Component: geometry | Keywords:
Reviewer: | Author:
Merged: |
----------------------+-----------------------------------------------------
In the nfacets method, for non-reflexive lattice polytopes this is
computed from:
{{{
self._nfacets = len(self._facet_normals)
}}}
but self._facet_normals is a matrix, which does not have a len method. So
I think this should instead be
{{{
self._nfacets = self._facet_normals.nrows()
}}}
A doctest should also be added for this case.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6991>
Sage <http://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
-~----------~----~----~----~------~----~------~--~---
