Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The calculator is not working properly #16949

Closed
1 task
Armand1000 opened this issue Mar 10, 2022 · 8 comments
Closed
1 task

The calculator is not working properly #16949

Armand1000 opened this issue Mar 10, 2022 · 8 comments
Labels
Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. Run-Plugin Things that relate with PowerToys Run's plugin interface

Comments

@Armand1000
Copy link

Armand1000 commented Mar 10, 2022

Microsoft PowerToys version

0.56.2

Running as admin

  • Yes

Area(s) with issue?

PowerToys Run

Steps to reproduce

2022-03-10_07h11_07
The calculator is not working properly

✔️ Expected Behavior

0.51

❌ Actual Behavior

1.14

Other Software

No response

@Armand1000 Armand1000 added Issue-Bug Something isn't working Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Mar 10, 2022
@Armand1000
Copy link
Author

If you use a comma, the calculator will count correctly
2022-03-10_07h31_13

@Aaron-Junker
Copy link
Collaborator

If you use a comma, the calculator will count correctly
2022-03-10_07h31_13

Exactly. The calcuöator respects your localisation settings.

@Aaron-Junker Aaron-Junker added Resolution-Helped User Could help the user by a problem they had Product-PowerToys Run Improved app launch PT Run (Win+R) Window labels Mar 10, 2022
@franky920920
Copy link
Contributor

Duplicate of #13562

@franky920920 franky920920 marked this as a duplicate of #13562 Mar 10, 2022
@franky920920 franky920920 added the Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. label Mar 10, 2022
@Jay-o-Way Jay-o-Way removed Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. Resolution-Helped User Could help the user by a problem they had labels Mar 10, 2022
@Jay-o-Way Jay-o-Way marked this as not a duplicate of #13562 Mar 10, 2022
@Jay-o-Way
Copy link
Collaborator

@franky920920 that is not the same thing. The problem here is that the leading zeros from the fractional part are ignored. As you can see, the incorrect result does have a decimal in the correct place. If any decimal would be ignored, the result would be in the order of 30-thousand.
This is also a known issue, but definitely different.

@Jay-o-Way
Copy link
Collaborator

Jay-o-Way commented Mar 10, 2022

@crutkas

the issue here is NOT in the decimal character, because that IS recognised. The problem is that leading zeros for the fracture part are trimmed off before the calculation. Please tell me you see the difference?

#2265 might solve this too, but I really want to stress the difference. And since that issue does not mention the problem of the zeros, I am not marking this as duplicate for that issue.

@Jay-o-Way
Copy link
Collaborator

Duplicate of #12375

@Jay-o-Way Jay-o-Way marked this as a duplicate of #12375 Mar 10, 2022
@Jay-o-Way Jay-o-Way added the Run-Plugin Things that relate with PowerToys Run's plugin interface label Mar 10, 2022
@Jay-o-Way
Copy link
Collaborator

@jaimecbernardo Here we have the TRAILING zeros, on top of the error you fixed yesterday. Do you know the solution for this too? And when can we please ditch this flaud engine in favor of a good calculator?

@jaimecbernardo
Copy link
Collaborator

This should be fixed as well with the fix for #12375 . No ETA on changing engine for now.

@jaimecbernardo jaimecbernardo added Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. and removed Needs-Triage For issues raised to be triaged and prioritized by internal Microsoft teams labels Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug Something isn't working Product-PowerToys Run Improved app launch PT Run (Win+R) Window Resolution-Duplicate There's another issue on the tracker that's pretty much the same thing. Run-Plugin Things that relate with PowerToys Run's plugin interface
Projects
None yet
Development

No branches or pull requests

5 participants