Skip to content
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

Removal of Status TBH and DHW will possible break Things - C3 type device - Heat Pump Wi-Fi Controller #353

Open
erikgieseler opened this issue Oct 16, 2023 · 11 comments
Labels
bug Something isn't working

Comments

@erikgieseler
Copy link

HA core version

2023.10.3

Intergration version

0.3.22

Device type and model

C3 - Heat Pump Wi-Fi Controller 171H120F

Used App

SmartHome

The description of problem

Why were the sensors "binary_sensor.{DEVICEID}_status_tbh" and "binary_sensor.{DEVICEID}_status_dhw"
has been removed in 0.3.22 ?

These were working as expected until now.

Due to the removal of the sensor, it is no longer possible, for example, to query whether the heat pump is running in hot water or heating mode. If I read the source code correctly, you can only see whether the hot water mode is activated or not.

1

2

The logs

No response

@erikgieseler erikgieseler added the bug Something isn't working label Oct 16, 2023
@erikgieseler erikgieseler changed the title Removal of Status TBH and DHW will break possible break Things - C3 type device - Heat Pump Wi-Fi Controller Removal of Status TBH and DHW will possible break Things - C3 type device - Heat Pump Wi-Fi Controller Oct 16, 2023
@erikgieseler
Copy link
Author

This is the view from version 0.3.21.
Heating and DHW (Warmwasser) modes are both enabled.
On the screenshot you can see that currently only DHW is running, not Heating.

3

@vilkur
Copy link

vilkur commented Oct 16, 2023

I support that TBH status and DHW status were necessary features. Please restore these functions if possible. Currently, my DHW automation is broken.

@georgezhao2010
Copy link
Owner

The version v0.3.22 includes two switches, tbh and dhw_power, which have the same functionality as status_tbh and status_dhw. Therefore, I have removed the latter two. You can use the status of these two switches in automation, can’t you?

@vilkur
Copy link

vilkur commented Oct 17, 2023

Thanks for the advice, I’ll try to use the THB switch instead of the THB status in automation. But it turned out that the problem is that none of the switches, neither disinfection nor THB, can be turned off. If the THB relay is enabled via HA, it can only be disabled via the app and not via the HA switch.

@georgezhao2010
Copy link
Owner

Thanks for the advice, I’ll try to use the THB switch instead of the THB status in automation. But it turned out that the problem is that none of the switches, neither disinfection nor THB, can be turned off. If the THB relay is enabled via HA, it can only be disabled via the app and not via the HA switch.

Well, I’ll take a look at this issue. Could you show me the logs where you turned tbh on/off in ha?

@erikgieseler
Copy link
Author

I also have the same problem with the TBH.
home-assistant_midea_ac_lan_2023-10-17T15-26-10.281Z.log

@erikgieseler
Copy link
Author

The version v0.3.22 includes two switches, tbh and dhw_power, which have the same functionality as status_tbh and status_dhw. Therefore, I have removed the latter two. You can use the status of these two switches in automation, can’t you?

You cannot combine the switch with the status of the DHW. Both must be separate. You can activate the DHW mode on the heat pump with the switch but this does not automatically mean that the heat pump will run directly and produce hot water. For that you need the DHW status. The same applies to the TBH switch and status. In version 0.3.21 this was implemented correctly.

@georgezhao2010
Copy link
Owner

The version v0.3.22 includes two switches, tbh and dhw_power, which have the same functionality as status_tbh and status_dhw. Therefore, I have removed the latter two. You can use the status of these two switches in automation, can’t you?

You cannot combine the switch with the status of the DHW. Both must be separate. You can activate the DHW mode on the heat pump with the switch but this does not automatically mean that the heat pump will run directly and produce hot water. For that you need the DHW status. The same applies to the TBH switch and status. In version 0.3.21 this was implemented correctly.

OK, I got it.

@themayna
Copy link

please bring back those sensors :(

@erikgieseler
Copy link
Author

The sensors are included in the master branch again.
Simply download the version "master" again in the HACS menu under "Midea AC LAN".
After the final release you can switch back to the normal version.

@michaapyr
Copy link

I definitelly agree with this, the sensors dhw/heating/tbh status are critical for the automations with this device type, they never can't be combined in switches.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants