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

[Bug]: The bug about the ROUND function in the formula, why 21.9/0.2, keeping 0 decimal places, will get 109? In theory, the ROUND function should be rounded to 110. #10935

Open
yanyinqing opened this issue Apr 22, 2024 · 6 comments

Comments

@yanyinqing
Copy link

Describe the bug

The bug about the ROUND function in the formula, why 21.9/0.2, keeping 0 decimal places, will get 109? In theory, the ROUND function should be rounded to 110.

Video/Screenshots

image

Provide a link to the demo with the bug reproduction

No response

Handsontable version

14.3.0

Framework version

vue2

Your environment

macos,edge

@yanyinqing yanyinqing added the bug label Apr 22, 2024
@adrianszymanski89
Copy link
Contributor

adrianszymanski89 commented Apr 22, 2024

Hi @yanyinqing

Thank you for reporting this. I'll consult it with our developer and get back to you once I have feedback on this.

Example: https://jsfiddle.net/handsoncode/dub19jL6/

@yanyinqing
Copy link
Author

Hi @yanyinqing

Thank you for reporting this. I'll consult it with our developer and get back to you once I have feedback on this.

Example: https://jsfiddle.net/handsoncode/dub19jL6/

Hello, how is it going so far? Is there any solution or when to update the new version to solve this problem, because this potential problem seriously affects the accuracy of the data.

@AMBudnik
Copy link
Contributor

Hi @yanyinqing - my colleague is off for the week (we have some national holidays coming soon). I see that the subject was mentioned in the dev meeting, but there are no details. I think that we should get some updates here next week.

@yanyinqing
Copy link
Author

Hello, how is it going so far? Is there any solution or when to update the new version to solve this problem, because this potential problem seriously affects the accuracy of the data.

Hello, how is it going so far? Is there any solution or when to update the new version to solve this problem, because this potential problem seriously affects the accuracy of the data.

@yanyinqing
Copy link
Author

Hi @yanyinqing

Thank you for reporting this. I'll consult it with our developer and get back to you once I have feedback on this.

Example: https://jsfiddle.net/handsoncode/dub19jL6/

Hello, how is it going so far? Is there any solution or when to update the new version to solve this problem, because this potential problem seriously affects the accuracy of the data.

@adrianszymanski89
Copy link
Contributor

Hi @yanyinqing

I'm sorry for the late response. We determined that the precisionRounding limitation causes this issue, and while setting it to, for example, 15 should solve the hardcoded limit is actually 14. I will make sure to update you once this issue is added to the roadmap and fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants