search

ollo69 / ha-smartthinq-sensors

HomeAssistant custom integration for SmartThinQ LG devices configurable with Lovelace User Interface.
Apache License 2.0
1.16k stars 159 forks source link

Support for WashTower devices #649

Closed jonad0521 closed 9 months ago

jonad0521 commented 11 months ago

Is your feature request related to a problem? Please describe. It looks like the API is detecting my devices, but that it is an unknown device type of 223. It is a combined washer dryer tower that appears in the thinq app as one device config_entry-smartthinq_sensors-749924fa0c81aabb63a3f1d5792dee37.json.txt

jonad0521 commented 10 months ago

I added my device type to allow the program to get a little farther and attempt to poll the device. It was unsuccessful, but gave some useful info.

256cde1d-dd1e-4fe2-ac10-7ae09faeaff5.json Here is my model_data .json. The model_info.py is failing to validate the MonitoringValue key even though its present. I'm not sure what the issue is. Even if I force that check true, it still fails to find the MonitoringValue key while polling.

ollo69 commented 9 months ago

This new device require some adjustment, working on this.

jonad0521 commented 9 months ago

Great! Let me know if you need anything from me.

ollo69 commented 9 months ago

Support for this device is available with last release, please test and report here result. Please do not expect control to work: I take a look to the device info and remote start command are totally different from previous device. To implement this should be necessary capture some logs from LG apps.

jonad0521 commented 9 months ago

Wow! Looks great! At a glance the only missing thing for me would be the door open status on the dryer (its available on the washer). Here's a screenshot from HA: image image

I'll work on getting you logs from the LG app tonight.

jonad0521 commented 9 months ago

OK here's the flow from mitmproxy when I turn the dryer power on. Is this what you're looking for? flows.txt

ollo69 commented 9 months ago

In this flow I just see PowerOn command. What about other command (eg PowerOff). Can you also capture other commands that you consider useful? Thanks👍

jonad0521 commented 9 months ago

Here's another where I change washer options then start a washer cycle, do the same for a dryer cycle then turn both devices off. flows (2).txt

jonad0521 commented 9 months ago

flows (3).txt Here's another where I change lots of options then send it to the dryer, which can't start because remote start on the device was turned off locally.

ollo69 commented 9 months ago

New controls are implemented with last release. Please test it and report here issue.

jonad0521 commented 9 months ago

Works great. The control entities go unavailable when they aren't available to be used, either because remote start isn't available or its already running. Power works good too.

The only thing missing for me is accurate door open state. On the washer, it never changes from "open". On the dryer its not included anywhere.

Thanks for all the hard work.

ollo69 commented 9 months ago

The only thing missing for me is accurate door open state. On the washer, it never changes from "open". On the dryer its not included anywhere.

The door open is not available in the dryer payload. It's available in the washer payload but probably not supported. Is this information available by ThinQ app? You could try to generate diagnostics with door closed and open, so I can check if this information change inside the payload. Wait at least 1 minutes on status change to be sure that new status is properly polled.

jonad0521 commented 9 months ago

i don't see it. A shame, since I can get a notification about not taking the clothes out of the dryer/washer so I guess it figures it out somehow.

Otherwise, I thank you wholeheartedly for integrating this, I think this FR can be closed.