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

negative sensor temperature #4

Closed
Byter09 opened this issue Dec 15, 2022 · 3 comments
Closed

negative sensor temperature #4

Byter09 opened this issue Dec 15, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@Byter09
Copy link

Byter09 commented Dec 15, 2022

Hello!

First of, I would like to thank you for this project. I use it to send off data updates to InfluxDB and Home Assistant :)

Now, to the problem. Its getting quite cold, and the temperature at the battery probe went below 0°C.

It's unclear whether this is the Rover 20a having a problem, or how the data is retrieved. Just to make sure I'm opening this ticket. I checked both InfluxDB and Home Assistant and they both report the same value. I then went ahead and checked the webhook I use to receive the data I'm sending with your neat project, and my code doesn't seem to change any negative values (I sent mock data to check and it works perfectly), so that leaves me to believe it might be a bug in your code. If I knew more about python I'd have a look but it's not really my forte haha.

The resulting graph from last night looks like this haha
grafik

It went up to 129°C, which looks like an underflow to me.

If you need any more information, I'm happy to help out with whatever you need.

EDIT: Here's the raw data: It goes from 0 directly to 129°C.
grafik

@cyrils
Copy link
Owner

cyrils commented Dec 15, 2022

Could be problem with hex parsing, I'll look into it later tonight.

@cyrils cyrils added the bug Something isn't working label Dec 15, 2022
@cyrils
Copy link
Owner

cyrils commented Dec 16, 2022

@Byter09 I've updated the code, can you check it out? According to documentation 7th bit is for sign, which I've been ignoring. Only change is in Utils.py.

@Byter09
Copy link
Author

Byter09 commented Dec 16, 2022

I have updated my docker container and I guess we'll see if it works. But thank you for taking the time. If you want me to I can put some ice cubes on it haha :D

@cyrils cyrils closed this as completed Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants