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

Basic support for Amcor A/C protocol. #836

Merged
merged 4 commits into from
Jul 28, 2019
Merged

Basic support for Amcor A/C protocol. #836

merged 4 commits into from
Jul 28, 2019

Conversation

crankyoldgit
Copy link
Owner

  • Appears to be a simple 64 bit protocol with a manditory repeat.
  • sendAmcor() & decodeAmcor().
    • Protocol is NOT space encoded. It's a fixed length bit encoding.
  • Update IRMQTTServer to be able to send it.

NOTE: Bit ordering and bit flipping may happen. This is experimental.

For #834

* Appears to be a simple 64 bit protocol with a manditory repeat.
* sendAmcor() & decodeAmcor().
  - Protocol is NOT space encoded. It's a fixed length bit encoding.
* Update IRMQTTServer to be able to send it.

NOTE: Bit ordering and bit flipping may happen. This is experimental.

For #834
Copy link
Collaborator

@rahuljawale rahuljawale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

* Observations of temperature data have confirmed I got the bit ordering 
wrong.
  - Instead of just changing to LSBFirst order, use LSBF per byte. This 
will make things easier later.

For #834
@crankyoldgit
Copy link
Owner Author

@rahuljawale Thanks for the review. Can I get you to have another quick look at it as I just changed it fundamentally.

Copy link
Collaborator

@rahuljawale rahuljawale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Merge away!

@crankyoldgit
Copy link
Owner Author

Thanks again @rahuljawale

@crankyoldgit crankyoldgit merged commit 3c709e9 into master Jul 28, 2019
@crankyoldgit crankyoldgit deleted the amcor branch August 3, 2019 09:15
crankyoldgit added a commit that referenced this pull request Aug 28, 2019
_v2.6.5 (20190828)_

**[Bug Fixes]**
- IRMQTTServer: Remove duplicate MQTT_CLIMATE from HA discovery (#869)
- Fujitsu: Ensure `on()` is called in common a/c framework. (#862)
- Update `strToModel()` (#861)
- IRMQTTServer: Add missing header file. (#858)
- IRMQTTServer: Fix a compile error when HTML_PASSWORD_ENABLE is enabled. (#856)

**[Features]**
- IRrecv: Allow receive tolerance percentage to be set at run-time. (#865)
- Basic support for Daikin152 A/C protocol. (#874)
- Teco: Add light, humid, & save support. (#871)
- Detailed support for Amcor A/C protocol. (#836, #854)
- IRMQTTServer: Add ability to report Vcc at the ESP chip. (#845)
- Gree: Add timer support. (#849)
- IRac/Mitsubishi A/C: Support wide `swingh_t` mode (#844)
- IRMQTTServer: Generate protocol and bit size html selects (#838)

**[Misc]**
- New example code to show how to use the `IRac` class to control A/Cs (#839)
- Improve/fix `swingh_t::kWide` support (#846)
- Kelvinator: Optimise code a little to save space. (#843)
@crankyoldgit crankyoldgit mentioned this pull request Aug 28, 2019
crankyoldgit added a commit that referenced this pull request Aug 28, 2019
_v2.6.5 (20190828)_

**[Bug Fixes]**
- IRMQTTServer: Remove duplicate MQTT_CLIMATE from HA discovery (#869)
- Fujitsu: Ensure `on()` is called in common a/c framework. (#862)
- Update `strToModel()` (#861)
- IRMQTTServer: Add missing header file. (#858)
- IRMQTTServer: Fix a compile error when HTML_PASSWORD_ENABLE is enabled. (#856)

**[Features]**
- IRrecv: Allow receive tolerance percentage to be set at run-time. (#865)
- Basic support for Daikin152 A/C protocol. (#874)
- Teco: Add light, humid, & save support. (#871)
- Detailed support for Amcor A/C protocol. (#836, #854)
- IRMQTTServer: Add ability to report Vcc at the ESP chip. (#845)
- Gree: Add timer support. (#849)
- IRac/Mitsubishi A/C: Support wide `swingh_t` mode (#844)
- IRMQTTServer: Generate protocol and bit size html selects (#838)

**[Misc]**
- New example code to show how to use the `IRac` class to control A/Cs (#839)
- Improve/fix `swingh_t::kWide` support (#846)
- Kelvinator: Optimise code a little to save space. (#843)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants