search

google / mathsteps

Step by step math solutions for everyone
https://socratic.org
Apache License 2.0
2.12k stars 274 forks source link

fix cancel terms error #192

Closed aliang8 closed 7 years ago

aliang8 commented 7 years ago

sentry bugs: https://sentry.io/socratic/athena/issues/308248105/ https://sentry.io/socratic/athena/issues/308226210/

Another one: https://sentry.io/socratic/athena/issues/308235832/

The error was because polynomialTerms also includes symbols and powers, which I do not want to be cancelled.

ldworkin commented 7 years ago

@evykassirer I'm gonna go ahead and merge this one, just cause it's a hot fix for a bunch of errors we're seeing in Athena. If you have any comments on it, we can definitely address them tomorrow in another PR!