Skip to content

Fix decimal and checksum#5

Open
ruisilvalink wants to merge 1 commit into
adidax:masterfrom
ruisilvalink:master
Open

Fix decimal and checksum#5
ruisilvalink wants to merge 1 commit into
adidax:masterfrom
ruisilvalink:master

Conversation

@ruisilvalink

Copy link
Copy Markdown

My sensor is reporting decimal temperature values.
Because of that, the checksum wasn't working.

@MikeNatsu

Copy link
Copy Markdown

This fixed my issue with checksum error, why isn't this merge yet?

@k3ldar

k3ldar commented Jan 15, 2024

Copy link
Copy Markdown

Just checked this fix locally and it seems the decimal goes up in weird increments:

22.01
22.02
...
22.09
22.1

Changing lines 79/80 to * 0.1 fixed it for me so it seems more accurate to 1 decimal place.

For what it's worth but nice mod :-)

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.

3 participants