From a1011d30918f4de235e9616cf9e17c916664346a Mon Sep 17 00:00:00 2001 From: Matthias Koeppe Date: Wed, 24 Apr 2024 19:59:51 -0700 Subject: [PATCH] src/sage/topology: Remove chomp interface deprecated in #33777 (2022) --- src/sage/topology/cubical_complex.py | 24 ----------------- src/sage/topology/simplicial_complex.py | 36 ------------------------- 2 files changed, 60 deletions(-) diff --git a/src/sage/topology/cubical_complex.py b/src/sage/topology/cubical_complex.py index 3d53f3b1a4b..5c255a9973f 100644 --- a/src/sage/topology/cubical_complex.py +++ b/src/sage/topology/cubical_complex.py @@ -1690,30 +1690,6 @@ def algebraic_topological_model(self, base_ring=None): base_ring = QQ return algebraic_topological_model(self, base_ring) - def _chomp_repr_(self): - r""" - String representation of self suitable for use by the CHomP - program. This lists each maximal cube on its own line. - - This function is deprecated. - - EXAMPLES:: - - sage: C = cubical_complexes.Cube(0).product(cubical_complexes.Cube(2)) - sage: C.maximal_cells() - {[0,0] x [0,1] x [0,1]} - sage: C._chomp_repr_() - doctest:...: DeprecationWarning: the CHomP interface is deprecated; hence so is this function - See https://github.com/sagemath/sage/issues/33777 for details. - '[0,0] x [0,1] x [0,1]\n' - """ - deprecation(33777, "the CHomP interface is deprecated; hence so is this function") - s = "" - for c in self.maximal_cells(): - s += str(c) - s += "\n" - return s - def _simplicial_(self): r""" Simplicial complex constructed from self. diff --git a/src/sage/topology/simplicial_complex.py b/src/sage/topology/simplicial_complex.py index 625e270436d..62122c1aec7 100644 --- a/src/sage/topology/simplicial_complex.py +++ b/src/sage/topology/simplicial_complex.py @@ -4498,42 +4498,6 @@ def _translation_from_numeric(self): d = self._vertex_to_index return {idx: v for v, idx in d.items()} - def _chomp_repr_(self): - r""" - String representation of ``self`` suitable for use by the CHomP - program. This lists each facet on its own line, and makes - sure vertices are listed as numbers. - - This function is deprecated. - - EXAMPLES:: - - sage: S = SimplicialComplex([(0,1,2), (2,3,5)]) - sage: print(S._chomp_repr_()) - doctest:...: DeprecationWarning: the CHomP interface is deprecated; hence so is this function - See https://github.com/sagemath/sage/issues/33777 for details. - (2, 3, 5) - (0, 1, 2) - - A simplicial complex whose vertices are tuples, not integers:: - - sage: S = SimplicialComplex([[(0,1), (1,2), (3,4)]]) - sage: S._chomp_repr_() - '(0, 1, 2)\n' - """ - deprecation(33777, "the CHomP interface is deprecated; hence so is this function") - s = "" - numeric = self._is_numeric() - if not numeric: - d = self._translation_to_numeric() - for f in self.facets(): - if numeric: - s += str(f) - else: - s += '(' + ', '.join(str(d[a]) for a in f) + ')' - s += '\n' - return s - # this function overrides the standard one for GenericCellComplex, # because it lists the maximal faces, not the total number of faces. def _repr_(self):