search

dalinicus / homeassistant-acinfinity

AC Infinity integration for Home Assistant for UIS based controllers
MIT License
75 stars 4 forks source link

VPD defaults to incorrent Unit of Measurement (PSI and not kPa) #49

Closed almighty059 closed 6 months ago

almighty059 commented 6 months ago

The linked PR has nothing to do with this issue, as it was intended for a completely different integration by the same name that I do not maintain. That being said, HomeAssistant core changed something with how unit_of_measurement and native_unit_of_measurement work... I had a similar issue with another extension I maintain. Would you mind creating a separate issue thread for this, and I'll check it out?

example from the codebase. I assume I'll need to switch this to unit_of_measurement instead. image

Originally posted by @dalinicus in https://github.com/dalinicus/homeassistant-acinfinity/issues/37#issuecomment-1946842714

Issue involving VPD defaulting to incorrect Unit of Measurement for VPD. It seems to default to PSI when it should be kPa.

dalinicus commented 6 months ago

Thanks for splitting this out 👍🏻. Might have some time this weekend to knock this out.

dalinicus commented 6 months ago

This should be fixed in 1.4.1. Thanks for the report!