Unable to update from sensor: could not convert string to float: 'unknown'

zacs / ha-dualmodegeneric

Generic thermostat capable of heating and cooling

68 stars 26 forks source link

Unable to update from sensor: could not convert string to float: 'unknown' #5

Closed ciumagcatalin closed 4 years ago

ciumagcatalin commented 4 years ago

On Home Assistant Core 0.104 I am getting: Unable to update from sensor: could not convert string to float: 'unknown'. The setup was working fine on the older HA versions and my setup is:

platform: dualmode_generic name: Small Bedroom Thermostat heater: switch.small_bedroom_heating cooler: switch.living_air_conditioner target_sensor: sensor.small_bedroom_temperature min_temp: 18 max_temp: 30 away_temp: 20 initial_hvac_mode: "heat" cold_tolerance: 0.3 hot_tolerance: 0.1
platform: dualmode_generic name: Kitchen Thermostat heater: switch.kitchen_heating cooler: switch.living_air_conditioner target_sensor: sensor.living_temperature min_temp: 18 max_temp: 30 away_temp: 20 initial_hvac_mode: "heat" cold_tolerance: 0.3 hot_tolerance: 0.1

ciumagcatalin commented 4 years ago

I think this happen when my homeassistant load first time and the sensors are not populated. I don't get it anymore after some time.

zacs commented 4 years ago

Yah, this happens to me as well when the dualmode component gets loaded. I could potentially add some sort of delay to it, but I assume this happens with the official generic component as well. I’ll take a look at the latest code for that component and see if there is delay logic I can borrow.

On Thu, Jun 4, 2020 at 7:05 AM ciumagcatalin notifications@github.com wrote:

I think this happen when my homeassistant load first time and the sensors are not populated. I don't get it anymore after some time.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zacs/ha-dualmodegeneric/issues/5#issuecomment-638870820, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAGYOKM67AYHGAFPYPBBMDRU6SZXANCNFSM4NSRPKOA .

bbbates commented 4 years ago

I suspect this fix (https://github.com/zacs/ha-dualmodegeneric/pull/6/files#diff-f6c9a6f9f4807af09d43c079b5749e5aR214) will solve that issue for you.

ciumagcatalin commented 4 years ago

I suspect this fix (https://github.com/zacs/ha-dualmodegeneric/pull/6/files#diff-f6c9a6f9f4807af09d43c079b5749e5aR214) will solve that issue for you.

But I got the last version 3 days ago. Maybe I missed smth. Is there anything I can help with?

bbbates commented 4 years ago

I suspect this fix (https://github.com/zacs/ha-dualmodegeneric/pull/6/files#diff-f6c9a6f9f4807af09d43c079b5749e5aR214) will solve that issue for you.

But I got the last version 3 days ago. Maybe I missed smth. Is there anything I can help with?

I only pushed that PR today and it hasn’t been merged yet 🙂

bbbates commented 4 years ago

@zacs merged that potential fix in. @ciumagcatalin Can you give it a test with the latest?

ciumagcatalin commented 4 years ago

@bbbates , it works. I just put it today in and I don't get those messages anymore. Thanks!!

zacs commented 4 years ago

Thanks @bbbates for the fix! Fixed in 88a4611f517ef36985bcbbadd80904de30009eb5.