Skip to content

Move sage.misc.misc.coeff_repr, repr_lincomb to new module sage.misc.repr #29892

@mkoeppe

Description

@mkoeppe

In fact, we have 2 copies of coeff_repr and repr_lincomb: one in sage.misc.misc, one in sage.misc.latex. Perhaps they can be merged in a follow up ticket as well; and perhaps even #14509 be fixed...

sage.structure.formal_sum.FormalSum._latex_ has a comment (from nthiery 2012):

        return sage.misc.latex.repr_lincomb(symbols, coeffs)
        # TODO: finish merging sage.misc.latex.repr_lincomb and
        # sage.misc.misc.repr_lincomb and use instead:
        # return sage.misc.misc.repr_lincomb([[t,c] for c,t in self], is_latex=True)

Also sage.modular.modsym.element:

        # TODO: use repr_lincomb with is_latex=True
        return latex.repr_lincomb(v, c)

This ticket is part of an effort to reduce sage.misc.misc, motivated by #29865 (Modularization of sagelib: Break out a separate package sage-objects)

Depends on #29869

CC: @fchapoton @kcrisman @tscrim @slel @nthiery @DaveWitteMorris

Component: refactoring

Author: Matthias Koeppe

Branch/Commit: b5f63a5

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/29892

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions