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

Support for LG AKB73757604 remote (new model with vertical and horizontal swing) #1531

Closed
kzagatti opened this issue Jul 13, 2021 · 9 comments · Fixed by #1545
Closed

Support for LG AKB73757604 remote (new model with vertical and horizontal swing) #1531

kzagatti opened this issue Jul 13, 2021 · 9 comments · Fixed by #1545
Assignees
@crankyoldgit
Labels
enhancement

Comments

@kzagatti
Copy link

Hi @crankyoldgit

Helping you grow the support for LG remotes:

External A/C: A4UW30GFA2
Internal A/C: AMNW24GTPA1
Remote model: AKB73757604

Photos of the original remote:

image image

I have the wifi module for this unit so most of the time I'm using either HA or the LG ThinQ app to manage it. But to help others, I created the sheet with the codes I collected:

https://docs.google.com/spreadsheets/d/17C_Ay7OjsYNSAxxj8uXbh0Vi2jrqyrncwzIyUOGSuNo/edit?usp=sharing

Most of the normal operations are recognized correctly considering the development for AKB75215403 (Model: 2) and AKB74955603 (Model: 3), but this A/C unit has horizontal swing and you can also set the Vent Angle for the vertical swing positions.

And as we did for Model: 3, I tested adjusting the swing to a specific position, "hide" the remote and change the position and turn it off/on - the A/C unit "remembers" its last position and does not assume what's on the remote - the codes are fixed and there is a logic you can notice on the sheet.

So I think this might be simple to implement, but let me know if you need any other tests and I'll be happy to help!
@crankyoldgit crankyoldgit self-assigned this Jul 14, 2021
@crankyoldgit
Copy link
Owner

I'll try to take a longer look later. I really should address some other folks issues for a bit. ;-)

the codes are fixed and there is a logic you can notice on the sheet.

If "there is a logic you can notice", especially in the bit patterns. Please spell it out.

@kzagatti
Copy link
Author

Here's a photo of the A/C unit:

image

It has 4 vanes (4 independent vertical swings) and you can set their position using the remote.

If "there is a logic you can notice", especially in the bit patterns. Please spell it out.

Sure, they're fixed codes, just as in Model 3. With Model 3, IRrecvDumpV2 identifies the swing positions from 4 (lowest) to 9 (highest) along with Auto (20) and Off (21). This "Model 4" implements some new codes, 22 and 23 for horizontal swing (auto and off respectively) and 33 to 62 for each vane / position - these are all on the sheet.

I should say that I have this A/C unit for a year but I've never changed the vane angle position either on the remote or on the app - I'm barely using the auto vertical (which is already supported). It's great to work on this and discover "new" features.

crankyoldgit added a commit that referenced this issue Aug 3, 2021
@crankyoldgit
[LG] Add support for AKB73757604 model
8bd0555 
* Support for Vertical Swing on multiple vanes.
* Support for Horizontal Swing.
* Model detection.
* Unit tests added & updated.

Fixes #1531
@crankyoldgit
Copy link
Owner

Hey @kzagatti can you please try out branch: https://github.com/crankyoldgit/IRremoteESP8266/tree/AKB73757604 hopefully it adds the support you've pointed out.

@crankyoldgit crankyoldgit added the Pending Confirmation Waiting for confirmation from user label Aug 3, 2021
@crankyoldgit crankyoldgit mentioned this issue Aug 3, 2021
Merged
@kzagatti
Copy link
Author

kzagatti commented Aug 3, 2021

@crankyoldgit

Almost there! Tested and horizontal swing and vanes 0 and 2 are OK, but vanes 1 and 3 are showing up as UNKNOWN. Probably a little adjust...

Vane 1 results: https://docs.google.com/spreadsheets/d/17C_Ay7OjsYNSAxxj8uXbh0Vi2jrqyrncwzIyUOGSuNo/edit#gid=0&range=A47:G52
Vane 3 results: https://docs.google.com/spreadsheets/d/17C_Ay7OjsYNSAxxj8uXbh0Vi2jrqyrncwzIyUOGSuNo/edit#gid=0&range=A59:G64

Tks

@crankyoldgit
Copy link
Owner

Ah yep, I know what I did wrong. Will fix it soon. Thanks for the prompt & detailed feedback. That sort of thing really helps.

crankyoldgit added a commit that referenced this issue Aug 4, 2021
@crankyoldgit
Fix problem with vane position decoding.
46573ca 
* Fix a dumb math fail.
* Add unit test to make sure it stays fixed.

Ref: #1531 (comment)
@crankyoldgit
Copy link
Owner

Almost there! Tested and horizontal swing and vanes 0 and 2 are OK, but vanes 1 and 3 are showing up as UNKNOWN. Probably a little adjust...

Now fixed in the latest commit to that branch. Please verify and let me know if everything is all good.

@kzagatti
Copy link
Author

kzagatti commented Aug 4, 2021

Ah yep, I know what I did wrong. Will fix it soon. Thanks for the prompt & detailed feedback. That sort of thing really helps.

Happy to help!

Now fixed in the latest commit to that branch. Please verify and let me know if everything is all good.

Tested everything, all commands again and I confirm, it's all good!

@crankyoldgit
Copy link
Owner

Excellent. Thanks for the confirmation.

crankyoldgit added a commit that referenced this issue Aug 4, 2021
@crankyoldgit
[LG] Add support for AKB73757604 model (#1545)
1c9a75b 
* Support for Vertical Swing on multiple vanes.
* Support for Horizontal Swing.
* Model detection.
* Unit tests added & updated.

Fixes #1531
@crankyoldgit crankyoldgit removed the Pending Confirmation Waiting for confirmation from user label Aug 4, 2021
crankyoldgit added a commit that referenced this issue Aug 28, 2021
@crankyoldgit
v2.7.20 release
1fbac6c 
_v2.7.20 (20210828)_

**[Bug Fixes]**
- Make `strToSwingH()` match "Right Max" (#1550 #1551)

**[Features]**
- Experimental Bose remote support (#1579)
- Added MitsubishiAC VaneLeft (#1572 #1576)
- HAIER_AC176: Add experimental detailed support (#1480 #1571)
- Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568)
- Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507)
- SamsungAc: Use `sendExtended()` going forward. (#1484 #1562)
- SamsungAc: Redo/fix checksum calculations. (#1538 #1554)
- LG: Add support for `AKB73757604` model (#1531 #1545)
- Daikin176: Add support for Unit Id. (#1543 #1544)
- Daikin2: Add support for Humidity setting/operation. (#1535 #1540)
- TCL112AC: Add support for quiet/mute setting. (#1528 #1529)
- LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530)
- Add Mitsubishi AC "fan only" mode (#1527)

**[Misc]**
- Fix pylint issues due to pylint update. (#1569 #1570)
- DAIKIN216: Update supported models. (#1552 #1567)
- IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541)
- Reduce memory fragmentation cause by String usage. (#1493 #1536)
- Refactor `decodeMitsubishiAC()` (#1523 #1532)
- Fix incorrect comment.
- Migrate from Travis to GitHub Actions (#1522 #1526)
- Documentation update with additional supported Panasonic AC models (#1525)
@crankyoldgit crankyoldgit mentioned this issue Aug 28, 2021
Merged
crankyoldgit added a commit that referenced this issue Aug 28, 2021
@crankyoldgit
v2.7.20 release
e8dee1a 
_v2.7.20 (20210828)_

**[Bug Fixes]**
- Make `strToSwingH()` match "Right Max" (#1550 #1551)

**[Features]**
- Experimental Bose remote support (#1579)
- Added MitsubishiAC VaneLeft (#1572 #1576)
- HAIER_AC176: Add experimental detailed support (#1480 #1571)
- Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568)
- Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507)
- SamsungAc: Use `sendExtended()` going forward. (#1484 #1562)
- SamsungAc: Redo/fix checksum calculations. (#1538 #1554)
- LG: Add support for `AKB73757604` model (#1531 #1545)
- Daikin176: Add support for Unit Id. (#1543 #1544)
- Daikin2: Add support for Humidity setting/operation. (#1535 #1540)
- TCL112AC: Add support for quiet/mute setting. (#1528 #1529)
- LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530)
- Add Mitsubishi AC "fan only" mode (#1527)

**[Misc]**
- Change when some github workflows run (#1583)
- Add/update supported device info (#1580 #1581 #1585)
- Fix pylint issues due to pylint update. (#1569 #1570)
- DAIKIN216: Update supported models. (#1552 #1567)
- IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541)
- Reduce memory fragmentation cause by String usage. (#1493 #1536)
- Refactor `decodeMitsubishiAC()` (#1523 #1532)
- Fix incorrect comment.
- Migrate from Travis to GitHub Actions (#1522 #1526)
- Documentation update with additional supported Panasonic AC models (#1525)
crankyoldgit added a commit that referenced this issue Aug 28, 2021
@crankyoldgit
v2.7.20 release (#1582)
21e7dd4 
## _v2.7.20 (20210828)_

**[Bug Fixes]**
- Make `strToSwingH()` match "Right Max" (#1550 #1551)

**[Features]**
- Experimental Bose remote support (#1579)
- Added MitsubishiAC VaneLeft (#1572 #1576)
- HAIER_AC176: Add experimental detailed support (#1480 #1571)
- Detailed support for Tornado/Sanyo 88-bit A/C protocol (#1503 #1568)
- Add support for new `TROTEC_3550` A/C protocol (#1563 #1566 #1507)
- SamsungAc: Use `sendExtended()` going forward. (#1484 #1562)
- SamsungAc: Redo/fix checksum calculations. (#1538 #1554)
- LG: Add support for `AKB73757604` model (#1531 #1545)
- Daikin176: Add support for Unit Id. (#1543 #1544)
- Daikin2: Add support for Humidity setting/operation. (#1535 #1540)
- TCL112AC: Add support for quiet/mute setting. (#1528 #1529)
- LG2: Add Fan speed, Swing, & Light support for new `AKB74955603` model (#1513 #1530)
- Add Mitsubishi AC "fan only" mode (#1527)

**[Misc]**
- Change when some github workflows run (#1583)
- Add/update supported device info (#1580 #1581 #1585)
- Fix pylint issues due to pylint update. (#1569 #1570)
- DAIKIN216: Update supported models. (#1552 #1567)
- IRMQTTServer: Build a minimal OTA image via PlatformIO. (#1513 #1541)
- Reduce memory fragmentation cause by String usage. (#1493 #1536)
- Refactor `decodeMitsubishiAC()` (#1523 #1532)
- Fix incorrect comment.
- Migrate from Travis to GitHub Actions (#1522 #1526)
- Documentation update with additional supported Panasonic AC models (#1525)
@crankyoldgit
Copy link
Owner

FYI, the changes mentioned above have been included in the just released v2.7.20 of the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@crankyoldgit crankyoldgit

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[LG] Add support for AKB73757604 model crankyoldgit/IRremoteESP8266
2 participants
@kzagatti @crankyoldgit