hristo-atanasov / Tasmota-IRHVAC

Home Assistant platform for controlling IR Air Conditioners via Tasmota IRHVAC command and compatible hardware
200 stars 67 forks source link

New UI controls incomplete / not working #131

Closed phoenixtechnam closed 3 months ago

phoenixtechnam commented 9 months ago

Since HA update to 2023.12.0 my airconditioners based on IRHVAC cannot be controlled properly anymore. Target temperature is not displaying and cannot be be set via UI. See screenshot below:

image

cociweb commented 9 months ago

Similar to this: https://github.com/home-assistant/frontend/issues/18950

phoenixtechnam commented 9 months ago

yes, but the problem is not the missing modes, those can be added via additional card configuration. The real problem is that I cannot set target temperatures anymore. It also seems to mix up the current temperature and the target temperature.

cociweb commented 9 months ago

As You can see in the legacy ticket, the target temp. is also missing. (The 70F -next to the icon- should be the current temp.)

cociweb commented 9 months ago

well, I was aware to reproduce by a faulty formed json: if you are not initialize the temperature value, and the default value is null, then it can happen that you miss the +/- buttons and you are not able set the temp. <= Since you are not able to set any target temp, then you will get back null. :D - vicious circle... As a workaround try to set it at least once with your IR remote, or send a correctly formed mqtt message for tasmota.

cociweb commented 9 months ago

According to this, there is another way of thinking in my mind... What if.... if you are not able to set the temp by the thermostat card.... from your default configuration temperature should derive. Have you set default temperature in your yaml config? anyways, it should be set other than 'null' by default.... So 3 option can pops up: a) the new thermostat card overwrites the default target_temp value with 'null' b) the default values are not working somehow as here reported c) somehow Tasmota returns 'null' value instead of None and it is not handled well here

Eg:

{"StateMode": "SendStore", 
....
"Power": "off", 
"Mode": "off", "
Celsius": "on", 
"Temp": null, 
"FanSpeed": "max",
"SwingV": "auto", 
"SwingH": "off", 
...}

Instead of:

{"StateMode": "SendStore", 
....
"Power": "off", 
"Mode": "off", "
Celsius": "on", 
"Temp": 25, 
"FanSpeed": "max",
"SwingV": "auto", 
"SwingH": "off", 
...}

Some developer opinion would be appreciated!

phoenixtechnam commented 9 months ago

According to this, there is another way of thinking in my mind... What if.... if you are not able to set the temp by the thermostat card.... from your default configuration temperature should derive. Have you set default temperature in your yaml config? anyways, it should be set other than 'null' by default.... So 3 option can pops up: a) the new thermostat card overwrites the default target_temp value with 'null' b) the default values are not working somehow as here reported c) somehow Tasmota returns 'null' value instead of None and it is not handled well here

Eg:

{"StateMode": "SendStore", 
....
"Power": "off", 
"Mode": "off", "
Celsius": "on", 
"Temp": null, 
"FanSpeed": "max",
"SwingV": "auto", 
"SwingH": "off", 
...}

Instead of:

{"StateMode": "SendStore", 
....
"Power": "off", 
"Mode": "off", "
Celsius": "on", 
"Temp": 25, 
"FanSpeed": "max",
"SwingV": "auto", 
"SwingH": "off", 
...}

Some developer opinion would be appreciated!

min_temp, max_temp and target_temp are all set in my config

reloadxero commented 9 months ago

here is what i did to restore functionality, you only have to do it once per entity:

  1. In HA go to "Developer Tools"
  2. Select "Services" tab
  3. Select service "Climate: Set target temperature"
  4. Select target climate entity and enter a temperature (any temperature is ok)
  5. Hit "Call Service"

As soon as you make this service call the thermostat entity is going to get the target temperature information, the plus and minus buttons as well as full functionality.

BigHomie90 commented 9 months ago

Thank you @reloadxero ! Even after HA restart, works as it should.

phoenixtechnam commented 9 months ago

Thank you @reloadxero, confirmed working

Michel01975 commented 9 months ago

Thank you @reloadxero

parcox commented 7 months ago

here is what i did to restore functionality, you only have to do it once per entity:

  1. In HA go to "Developer Tools"
  2. Select "Services" tab
  3. Select service "Climate: Set target temperature"
  4. Select target climate entity and enter a temperature (any temperature is ok)
  5. Hit "Call Service"

As soon as you make this service call the thermostat entity is going to get the target temperature information, the plus and minus buttons as well as full functionality.

Thank you @reloadxero, it's working. FYI, I have 3 tasmota-irhvac configured devices, one of them had this issue after HA 2024.2 update.

cociweb commented 3 months ago

here is what i did to restore functionality, you only have to do it once per entity:

  1. In HA go to "Developer Tools"
  2. Select "Services" tab
  3. Select service "Climate: Set target temperature"
  4. Select target climate entity and enter a temperature (any temperature is ok)
  5. Hit "Call Service"

As soon as you make this service call the thermostat entity is going to get the target temperature information, the plus and minus buttons as well as full functionality.

@nao-pon Do you have any suggestion how can we resolve it in the code during init? (I would like to avoid to send any/fake temperature to the tasmota after HA restart, because my devices are beeping after any command, and it is annoying)

nao-pon commented 3 months ago

For some reason, it seems that the saved target temperature cannot be restored. I'm not sure if this will affect it, but I'm making some modifications to the master, so can I try it out with the current master first to see how it behaves?

You can install it by selecting master in the HACS redownload selection.

cociweb commented 3 months ago

I'm not sure if this will affect it, but I'm making some modifications to the master

Oh my holly cow! It is initialized properly! So, now the Temperature attribute is not null anymore after HA restart/update which resulted in a temperature ping-ping where the ball was a huge null. :D Thank you! - eagerly waiting for the released version, to close this long-runner Képernyőkép ekkor: 2024-06-22 22-52-49

nao-pon commented 3 months ago

It fixed on 2024.6.2. Thanks! 👍