RobertD502 / home-assistant-flair

Custom component for Home Assistant Core for Flair pucks, vents, rooms, structures, and minisplits
MIT License
87 stars 12 forks source link

Vent temperature #51

Closed tsuckow closed 1 year ago

tsuckow commented 1 year ago

The vent temperature appears to be treated as unsigned so when the temperature goes below freezing (don't ask) home assistant claims the temperature is thousands of degrees.

RobertD502 commented 1 year ago

I'm not sure this is something that I can address as the temperature conversion/presentation is being handled by home assistant.

As you can see here, the native value sent from Flair is in Celsius which I reference in the native unit of measurement. Home Assistant then handles converting/not converting based on the user's units defined in Home Assistant.

Out of curiosity, would you be able to send a request to Flair's API when you notice this is happening and see what value Flair is reporting for the duct temp? Perhaps the issue originates from their end.

tsuckow commented 1 year ago

Screenshot_20230223-115817.png

It isn't the API but this is their app which appears to imply a signed 16bit number is used to store the temperature in 100ths of a degree c.

RobertD502 commented 1 year ago

Their app uses their API. This alone shows me that the issue is originating on their end. I'd send customer support a message with that screenshot.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.