typo in _strip_string?

[sieu-n]

Description

There appears to be a typo in the _strip_string function of the math_equivalence.py file, where an unnecessary escape character is used with the percent sign. The code uses string.replace("\%", "") when it seems to me that string.replace("%", "") was intended. I'm not 100% certain whether this behavior is unintentional but currently is_equiv("50%", "50") == False. The current code also throws a warning for python 3.11.

math/modeling/math_equivalence.py

Line 104 in 357963a

string = string.replace("\%", "")

Suggestion

string = string.replace("\%", "")
->
string = string.replace("%", "")

[sieu-n]

Just did some research for 5 mins

Everything in the data should be correctly normalized since they all seem to be formatted like 50\%.

The only difference due to this is when the model predicts 50% instead of 50 or 50\% because is_equiv("50\\%", "50%") == False. There are 51 % signs in the test set(~1%), and the results are only impacted if the model acts in a certain way.

edit: typo