#6778: [with patch, needs review] Fix nfacets for non-reflexive lattice 
polytopes
----------------------+-----------------------------------------------------
 Reporter:  novoselt  |       Owner:  mhampton  
     Type:  defect    |      Status:  new       
 Priority:  major     |   Milestone:  sage-4.1.2
Component:  geometry  |    Keywords:            
 Reviewer:            |      Author:            
   Merged:            |  
----------------------+-----------------------------------------------------
 There is a silly bug in computing the number of facets of non-reflexive
 lattice polytopes:
 {{{
 sage: p = LatticePolytope(matrix([1, 2]))
 sage: p.nfacets()
 Traceback (most recent call last):
 ...
 TypeError: object of type
 'sage.matrix.matrix_integer_dense.Matrix_integer_dense' has no len()
 }}}
 The attached one-line patch fixes it:
 {{{
 sage: p = LatticePolytope(matrix([1, 2]))
 sage: p.nfacets()
 2
 }}}

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/6778>
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to