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

Precise decimal #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

rally25rs
Copy link

Added a preciseDecimal parser that returns the un-rounded value from the temp probe.
For my application, I intend to convert to Fahrenheit. I noticed that the decimal parser would round a value of 19.250 to 19.3. If converting to F, 19.250 => 66.65, and 19.3 => 66.74. This is almost a 0.1 degree difference due to rounding. A more precise conversion would be yielded from converting the full 19.250 precision number to F first, then perform rounding to the nearest tenth of a degree.

Default `decimal` parser rounds to tenth of a degree Celsius. For users converting to Fahrenheit, the full 1000'ths precision can be used and the rounding done after conversion. For example 19.250 C would be rounded to 19.3 C.  If converted to F there is a 0.1 degree difference between these values, which may be significant depending on the application.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant