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

AC display light switch not working #475

Closed
gabi1324 opened this issue Jan 9, 2023 · 68 comments
Closed

AC display light switch not working #475

gabi1324 opened this issue Jan 9, 2023 · 68 comments

Comments

@gabi1324
Copy link

gabi1324 commented Jan 9, 2023

Describe the bug
Hello! I have a problem with the display light on an S09ET AC unit. The switch is always off and if I toggle it the unit beeps but doesn't do anything. I can control the light from the remote but not from the LG app.

Expected behavior
control the display light from HA

Screenshots
If applicable, add screenshots to help explain your problem.

Environment details:

  • Environment (HASSIO, Raspbian, etc):
  • Home Assistant version installed:
  • Component version installed:
  • Last known working version:
  • LG device type and model with issue: AC (S09ET)
  • LG devices connected (list):

Output of HA logs
Paste the relevant output of the HA log here.

Additional context
Add any other context about the problem here.

@ollo69
Copy link
Owner

ollo69 commented Jan 10, 2023

Please attach diagnostic

@gabi1324
Copy link
Author

@SN0WTR00PER
Copy link

Same thing for me... Its not working.
Here my diag file
config_entry-smartthinq_sensors-fc6f45dbc8d08d7c44080f6b4f49c7f5.json.txt

@github-actions
Copy link

github-actions bot commented Mar 3, 2023

This issue is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@gabi1324
Copy link
Author

Hi! Are there any updates on this issue?

@github-actions
Copy link

This issue is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label May 15, 2023
@gabi1324
Copy link
Author

Still an issue.

@github-actions github-actions bot removed the Stale label May 18, 2023
@vincentwolsink
Copy link

I also have issues with S09ET and S12ET (RAC_056905_WW-RAC) airco's. The display light button is present for both ACs. When toggling the button the ACs do make a beep sound, but nothing happens.

See also #515

@vifrost
Copy link

vifrost commented Jun 22, 2023

  1. download statistic information ( device informaion )

image

  1. find "airState.lightingState.displayControl"
  2. check number of @AC_LED_OFF_W and @AC_LED_ON_W
  3. edit ac.py
    ( /root/config/custom_components/smartting_sensors/wideq/devices/ac.py )
  4. working right ON/OFF

image

@gabi1324
Copy link
Author

Hi! I tried modifying ac.py but it doesn't work. The AC unit beeps and doesn't do anything. In the diagnostic file, I have "airState.lightingState.displayControl": 0.0 . I think this modification works for other AC models, but it isn't working for S09ET and other different variations.

@SiboutVanLoo
Copy link

Hi there, tried it to, but does'nt help, but seems to point in the right direction.
Changed the setting to 11 and 12, like you suggested, and now the 'light-state' is displayed correctly, but the command does'nt seem to get thru.
If i turn it off by remote controle, it shows "off" after a few minutes, 5 i guess, like the code suggests.
So that's a good thing, so any ideas why the "command" aspect seems to stay behind?
Was looking in sections further ahead in the code, mentioning "display", but my python knowledge seems to be lacking, and i would'nt want to wreck anything.
Thanks!

@jackbrown1993
Copy link

Also hitting this issue with the below model, unit beeps but display remains on.

RAC_056905_WW-RAC (AC)
Firmware: clip_hna_v1.9.188

@SiboutVanLoo
Copy link

Ok,
This is the return information of the AC unit.

"airState.lightingState.displayControl": 1.0,
"airState.wDir.hStep": 3.0,
"mid": 6982764.0,
"airState.energy.onCurrent": 60.0,
"airState.wMode.airClean": 1.0,
"airState.quality.sensorMon": 0.0,
"airState.tempState.target": 25.0,

Above is Display light off (From aircon data.)

"airState.lightingState.displayControl": 0.0,
"airState.wDir.hStep": 3.0,
"mid": 7716930.0,
"airState.energy.onCurrent": 152.0,
"airState.wMode.airClean": 1.0,
"airState.quality.sensorMon": 0.0,
"airState.tempState.target": 24.0,

This was display ON

Below is "airclean data "on" from unit

"airState.lightingState.displayControl": 0.0,
"airState.wDir.hStep": 3.0,
"mid": 179596276.0,
"airState.energy.onCurrent": 146.0,
"airState.wMode.airClean": 0.0,
"airState.quality.sensorMon": 0.0,
"airState.tempState.target": 23.0,

Below is airclean data "off" from unit.

"airState.lightingState.displayControl": 0.0,
"airState.wDir.hStep": 3.0,
"mid": 179391791.0,
"airState.energy.onCurrent": 146.0,
"airState.wMode.airClean": 1.0,
"airState.quality.sensorMon": 0.0,
"airState.tempState.target": 23.0,

Here is the airclean mapping data from unit.

          "airState.wMode.airClean": {
            "data_type": "enum",
            "default": "0",
            "value_mapping": {
              "0": "@AC_MAIN_AIRCLEAN_OFF_W",
              "1": "@AC_MAIN_AIRCLEAN_ON_W"

Below is the mapping data of the display.

"airState.lightingState.displayControl": {
"data_type": "enum",
"default": "0",
"value_mapping": {
"0": "@RAC_LED_ON",
"1": "@RAC_LED_OFF",
"2": "@non",
"3": "@non",
"4": "@non",
"5": "@non",
"6": "@non",
"7": "@non",
"8": "@non",
"9": "@non",
"10": "@non",
"11": "@AC_LED_OFF_W",
"12": "@AC_LED_ON_W",
"13": "@AC_LED_AUTO_W"

Here is the python command setting code.

ADD_FEAT_POLL_INTERVAL = 300 # 5 minutes

LIGHTING_DISPLAY_OFF = "0"
LIGHTING_DISPLAY_ON = "1"

MODE_OFF = "@off"
MODE_ON = "@on"

MODE_AIRCLEAN_OFF = "@AC_MAIN_AIRCLEAN_OFF_W"
MODE_AIRCLEAN_ON = "@AC_MAIN_AIRCLEAN_ON_W"

"airclean" works fine, but display does not.

Changing 1 & 0 to
LIGHTING_DISPLAY_OFF = "@RAC_LED_OFF"
LIGHTING_DISPLAY_ON = "@RAC_LED_ON"
Or
LIGHTING_DISPLAY_OFF = "@AC_LED_OFF_W"
LIGHTING_DISPLAY_ON = "@AC_LED_ON_W"
Or
LIGHTING_DISPLAY_OFF = "1.0"
LIGHTING_DISPLAY_ON = "0.0"
Or
LIGHTING_DISPLAY_OFF = 1
LIGHTING_DISPLAY_ON = 0
Or
LIGHTING_DISPLAY_OFF = 1.0
LIGHTING_DISPLAY_ON = 0.0

Did not work.
Restarted HA every change, reloaded the thinq integration.

@filipef101
Copy link

The homebridge lge plugin works as expected, could be worth taking a look at what they are doing

@github-actions
Copy link

This issue is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale label Aug 19, 2023
@SiboutVanLoo
Copy link

Bump

@ollo69
Copy link
Owner

ollo69 commented Aug 23, 2023

The problem with controls is that I need the right command to use.
Someone should use a mitm connetion to capture this. There is a good guide to so this, as reported here. If someone can do this, that I can try to fix this issue.

@jackbrown1993
Copy link

jackbrown1993 commented Aug 23, 2023

@ollo69 There isn't an option for 'display light' in the app, there is a button on the physical remote control though which works.

@ollo69
Copy link
Owner

ollo69 commented Aug 23, 2023

There isn't an option for 'display light' in the app, there is a button on the physical remote control though which works.

But I suppose that remote use infrared, doesn't it? So this should not be possible to implement at all.
The only way that is that someone with an AC device play with the integration code here, trying to find a command that control the light. In case I will be happy to implement in a new release.

@jackbrown1993
Copy link

Yes, the remote is infrared.

@elpigeondindo
Copy link

There isn't an option for 'display light' in the app, there is a button on the physical remote control though which works.

But I suppose that remote use infrared, doesn't it? So this should not be possible to implement at all. The only way that is that someone with an AC device play with the integration code here, trying to find a command that control the light. In case I will be happy to implement in a new release.

I can try but I don't know how to code in python too much. If someone tells me what to do I can try

@Coubi028
Copy link

@ollo69 There isn't an option for 'display light' in the app, there is a button on the physical remote control though which works.

Yes I tried too and there is no option to activate/deactivate the light in the LG Thinq androïd App...

@dougiteixeira
Copy link
Contributor

@ollo69, I tested version 0.35.1 here, but the swicth is inverted in relation to the real status of the display.

Diagnostic files:

@dougiteixeira
Copy link
Contributor

For now I do not consider the AC status (on/off) so that anybody can test switch behavior.

I believe that the switch should remain active regardless of the conditioner being turned on, as the configuration will be changed in the same way, but the physical display will only be turned on if the conditioner is in operation and the switch is active.

In other words, I can change the setting regardless of the mode the conditioner is in (on/off).

And when we use the remote control, this behavior occurs as well.

@SiboutVanLoo
Copy link

SiboutVanLoo commented Nov 2, 2023 via email

@ollo69
Copy link
Owner

ollo69 commented Nov 2, 2023

but the swicth is inverted in relation to the real status of the display.

I'm not understanding how to determinate this. Let's wait for others feedback.

@rodolfomachado
Copy link

rodolfomachado commented Nov 2, 2023

Well, I'm not at home till Sunday to test but I can say that was usefull to have the switch even when the AC was off. Another point, to me the switch was not inverted, so no need to invert the logic.
The way was before was perfect to me

@ollo69
Copy link
Owner

ollo69 commented Nov 2, 2023

It should be great have diagnostic from someone that say that logic is inverted, as @mukaschultze or @pedropaislopes

@mukaschultze
Copy link

@ollo69 here you go, diagnostic data and logs when toggling the switch

ac-off-light-off.json
ac-off-light-on.json
ac-on-light-off.json
ac-on-light-on.json
toggling-light.log

@github-actions github-actions bot removed the Stale label Nov 3, 2023
@pedropaislopes
Copy link

pedropaislopes commented Nov 3, 2023 via email

@ollo69
Copy link
Owner

ollo69 commented Nov 3, 2023

@ALL,

please test last release and let me know if switch logic is correct for all.

@ldotlopez
Copy link
Contributor

ldotlopez commented Nov 3, 2023

please test last release and let me know if switch logic is correct for all.

Logic is inverted for me now. (v0.35.1)
Also: if the unit is off light switch is on[1], toggling does nothing but a beep.

Edit: 1. The switch in on with the unit off at startup, after that it respects the (inverted) state.

diagnostics-light-on-sensor-off.json.txt
diagnostics-light-off-sensor-on.json.txt

@dougiteixeira
Copy link
Contributor

dougiteixeira commented Nov 3, 2023

please test last release and let me know if switch logic is correct for all.

Now everything works fine... the switch matches the real state of the display (v0.35.2).

Diagnostic files:

@dougiteixeira
Copy link
Contributor

Also: if the unit is off light switch is on, toggling does nothing but a beep.

This is because the setting is being changed successfully, but the display on some conditioners will only be limited if the conditioner is operating.

@dougiteixeira
Copy link
Contributor

Logic is inverted for me now. Also: if the unit is off light switch is on[1], toggling does nothing but a beep.

Edit: 1. The switch in on with the unit off at startup, after that it respects the (inverted) state.

diagnostics-light-on-sensor-off.json.txt diagnostics-light-off-sensor-on.json.txt

Hey @ldotlopez, version 0.35.2 was released which corrected this inversion for me, your diagnostic files show version 0.35.1, can you confirm if updating to version 0.35.2 makes any changes?

@ldotlopez
Copy link
Contributor

Hey @ldotlopez, version 0.35.2 was released which corrected this inversion for me, your diagnostic files show version 0.35.1, can you confirm if updating to version 0.35.2 makes any changes?

Thanks @dougiteixeira, I missed that second update.

It works OK now

@mukaschultze
Copy link

@ollo69 latest version works like a charm for me, thank you very much for looking into this! Bought you some coffee

@elpigeondindo
Copy link

For me no change. Unable to adjust the display. There is a beep but no change.

@ollo69
Copy link
Owner

ollo69 commented Nov 3, 2023

@elpigeondindo,

may be this doesn't work for all devices, but attach device diagnostics so I can check if exist some difference with known working devices.

@elpigeondindo
Copy link

@ollo69
Okay but how i can do this?

@ollo69
Copy link
Owner

ollo69 commented Nov 3, 2023

Go on integration page and select download diagnostic.

@elpigeondindo
Copy link

@pedropaislopes
Copy link

Hi! I can confirm that everything is OK now, on displays with correct and on displays with inverted logic. Thanks a lot!

@ollo69
Copy link
Owner

ollo69 commented Nov 6, 2023

@elpigeondindo,

I checked your diagnostic but it is identical to the others, so I have no idea why is not working for you.

@ollo69
Copy link
Owner

ollo69 commented Nov 6, 2023

I close this issue, everything should be done was done. If there are other issue or request please open a new thread.

@ollo69 ollo69 closed this as completed Nov 6, 2023
@joaolucasd
Copy link

Just adding this here for people looking for a way arround with the model RAC_056905_WW. I could not find the fix, but just a diagnosis.
Although API is not working, when you press the light button on the IR controller, the state of the light changes when looking at the snapshot:
'airState.lightingState.displayControl': 0.0 = lights on
'airState.lightingState.displayControl': 1.0 = lights off
But when you send the command (Setting new state for device XXXXX: basicCtrl - Set - airState.lightingState.displayControl - 1 or Setting new state for device XXXXX: basicCtrl - Set - airState.lightingState.displayControl - 0) nothing happens, just the beep sound.
I guess the AC can show the light state, but cannot change it through API. That´s a big bummer, cause this AC is on my toddler's room and having that hability would've been awesome.

@filipef101
Copy link

Kind of random, but kind of related, has anyone tried to disabled the beeping sound? Some remotes have that option but mine doesn't assume AC doesn't support disabling the beeping 😔

@ollo69
Copy link
Owner

ollo69 commented Oct 11, 2024

@ALL,

this was already discussed a lot. The light control is there (exactly the same used by other integration) but not all AC device support this control via API.
There is nothing that I can do, if the integration control just make a beep and nothing change means that the control doesn't work and the only thing you can do is disable it.

@luiscamalmeida
Copy link

Since I don't believe we'll ever get these functions added by LG let me share how I've modded the units around my house to solve those problems. For the beeping sound just take the cap of the piezo beeper and remove the metal disc (save it for a later date should you ever feel nostalgic). For he display, I've taped some aluminium tape on top of the display and it dims the light enough for it to be used on our son's bedroom.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests