Remove commented out methods

Author: Jake-Moss

src/flint/types/fmpq_mpoly.pyx

@@ -548,28 +548,6 @@ cdef class fmpq_mpoly(flint_mpoly):
 
         return res
 
-    # def terms(self):
-    #     """
-    #     Return the terms of this polynomial as a list of fmpq_mpolys.
-
-    #         >>> ctx = fmpq_mpoly_ctx.get(('x', 2), 'lex')
-    #         >>> f = ctx.from_dict({(0, 0): 1, (1, 0): 2, (0, 1): 3, (1, 1): 4})
-    #         >>> f.terms()
-    #         [4*x0*x1, 2*x0, 3*x1, 1]
-
-    #     """
-    #     cdef:
-    #         fmpq_mpoly term
-    #         slong i
-
-    #     res = []
-    #     for i in range(len(self)):
-    #         term = create_fmpq_mpoly(self.ctx)
-    #         fmpq_mpoly_get_term(term.val, self.val, i, self.ctx.val)
-    #         res.append(term)
-
-    #     return res
-
     def subs(self, dict_args) -> fmpq_mpoly:
         """
         Partial evaluate this polynomial with select constants. Keys must be generator names or generator indices,

src/flint/types/fmpz_mod_mpoly.pyx

@@ -556,28 +556,6 @@ cdef class fmpz_mod_mpoly(flint_mpoly):
 
         return res
 
-    # def terms(self):
-    #     """
-    #     Return the terms of this polynomial as a list of fmpz_mod_mpolys.
-
-    #         >>> ctx = fmpz_mod_mpoly_ctx.get(('x', 2), 11, 'lex')
-    #         >>> f = ctx.from_dict({(0, 0): 1, (1, 0): 2, (0, 1): 3, (1, 1): 4})
-    #         >>> f.terms()
-    #         [4*x0*x1, 2*x0, 3*x1, 1]
-
-    #     """
-    #     cdef:
-    #         fmpz_mod_mpoly term
-    #         slong i
-
-    #     res = []
-    #     for i in range(len(self)):
-    #         term = create_fmpz_mod_mpoly(self.ctx)
-    #         fmpz_mod_mpoly_get_term(term.val, self.val, i, self.ctx.val)
-    #         res.append(term)
-
-    #     return res
-
     def subs(self, dict_args) -> fmpz_mod_mpoly:
         """
         Partial evaluate this polynomial with select constants. Keys must be generator names or generator indices,
@@ -948,33 +926,6 @@ cdef class fmpz_mod_mpoly(flint_mpoly):
         fmpz_mod_mpoly_factor_clear(fac, self.ctx.val)
         return c, res
 
-    # TODO: Rethink context conversions, particularly the proposed methods in #132
-    # def project_to_context(self, ctx):
-    #     cdef:
-    #         fmpz_mod_mpoly res
-    #         slong *C
-    #         slong i
-
-    #     if not typecheck(ctx, fmpz_mod_mpoly_ctx):
-    #         raise ValueError("provided context is not a fmpz_mod_mpoly_ctx")
-
-    #     if self.ctx is ctx:
-    #         return self
-
-    #     C = <slong *> libc.stdlib.malloc(self.ctx.val.minfo.nvars * sizeof(slong *))
-    #     if C is NULL:
-    #         raise MemoryError("malloc returned a null pointer")
-    #     res = create_fmpz_mod_mpoly(self.ctx)
-
-    #     vars = {x: i for i, x in enumerate(ctx.py_names)}
-    #     for i, var in enumerate(self.ctx.py_names):
-    #         C[i] = <slong>vars[var]
-
-    #     fmpz_mod_mpoly_compose_fmpz_mod_mpoly_gen(res.val, self.val, C, self.ctx.val, (<fmpz_mod_mpoly_ctx>ctx).val)
-
-    #     libc.stdlib.free(C)
-    #     return res
-
     def derivative(self, var):
         """
         Return the derivative of this polynomial with respect to the provided variable.

src/flint/types/fmpz_mpoly.pyx

@@ -538,28 +538,6 @@ cdef class fmpz_mpoly(flint_mpoly):
 
         return res
 
-    # def terms(self):
-    #     """
-    #     Return the terms of this polynomial as a list of fmpz_mpolys.
-
-    #         >>> ctx = fmpz_mpoly_ctx.get(('x', 2), 'lex')
-    #         >>> f = ctx.from_dict({(0, 0): 1, (1, 0): 2, (0, 1): 3, (1, 1): 4})
-    #         >>> f.terms()
-    #         [4*x0*x1, 2*x0, 3*x1, 1]
-
-    #     """
-    #     cdef:
-    #         fmpz_mpoly term
-    #         slong i
-
-    #     res = []
-    #     for i in range(len(self)):
-    #         term = create_fmpz_mpoly(self.ctx)
-    #         fmpz_mpoly_get_term(term.val, self.val, i, self.ctx.val)
-    #         res.append(term)
-
-    #     return res
-
     def subs(self, dict_args) -> fmpz_mpoly:
         """
         Partial evaluate this polynomial with select constants. Keys must be generator names or generator indices,

src/flint/types/nmod_mpoly.pyx

@@ -539,28 +539,6 @@ cdef class nmod_mpoly(flint_mpoly):
         """
         return [nmod_mpoly_get_term_coeff_ui(self.val, i, self.ctx.val) for i in range(len(self))]
 
-    # def terms(self):
-    #     """
-    #     Return the terms of this polynomial as a list of nmod_mpolys.
-
-    #         >>> ctx = nmod_mpoly_ctx.get(('x', 2), 11, 'lex')
-    #         >>> f = ctx.from_dict({(0, 0): 1, (1, 0): 2, (0, 1): 3, (1, 1): 4})
-    #         >>> f.terms()
-    #         [4*x0*x1, 2*x0, 3*x1, 1]
-
-    #     """
-    #     cdef:
-    #         nmod_mpoly term
-    #         slong i
-
-    #     res = []
-    #     for i in range(len(self)):
-    #         term = create_nmod_mpoly(self.ctx)
-    #         nmod_mpoly_get_term(term.val, self.val, i, self.ctx.val)
-    #         res.append(term)
-
-    #     return res
-
     def subs(self, dict_args) -> nmod_mpoly:
         """
         Partial evaluate this polynomial with select constants. Keys must be generator names or generator indices,
@@ -926,33 +904,6 @@ cdef class nmod_mpoly(flint_mpoly):
         nmod_mpoly_factor_clear(fac, self.ctx.val)
         return constant, res
 
-    # TODO: Rethink context conversions, particularly the proposed methods in #132
-    # def project_to_context(self, ctx):
-    #     cdef:
-    #         nmod_mpoly res
-    #         slong *C
-    #         slong i
-
-    #     if not typecheck(ctx, nmod_mpoly_ctx):
-    #         raise ValueError("provided context is not a nmod_mpoly_ctx")
-
-    #     if self.ctx is ctx:
-    #         return self
-
-    #     C = <slong *> libc.stdlib.malloc(self.ctx.val.minfo.nvars * sizeof(slong *))
-    #     if C is NULL:
-    #         raise MemoryError("malloc returned a null pointer")
-    #     res = create_nmod_mpoly(self.ctx)
-
-    #     vars = {x: i for i, x in enumerate(ctx.py_names)}
-    #     for i, var in enumerate(self.ctx.py_names):
-    #         C[i] = <slong>vars[var]
-
-    #     nmod_mpoly_compose_nmod_mpoly_gen(res.val, self.val, C, self.ctx.val, (<nmod_mpoly_ctx>ctx).val)
-
-    #     libc.stdlib.free(C)
-    #     return res
-
     def derivative(self, var):
         """
         Return the derivative of this polynomial with respect to the provided variable.