remove_stray_braces quickfix

[mz71]

List of Changes

Fixing #225. (In short, TexMobject("\\{") is giving a LaTeX error.)

Motivation

To make compound TexMobjects consisting of literal braces, e.g. set-builder notation: TexMobject("\\{,"x","\mid","x>2","\\}")

Explanation for Changes

It's just a one-line fix, that substracts \\{ and \\} from the count. (It also adds back \\\\{ and \\\\} because those are not literal braces)

Testing Status

As this is just a one-line fix, I only provide the minimal example that didn't work

class LiteralBraceTest(Scene):
    def construct(self):
        a = TexMobject("\\{")
        self.add(a)

:

Acknowledgement

• I have read the Contributing Guidelines

[leotrs]

Great! Can we think of a quick test to add before merging?

[PgBiel]

agree with leo

[leotrs]

@mz71 thoughts on adding tests so this never happens again?

[leotrs]

Merging and adding to #193

[mz71]

@mz71 thoughts on adding tests so this never happens again?

Sorry for late reply, probably bundling up some complicated Latex formulas in an image is a good idea, but I don't have any experience in writing tests for manim, so I don't know.

[leotrs]

Thank you for your reply!