rpn(1): fixed exponentiation #57

Closed

emma wants to merge 0 commits from rpn-fix-exp into main

emma commented

2024-02-13 23:58:31 +00:00

Owner

No description provided.

emma added the

bug

label 2024-02-13 23:58:31 +00:00

emma added 1 commit 2024-02-13 23:58:32 +00:00

5b364e104e

rpn(1): fixed exponentiation

emma requested review from trinity 2024-02-13 23:59:53 +00:00

trinity commented

2024-02-15 00:15:14 +00:00

Owner

I figured exponentiating (is that the word?) x to the power of y (Python: x ** y) would be more intuitive for existing RPN users (because this matches the operation on HP and Elektronika calculators). It looks like the powf method is used like base.powf(exponent). Wouldn't y.powf(x) be bringing y to the power of x?

I figured exponentiating (is that the word?) `x` to the power of `y` (Python: `x ** y`) would be more intuitive for existing RPN users (because this matches the operation on HP and Elektronika calculators). It looks like [the `powf` method](https://doc.rust-lang.org/src/std/f32.rs.html#364) is used like `base.powf(exponent)`. Wouldn't `y.powf(x)` be bringing `y` to the power of `x`?

trinity commented

2024-02-15 05:34:27 +00:00

Owner

Never mind, I looked it up and the HP 41C (basically the gold standard in terms of RPN calculators) is indeed y**x. It looks like Elektronika (which I was referencing, particularly the MK-61 keypad) is an outlier in this regard.

trinity approved these changes 2024-02-15 05:34:34 +00:00

emma closed this pull request

2024-02-15 06:07:29 +00:00