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.

Sign up for GitHub

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

Jump to bottom

LG Therma V heat pump-water heater #583

Closed
HaD3sZ opened this issue Aug 17, 2023 · 24 comments
Closed

LG Therma V heat pump-water heater #583

HaD3sZ opened this issue Aug 17, 2023 · 24 comments
Labels
Feature Request

Comments

@HaD3sZ
Copy link

HaD3sZ commented Aug 17, 2023

Hi

I have a LG Therma V (AWHP_019101_WW-SAC_AWHP (AC) Fw: 2.6.0_RTOS_3K) heat pump. The device works well, but the water heater (boiler) not visible (unsupported_devices). I attached the diagnose file.
smartthinq_sensors-700654ba51e9ab1d0cf4014e8ab3f5cd-Levego-viz.hoszivattyu-0a2c882661454b401994bc16dba3c9eb.json.txt

Have you any idea, what I have to do?
@nafets2
Copy link

nafets2 commented Sep 22, 2023 •
edited
Loading

Hi, I have also a problem with the ThemaV-Waterpump Hotwater-Heater:
When I turn the heater on via HA Dashboard GUI - everything works fine.
But when I use the HA automation to turn the heater on, I get the following error: NotImplementedError

File "/usr/src/homeassistant/homeassistant/components/water_heater/__init__.py", line 318, in async_turn_off
    await self.hass.async_add_executor_job(ft.partial(self.turn_off, **kwargs))
  File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/water_heater/__init__.py", line 314, in turn_off
    raise NotImplementedError()
NotImplementedError

Have you any Idea?

@ollo69
Copy link
Owner

ollo69 commented Sep 22, 2023

Seems to be 2 different problems. I will analyze as soon as I will have some available time.

@ollo69
Copy link
Owner

ollo69 commented Sep 22, 2023

@nafets2,

Please attach diagnostic file

@nafets2
Copy link

nafets2 commented Sep 22, 2023 via email •
edited by ollo69
Loading

@pabsi
Copy link

pabsi commented Sep 27, 2023

This is interesting.
I have an LG Therma V with an air source heat pump and it works fine As you said "boiler", does this mean yours does not have an R32 Monobloc heat pump?
See screenshot of mine:
image

The only thing missing is the ability to turn the "Quick DHW heating" (which basically makes the Therma V use the electric heater, instead of the external air source heater pump). This feature, in the LG ThinQ app, it's a toggle basically (not sure it'd fit under the water heater modes?). Perhaps it's meant to be the "PERFORMANCE" mode I saw in the code?

Regards, and thank you so much for this integration! 🙏 👏

@ollo69
Copy link
Owner

ollo69 commented Sep 27, 2023

@nafets2,

I modify your message replacing the json text with attachment to keep the thread more readable, but in any case the diagnostic is not complete (it is truncated at the end).
Can you please attach new file with complete diagnostics?

@nafets2
Copy link

nafets2 commented Sep 27, 2023 via email

@pabsi
Copy link

pabsi commented Sep 27, 2023

Hey @nafets2
The electric heater is built-in in the inside unit. This one:
image

The LG Therma V normally (by default) works on cycles of 30 minutes: 30 mins for domestic hot water heating, and 30 minutes for air/floor cooling/heating. And then there is a setting in which, if after a number of DHW cycles the temperature did not reach the target, then the electric heating triggers. This is 1 cycle by default, but can be tweaked. BUT, one also has the ability to use the "Quick DHW heating" which basically forces the use of the electric heating component (for example, if using too much domestic hot water and the air pump can't keep up). This is the feature I mean is missing.

Re turning on and/or off via automations, is not something I do, since I just don't turn the machine off (it's meant to stay on, basically, by design, as the startups consume way too much energy).

Regards,

@nafets2
Copy link

nafets2 commented Sep 27, 2023 via email

@HaD3sZ
Copy link
Author

HaD3sZ commented Sep 27, 2023

I would like that too. That's why I created this ticket..:)

Hi, I don´t have or use the optional "LG Quick DHW heating". My emergency plan is to use the built-in heating element of the hot water tank. This works even if the heat pump's electronics are defective.;-) According to my energy measurements, the heat pump needs more electricity in the summer if it is switched on for 24 hours than if it is only switched on for 1 hour to produce hot water. My use case is that I only switch on the hot water generation when I get enough electricity from my PV system. In winter, my heat-pump is always switched on (AUTO Mode), but I would also like to be able to regulate the start time of hot water generation depending on how much electricity the PV generates. It would also be great if you could set the temperature of the hot water not only via the HA-Gui, but also via HA automation. Regards, Stephan Am Mi., 27. Sept. 2023 um 11:19 Uhr schrieb Pablo @.***
…
: Hey @nafets2 https://github.com/nafets2 The electric heater is built-in in the inside unit. This one: [image: image] https://user-images.githubusercontent.com/5247611/270928520-f1912e16-15f5-4f2f-92ea-89327c3a9cbb.png The LG Therma V normally (by default) works on cycles of 30 minutes: 30 mins for domestic hot water heating, and 30 minutes for air/floor cooling/heating. And then there is a setting in which, if after a number of DHW cycles the temperature did not reach the target, then the electric heating triggers. This is 1 cycle by default, but can be tweaked. BUT, one also has the ability to use the "Quick DHW heating" which basically forces the use of the electric heating component (for example, if using too much domestic hot water and the air pump can't keep up). This is the feature I mean is missing. Re turning on and/or off via automations, is not something I do, since I just don't turn the machine off (it's meant to stay on, basically, by design, as the startups consume way too much energy). Regards, — Reply to this email directly, view it on GitHub <#583 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/BCFGWGM7LH7UAYXSQRYW7X3X4PVQZANCNFSM6AAAAAA3T5VO4A . You are receiving this because you were mentioned.Message ID: @.***>

@pabsi
Copy link

pabsi commented Sep 27, 2023

My emergency plan is to use the built-in heating element of the hot water tank

That is indeed the very same component I refer about... Unless we are misunderstanding each other. And that is not "your" emergency plan, that's for when the air pump can't heat the water due to high temperatures outside, as the working range for the outdoors unit is -25 ~ 35 degrees celsius (so, anything higher than 35, should trigger the "built-in heating element", which is just an electric resistance, to heat the domestic hot water.

These:
image
image

More details: https://www.youtube.com/watch?v=UWjET2ZqDZM

(Menu on the LG ThinQ mobile app may differ, as it does mine, for example).

Now, back to my question, ☝️ this is what I am missing in the home assistant integration. Unsure if it is available via the API either. But happy to help debug/troubleshoot/investigate further.

Thanks!

@nafets2
Copy link

nafets2 commented Sep 27, 2023 via email •
edited
Loading

@nafets2
Copy link

nafets2 commented Sep 27, 2023 •
edited
Loading

Here you can see the working API Calls over the HA-Gui:
Debug.txt

  • Turn Hotwater Mode on: Set - airState.miscFuncState.hotWater - 1
  • Change Temperature: Set - airState.tempState.hotWaterTarget - 44
  • Turn Hotwater Mode off: Set - airState.miscFuncState.hotWater - 0

@ollo69 I Hope you can make it possible to use this API calls also via HA-Automation - that would be great!

At the moment I can select only this actions in de HA Automation Menue:

grafik

@nafets2
Copy link

nafets2 commented Sep 27, 2023

Here I have a example from a working ThinQ Call via HA-Automation:
Debug2.txt
I hope that helps?

@nafets2
Copy link

nafets2 commented Sep 28, 2023

Hi @ollo69,

regarding to following Error:

"/usr/src/homeassistant/homeassistant/components/water_heater/__init__.py", line 306, in turn_on
    raise NotImplementedError()
NotImplementedError

maybe, the call from the HA-Automation is wrong. As we cas see in the debug logs, The LG-ThinQ-API need "0" and "1" to turn the HotWater function ON/OFF. Maybe the HA-Automation "Enable" function calls "turn_on" instead of "1" and this value is not supportet?

or maybe the wrong hotwater function will be called. there are 3 different types of "hotwater" functions.:

            "airState.miscFuncState.hotWater": {
               "_comment": "##\uc628\uc218\uc6b4\uc804",
               "data_type": "enum",
               "default": "0",
               "value_mapping": {
                 "0": "@OFF",
                 "1": "@ON"
               }
             },
             "airState.miscFuncState.hotWaterMode": {
               "_comment": "##\uc628\uc218\ub2e8\ub3c5\ubaa8\ub4dc",
               "data_type": "enum",
               "default": "0",
               "value_mapping": {
                 "0": "@OFF",
                 "1": "@ON"
               }
             },
             "airState.miscFuncState.powerHotWater": {
               "_comment": "##\ud30c\uc6cc\uae09\ud0d5",
               "data_type": "enum",
               "default": "0",
               "value_mapping": {
                 "0": "@OFF",
                 "1": "@ON"
               }
             },

hotWater: => I tink this is the correct function
hotWaterMode: => no idea - maybe the same as only "hotWater"?
powerHotWater => maybe the function which @pabsi needs (optinal electical water heater)

@ollo69
Copy link
Owner

ollo69 commented Oct 11, 2023

The problem here is that turn_on and turn_off methods for water_heater entity was introduced in HA 2023.8.0 with PR94817.
Because this integration is still based to HA 2023.7, this to methods are still not implemented. I have to move compatibility forward to 2023.8.0 to implement this, I will do on future releases.

By the way on this thread I read a lot of comment, but at the end is not present a complete diagnostic file (with the exception of the one in the first message).

@nafets2 (or any other interested),

Could you please add full diagnostic (attaching file) so that I will have element to implement the methods?

@HaD3sZ ,

I do not understand the unsupported_devices you are talking about. I don't see any unsupported_devices in your diag file.

@HaD3sZ
Copy link
Author

HaD3sZ commented Oct 13, 2023

The problem here is that turn_on and turn_off methods for water_heater entity was introduced in HA 2023.8.0 with PR94817. Because this integration is still based to HA 2023.7, this to methods are still not implemented. I have to move compatibility forward to 2023.8.0 to implement this, I will do on future releases.

By the way on this thread I read a lot of comment, but at the end is not present a complete diagnostic file (with the exception of the one in the first message).

@nafets2 (or any other interested),

Could you please add full diagnostic (attaching file) so that I will have element to implement the methods?

@HaD3sZ ,

I do not understand the unsupported_devices you are talking about. I don't see any unsupported_devices in your diag file.

Yes, It is not found in the diag file. But I have boiler (water heater) which is part of LG ThermaV. The water heater is showing in the LG app like in pabsi's post. But, I don't see it in your add-on, I can't use it.

@nafets2
Copy link

nafets2 commented Oct 13, 2023

Hello @ollo69,

Thank you for analysinf this issue. I will wait, until you have time to upgrade to the new standard (HA 2023.8.0).
You can see the Debug Logfile entrys from the HA ThinQ Integration in this Attachment: (I hope this time you get everitying you need for debug)
home-assistant_smartthinq_sensors_2023-10-13T11-40-22.429Z.log

@ollo69
Copy link
Owner

ollo69 commented Oct 13, 2023

You can see the Debug Logfile entrys from the HA ThinQ Integration in this Attachment

@nafets2,

what I need is integration diagnostic and not debug log. Thanks.

@nafets2
Copy link

nafets2 commented Oct 13, 2023

@ollo69
Ok Sorry, Here is the diagnoc file
smartthinq_sensors-1e98bb5c4384a761f89093a373ee3e31-Luft-Wasser-Wärmepumpe-176a0e6401611f0d2509bb014526cbdf.json(1).txt

ollo69 added a commit that referenced this issue Oct 14, 2023
@ollo69
Add turn_on and turn_off methods to Water Heater (issue #583)
2a4f013
@ollo69
Copy link
Owner

ollo69 commented Oct 14, 2023

@HaD3sZ,

in last release I added support for Water Heater for AC ApiV1 devices (as yours), so now you should be able to manage it via HA. Because is not possible for me to emulate ApiV1 device, I don't know if everything is working properly, please report here your tests result.

@ALL,

now turn_off and turn_on methods for Water Heater are implemented. Of course you need at least HA 2023.8.x to install new release.

@nafets2
Copy link

nafets2 commented Oct 14, 2023

Hello @ollo69,

perfect - it works :-)
You are the best! -> Coffe is comming
br
nafets

@HaD3sZ
Copy link
Author

HaD3sZ commented Oct 20, 2023

Hello @ollo69,

Thanks, I tested the latest release. My water heater works!
But I can't heat the water at the touch of a button. This function is not in the LG app either (but it is in the LG control panel). I did an automatition, it turns off the water heater and then turns on the heating function. This solved my problem.
I think your release works fine. Thank you very, very much!

@ollo69
Copy link
Owner

ollo69 commented Oct 20, 2023

Ok, fine. I close this issue, if there are other problems or requests please open a new one.

@ollo69 ollo69 closed this as completed Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pabsi @HaD3sZ @ollo69 @nafets2