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

Adjust for AKB74955603 (LG2 Model 3) #1737

Closed
kzagatti opened this issue Jan 15, 2022 · 9 comments · Fixed by #1738
Closed

Adjust for AKB74955603 (LG2 Model 3) #1737

kzagatti opened this issue Jan 15, 2022 · 9 comments · Fixed by #1738
Assignees

Comments

@kzagatti
Copy link

kzagatti commented Jan 15, 2022

Hi @crankyoldgit,

It's been a while since we worked on AKB74955603 and AKB73757604, rock solid with IRMQTTServer and Home Assistant.

I have a bedroom where along with HA, I started using the original remote (you know, wife). The IR device properly decodes the IR code sent by the remote and things such as modes, on/off, temp, etc, are all OK. But when I press Light Toggle, it shows as "off" on Home Assistant (although it's still on).

This is the output from IRrecvDumpV2 (it's OK):

Protocol  : LG2
Code      : 0x88C00A6 (28 Bits)
Mesg Desc.: Model: 3 (AKB74955603), Light Toggle: On
uint16_t rawData[59] = {3424, 9810,  588, 1494,  588, 492,  538, 502,  538, 494,  538, 1524,  544, 502,  538, 494,  538, 496,  536, 1538,  540, 1526,  540, 474,  564, 494,  538, 494,  538, 496,  538, 494,  538, 504,  538, 476,  564, 494,  538, 496,  538, 484,  564, 1516,  566, 474,  564, 1534,  542, 494,  538, 496,  538, 1518,  566, 1532,  564, 476,  564};  // LG2 88C00A6
uint32_t address = 0x88;
uint32_t command = 0xC00A;
uint64_t data = 0x88C00A6;

But these are the MQTT messages sent (latest first):

1-) When I turn the A/C on:

Message 11 received on ir_server/ac/stat/fanspeed at 4:20 PM:
Min
QoS: 0 - Retain: false
Message 10 received on ir_server/ac/stat/temp at 4:20 PM:
18
QoS: 0 - Retain: false
Message 9 received on ir_server/ac/stat/mode at 4:20 PM:
fan_only
QoS: 0 - Retain: false
Message 8 received on ir_server/ac/stat/power at 4:20 PM:
on
QoS: 0 - Retain: false

2-) When I press the Light button:

Message 16 received on ir_server/ac/stat/light at 4:20 PM:
off
QoS: 0 - Retain: false
Message 15 received on ir_server/ac/stat/fanspeed at 4:20 PM:
High
QoS: 0 - Retain: false
Message 14 received on ir_server/ac/stat/temp at 4:20 PM:
15
QoS: 0 - Retain: false
Message 13 received on ir_server/ac/stat/mode at 4:20 PM:
off
QoS: 0 - Retain: false
Message 12 received on ir_server/ac/stat/power at 4:20 PM:
off
QoS: 0 - Retain: false

Please, is it possible to adjust the code to not update the power mode when the Light Toggle is pressed?

Not sure if you'd need this too, but following you'll find my HA configuration (as stated on IRMQTTServer's guide):

climate:
  - platform: mqtt
    name: AC MQTT
    modes:
      - "off"
      - "cool"
      - "heat"
      - "fan_only"
    fan_modes:
      - "Auto"
      - "Min"
      - "Low"
      - "Medium"
      - "High"
      - "Max"
    swing_modes:
      - "Off"
      - "Auto"
      - "Highest"
      - "High"
      - "Middle"
      - "Low"
      - "Lowest"
    #power_command_topic: "ir_server/ac/cmnd/power"
    mode_command_topic: "ir_server/ac/cmnd/mode"
    mode_state_topic: "ir_server/ac/stat/mode"
    temperature_command_topic: "ir_server/ac/cmnd/temp"
    temperature_state_topic: "ir_server/ac/stat/temp"
    current_temperature_topic: "tele/tasmota_94A3B6/SENSOR"
    current_temperature_template: "{{ value_json['DHT11'].Temperature }}"
    fan_mode_command_topic: "ir_server/ac/cmnd/fanspeed"
    fan_mode_state_topic: "ir_server/ac/stat/fanspeed"
    swing_mode_command_topic: "ir_server/ac/cmnd/swingv"
    swing_mode_state_topic: "ir_server/ac/stat/swingv"
    min_temp: 18
    max_temp: 30
    temp_step: 1
    retain: false

Thank you

@crankyoldgit
Copy link
Owner

I'll see what I can do.

crankyoldgit added a commit that referenced this issue Jan 16, 2022
Don't override previous settings if the current message being processed is a Light Toggle message.

Fixes #1737
@crankyoldgit
Copy link
Owner

@kzagatti Can you please download and try out branch https://github.com/crankyoldgit/IRremoteESP8266/tree/Issue1737
Hopefully it will do what you want. Please let us know asap. If it works, I'll make it into a PR and get it merged.

@kzagatti
Copy link
Author

@crankyoldgit compiled IRMQTT from the branch but the final result is the same, when I press the Light button on the remote, it shows as "off" on HA. Not sure if it helps, but below you'll also find the prints from IRMQTTServer Air Conditioner Control:

1-) When I press the power on button:

image

2-) Right after pressing Light button:

image

@crankyoldgit
Copy link
Owner

Hmmm.
My tests simulating the issue indicate it should be fixed. i.e. Power: On should remain after it receives the "Light Toggle" button press from a real remote.
Can you please confirm/check you're building the example code against the downloaded version from that branch by modifying the src/IRremoteESP8266.h file where you downloaded it. Modify these lines:

// Library Version Information
// Major version number (X.x.x)
#define _IRREMOTEESP8266_VERSION_MAJOR 2
// Minor version number (x.X.x)
#define _IRREMOTEESP8266_VERSION_MINOR 8
// Patch version number (x.x.X)
#define _IRREMOTEESP8266_VERSION_PATCH 1

to something unique. e.g. #define _IRREMOTEESP8266_VERSION_MAJOR 4 or something.
You can check on the "Info" page of your ESP running IRMQTTServer what version it's running.

Also, can you provide the annotated MQTT log your provided earlier for your tests? That was quite helpful in track it down.

@crankyoldgit
Copy link
Owner

and while I think about it, Please also include the MQTT messages sent to the "IR Received topic" (which is listed on the "Info" page.

That should be ir_server/received if I've read the data you've supplied correctly.

@kzagatti
Copy link
Author

You got it!

Although I was compiling it from the right branch, it was actually getting the library from the default folder. I deleted the original library and pasted from the branch and it is working!

From IRMQTTServer info page:

Version: v1.6.0-kzag
Built: Jan 17 2022 12:08:39
IR Lib Version: 4.8.1

MQTT Messages (8-11 when I press power on, 12-15 pressing light toggle):

Message 15 received on ir_server/ac/stat/light at 12:22 PM:
on
QoS: 0 - Retain: false
Message 14 received on ir_server/ac/stat/light at 12:22 PM:
off
QoS: 0 - Retain: false
Message 13 received on ir_server/ac/stat/light at 12:22 PM:
on
QoS: 0 - Retain: false
Message 12 received on ir_server/ac/stat/light at 12:22 PM:
off
QoS: 0 - Retain: false
Message 11 received on ir_server/ac/stat/fanspeed at 12:22 PM:
Max
QoS: 0 - Retain: false
Message 10 received on ir_server/ac/stat/temp at 12:22 PM:
18
QoS: 0 - Retain: false
Message 9 received on ir_server/ac/stat/mode at 12:22 PM:
fan_only
QoS: 0 - Retain: false
Message 8 received on ir_server/ac/stat/power at 12:22 PM:
on
QoS: 0 - Retain: false

From the ir_server/received topic:

Message 3 received on ir_server/received at 12:26 PM:
51,0x88C00A6,28
QoS: 0 - Retain: false
Message 2 received on ir_server/received at 12:26 PM:
51,0x88C00A6,28
QoS: 0 - Retain: false
Message 1 received on ir_server/received at 12:26 PM:
51,0x88C00A6,28
QoS: 0 - Retain: false
Message 0 received on ir_server/received at 12:26 PM:
51,0x88C00A6,28
QoS: 0 - Retain: false

Thanks again!!!

@crankyoldgit
Copy link
Owner

I deleted the original library and pasted from the branch and it is working!

Thanks for doing that. Is that it is all working as expected now, or is there something else/more I need to address?

@kzagatti
Copy link
Author

Thanks for doing that. Is that it is all working as expected now, or is there something else/more I need to address?

It is all working as expected!

crankyoldgit added a commit that referenced this issue Jan 18, 2022
Don't override previous settings if the current message being processed is a Light Toggle message.

Confirmed working.

Fixes #1737
crankyoldgit added a commit that referenced this issue Mar 14, 2022
_v2.8.2 (20220314)_

**[Bug Fixes]**
- ESP32-C3: Fix reboot/crashes on ESP32-C3s when receiving. (#1768 #1751)

**[Features]**
- HITACHI_AC296: Add `IRac` class support & tests. (#1776 #1758 #1757)
- Support for Hitachi RAS-70YHA3 (remote RAR-3U3) (#1758 #1757)
- LG: Add Swing Toggle support for Model `LG6711A20083V` (#1771 #1770)
- IRMQTTServer: add `MQTT_SERVER_AUTODETECT_ENABLE` via mqtt mDNS (#1769)
- Experimental basic support for Kelon 168 bit / 21 byte protocol. (#1747 #1745 #1744)
- MitsubishiAC: Tweak repeat gap timing. (#1760 #1759)
- Gree YAP0F8 (Detected as Kelvinator) vertical position set support (#1756)
- Make KELON (48 bit) protocol decoding stricter. (#1746 #1744)
- IRMQTTServer V1.6.1 (#1740 #1739 #1729)
- HITACHI_AC264: Add minimal detailed support. (#1735 #1729)
- LG2: Improve Light toggle msg handling. (#1738 #1737)
- MIDEA: Add support for Quiet, Clean & Freeze Protect controls. (#1734 #1733)
- Add basic support for HITACHI_AC264 264bit protocol. (#1730 #1729)
- ESP32-C3: Work around for some C3 specific compiler issues again. (#1732 #1695)

**[Misc]**
- MIDEA: Update supported devices (#1774 #1773 #1716)
- Update devices supported by ELECTRA_AC (#1766 #1765)
- Improve documentation for `encodePioneer()` (#1761 #1749)
- Update (un)supported DAIKIN128 devices. (#1752)
- Refactor `decodeCOOLIX()` code & add another test case. (#1750 #1748)
- Simplify code based on state_t being initialised by default. (#1736 #1699)
- Add comments to help Teknopoint users. (#1731 #1728)
- Fix library version string calculation. (#1727 #1725)
- Confirm we can reproduce `TurnOnFujitsuAC.ino` via IRac/IRMQTTServer. (#1726 #1701)
crankyoldgit added a commit that referenced this issue Mar 15, 2022
##_v2.8.2 (20220314)_

**[Bug Fixes]**
- ESP32-C3: Fix reboot/crashes on ESP32-C3s when receiving. (#1768 #1751)

**[Features]**
- HITACHI_AC296: Add `IRac` class support & tests. (#1776 #1758 #1757)
- Support for Hitachi RAS-70YHA3 (remote RAR-3U3) (#1758 #1757)
- LG: Add Swing Toggle support for Model `LG6711A20083V` (#1771 #1770)
- IRMQTTServer: add `MQTT_SERVER_AUTODETECT_ENABLE` via mqtt mDNS (#1769)
- Experimental basic support for Kelon 168 bit / 21 byte protocol. (#1747 #1745 #1744)
- MitsubishiAC: Tweak repeat gap timing. (#1760 #1759)
- Gree YAP0F8 (Detected as Kelvinator) vertical position set support (#1756)
- Make KELON (48 bit) protocol decoding stricter. (#1746 #1744)
- IRMQTTServer V1.6.1 (#1740 #1739 #1729)
- HITACHI_AC264: Add minimal detailed support. (#1735 #1729)
- LG2: Improve Light toggle msg handling. (#1738 #1737)
- MIDEA: Add support for Quiet, Clean & Freeze Protect controls. (#1734 #1733)
- Add basic support for HITACHI_AC264 264bit protocol. (#1730 #1729)
- ESP32-C3: Work around for some C3 specific compiler issues again. (#1732 #1695)

**[Misc]**
- MIDEA: Update supported devices (#1774 #1773 #1716)
- Update devices supported by ELECTRA_AC (#1766 #1765)
- Improve documentation for `encodePioneer()` (#1761 #1749)
- Update (un)supported DAIKIN128 devices. (#1752)
- Refactor `decodeCOOLIX()` code & add another test case. (#1750 #1748)
- Simplify code based on state_t being initialised by default. (#1736 #1699)
- Add comments to help Teknopoint users. (#1731 #1728)
- Fix library version string calculation. (#1727 #1725)
- Confirm we can reproduce `TurnOnFujitsuAC.ino` via IRac/IRMQTTServer. (#1726 #1701)
@crankyoldgit
Copy link
Owner

FYI, the changes mentioned above have now been included in the new v2.8.2 release of the library.

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

Successfully merging a pull request may close this issue.

2 participants