-
Notifications
You must be signed in to change notification settings - Fork 23
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
Decimal Precision Differs on Climate entity vs related Temp and Humidity Entities. #174
Comments
I'll take a look through, but I'm not sure how much of this is under the integration's control. I recall one spot where we set precision but maybe there are others where we could. |
Ah, for the entities, I manually adjusted the precision in their settings. It’s the climate entity’s precisions that are really the question. |
Hmm, all the sensors defined in https://github.com/dlarrick/hass-kumo/blob/master/custom_components/kumo/sensor.py declare that they are PRECISION_TENTHS. Are you sure you're using those sensors, not a template (as was necessary when this integration was first published)? Also: the default precision for Climate entities is TENTHS for celsius, WHOLE for farenheit. I'm concerned that if we hard-code the precision there to TENTHS then adjusting setpoints will become difficult using up/down buttons. But feel free to experiment and share your findings here. |
Because Kumo seems to use a lookup table to convert from C to F, there's a lot of "rounding" that happens during conversion both ways. However, I've found (anecdotally) that controlling via F using tenths of a degree does allow a more accurate setting after Kumo has converted it internally back to C using its LOT. I've added the following override kumo/climate.py
simple-thermostat lets you set this in the card, but it also looks like Climate entity has a target_temperature_step property you could customize. |
BTW, you SHOULD be able to customize precision from config.yaml…
... but state attributes don't look impacted by the setting:
I think this is an HA bug but can't seem to track it down. |
Is it possible to adjust the decimal precision to match across the entire Device? Right now I see:
I think 1 decimal all around would be appropriate.
Image to illustrate:
The text was updated successfully, but these errors were encountered: