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

Misleading guarantee of accuracy #5

Open
SoupyzInc opened this issue Aug 13, 2021 · 1 comment
Open

Misleading guarantee of accuracy #5

SoupyzInc opened this issue Aug 13, 2021 · 1 comment
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@SoupyzInc
Copy link
Member

Calculatte offers the option to round answers to a certain amount of decimal places, but does not guarantee precision for that amount of decimal places. Much research will be needed to be able to guarantee error within Calculatte, and much time will be needed to conduct said research.

Temporary Solution
No longer offer the ability to round to a certain amount of decimal places without guaranteed error. This solution will most likely be taken until the final solution can be implemented as to not stop the true focus of Calculatte. Calculatte was a passion project for calculus and programming--and more so for calculus. Due to this vision, I still wish to continue adding my knowledge of calculus into code and will add my knowledge of computer science when it is ready.

Solution
Be able to guarantee error and replace decimal rounding with the ability to change the precision of a calculation. I currently do not know what this solution will look like or what it entails.

/cc @jojongx

@SoupyzInc SoupyzInc added bug Something isn't working enhancement New feature or request labels Aug 13, 2021
@SoupyzInc SoupyzInc self-assigned this Aug 13, 2021
SoupyzInc added a commit that referenced this issue Aug 14, 2021
Removing all internal rounding as Calculatte cannot guarantee error as described in issue #5.
@SoupyzInc
Copy link
Member Author

Whilst removing rounding in PR #6, I noticed that unit tests will now have to account for long decimal results. Due to floating point inaccuracies, often Calculatte's calculation would be inaccurate, however, once rounded to 3-5 decimal places, the value would be 100% or near 100% accurate.

Perhaps, a default rounding value that cannot be changed should be decided upon. More testing on the accuracy of Calculatte's values when rounded need to be done.

/cc @jojongx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant