Opened 4 years ago

Last modified 4 years ago

#18861 new defect

Three apparently useless polyhedron methods

Reported by: ncohen Owned by:
Priority: major Milestone: sage-6.8
Component: geometry Keywords:
Cc: dimpase, vbraun, vdelecroix Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:


It seems that the three following functions are not used anywhere

(polgraph|✚2…)~/sage/geometry$ grep _make_polyh . -R
./polyhedron/    _make_polyhedron_face.
./polyhedron/    def _make_polyhedron_face(self, Vindices, Hindices):
./polyhedron/            sage: square._make_polyhedron_face((0,2), (1,))
./polyhedron/            return self._make_polyhedron_face(Vindices, Hindices)
(polgraph|✚2…)~/sage/geometry$ grep _init_fac . -R  
./polyhedron/    _init_facet_adjacency_matrix, _init_vertex_adjacency_matrix, and
./polyhedron/    def _init_facet_adjacency_matrix(self, verbose=False):
./polyhedron/            sage: p._init_facet_adjacency_matrix()
(polgraph|✚2…)~/sage/geometry$ grep _init_vertex . -R
./polyhedron/    _init_facet_adjacency_matrix, _init_vertex_adjacency_matrix, and
./polyhedron/    def _init_vertex_adjacency_matrix(self, verbose=False):
./polyhedron/            sage: p._init_vertex_adjacency_matrix()

Should they be removed, or used somewhere?


Change History (1)

comment:1 Changed 4 years ago by mkoeppe

_make_polyhedron_face is used in face_lattice.

Note: See TracTickets for help on using tickets.