f_vector of CombinatorialPolyhedron is a vector

sagemath · Oct 15, 2019 · 326602c · 326602c
1 parent 0f20b37
commit 326602c
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/src/sage/geometry/polyhedron/combinatorial_polyhedron/base.pyx b/src/sage/geometry/polyhedron/combinatorial_polyhedron/base.pyx
@@ -184,7 +184,7 @@ cdef class CombinatorialPolyhedron(SageObject):
     an integer::
 
         sage: CombinatorialPolyhedron(-1).f_vector()
-        (1,)
+        (1)
         sage: CombinatorialPolyhedron(0).f_vector()
         (1, 1)
         sage: CombinatorialPolyhedron(5).f_vector()
@@ -759,7 +759,7 @@ cdef class CombinatorialPolyhedron(SageObject):
 
             sage: C = CombinatorialPolyhedron(-1)
             sage: C.f_vector()
-            (1,)
+            (1)
             sage: C.n_facets()
             0
 
@@ -1117,12 +1117,19 @@ cdef class CombinatorialPolyhedron(SageObject):
             sage: C = CombinatorialPolyhedron(P)
             sage: C.f_vector()
             (1, 10, 45, 120, 185, 150, 50, 1)
+
+        TESTS::
+
+            sage: type(C.f_vector())
+            <type 'sage.modules.vector_integer_dense.Vector_integer_dense'>
         """
         if not self._f_vector:
             self._compute_f_vector()
         if not self._f_vector:
             raise ValueError("could not determine f_vector")
-        return self._f_vector
+        from sage.modules.free_module_element import vector
+        from sage.rings.all                   import ZZ
+        return vector(ZZ, self._f_vector)
 
     def face_iter(self, dimension=None, dual=None):
         r"""