Move remove unnecessary parens to NodeStatus creation about mathsteps

[kevinbarabash]

Even in the case of (2 + 3) + (x + y) there won't be any Parenthesis nodes, b/c it parses to:

( + ( + 2 3 ) ( + x y ) )

The only time we should need a Parenthesis node is if we want to show x + (0) = (1) in:

x + 3 = 4 => x + (3 - 3) = (4 - 3) => x + (0) = (1) => x + 0 = 1 => x = 1

[hmaurer]

When are new parens generated? I don't quite get this.

[evykassirer]

there aren't new parens, but rather sometimes parens stop being helpful

e.g. (2 + 3) * 10 => 5 * 10, not (5) * 10

so we get rid of parens that aren't necessary anymore after each step

does that make sense?

[kevinbarabash]

math-parser parses (2 + 3) * 10 to ( '*' ( '+' 2 3 ) 10 ) which has no parens in the AST. The parens should only show up when necessary when rendering to a string or to TeX. It looks like we're creating parens in a lot of places we probably shouldn't be.

[evykassirer]

oh, yeah the tree without as mnay parens and better printing function will make it way easier to not worry about parenthesis sweeet

though sometimes we'll still want them for pedagogical reasons (e.g. (2 + 3) + (x + y) to show grouping)

[evykassirer]

I'm wondering if we'd ever need to reflatten the tree midway through the step by step solution, or if we can get by with the original structure of the tree and the manipulations along the way

[kevinbarabash]

A Parenthesis node is also necessary if we wanted to show

x + (3 - 3) = (4 - 3)

instead of

x + (3 - 3) = 4 - 3