Evaluate 10% + 20% as 30% or 0.3 instead of 0.12

[hamodey]

Hi,

I am fairly new to using mathjs and I have been using it for my excel-like datasheets on my online application.

However, recently I have seen an issue with evaluating decimal numbers, even the simplest expression does not output the correct answer, or what I believe should be correct.

Describe the bug

const expression = '10.00% + 20.00%'
const evaluation = math.evaluate(expression);

I am getting a weird output as follows:

I believe the correct output should be 30 (%)

Could I get some clarification pls?

Thanks :)

[josdejong]

The expression parser evaluates this expression as follows:

math.evaluate('50%') 
// = 50 * 0.01
// = 0.5

math.evaluate('50% + 20%') 
// = (50 * 0.01) + 20%
// = 0.5 + 20%
// = 0.5 * 1.2
// = 0.6

If you want to get the result of 30%, you can provide parenthesis, like:

math.evaluate('(10 + 20)%') 
// 0.3

The percentage operator is listed here in the docs: https://mathjs.org/docs/expressions/syntax.html

Does that help?

[gwhitney]

I think the original post does raise a legitimate issue that 10% + 20% is at least ambiguous between adding the percentages and increasing the left operand by 20%. In fact I would say that the most common "everyday" interpretation of this expression would be to add the percentages and get 30%. So it seems as though perhaps it ought to be flagged as ambiguous or perhaps even be interpreted as the percentage addition. It's a real oddity that there isn't any parenthesization that disambiguates between these two interpretations-- note you had to remove one percent operator to get the desired 30% interpretation, with the current parser. I don't have a specific proposal offhand, but perhaps this should be moved to design discussions for further thought?

[josdejong]

Yes you're right, it would be much more intuitive to evaluate 10% + 20% as 30%.

OK I'll move this into a discussion/idea. We'll have to think though how we can support this and in what cases exactly.

[josdejong]

So, some cases we need to think through:

10% + 20%
10% - 20%

x + 10% + 20%
x - 10% - 20%
10% + 20% + x
10% - 20% - x

10% + 20% + 30%
10% + 20% - 30%

Are there more?

[gwhitney]

Well, we need to preserve things like 50 + 20% and 10cm + 20%
And what about:
x + 20%
20% + 5
10% + 5 + 20%

And is it associative? I.e. are
10% + 20% + 30%, (10% + 20%) + 30%, and 10% + (20% + 30%) all the same or not? What about 50 + 20% + 10% vs. 50 + (20% + 10%) ? If you add percentages first in the latter you get 50 + 30% = 65 whereas if you operate left to right in the former you get 60 + 10% = 66

And suppose you do want to increase a percentage by a percentage, i.e get 10% more than a 50% increase, i.e. a 55% increase, what do you write? (I mean, supposing that we change 50% + 10% to 60%, how do you get back the old behavior if you want it?)

[josdejong]

Good points @gwhitney , thanks. We'll need to think all of those cases through.

[gwhitney]

I haven't looked at parse trees yet so am not really ready to put forward any concrete possibilities. But conceivably 50% + 20% and (50%) + 20% could mean different things. Or is that too undesirable?

[josdejong]

Intuitively I would say (50%) + 20% is the same as 50% + 20%. But it may work out that way. Currently the % operator has a higher precedence then +, resulting in the current behavior. How we want to make changes in the precedence of these two operators in specific cases, so it may turn out that you have to use parenthesis like in (50%) + 20% to override default behavior if that doesn't match what you want to do.