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

Experimental support for Panasonic A/C messages. #535

Merged
merged 6 commits into from
Oct 2, 2018
Merged

Conversation

crankyoldgit
Copy link
Owner

Ref: #525

* sendPanasonicAC(), decodePanasonicAC(), & IRPanasonicAc class added.
* Unit tests for those.
* Code HEAVILY influenced by the work done at:
  https://github.com/ToniA/ESPEasy/blob/HeatpumpIR/lib/HeatpumpIR/PanasonicHeatpumpIR.cpp
* Should support LKE/DKE/JKE/NKE series units.
* Updated example code as required.

Ref: #525
* Panasonic has a Fan mode that requires the temp to be 27C.
* Some code lint improvements.
* Detail models supported etc.

Ref #525
@crankyoldgit
Copy link
Owner Author

FYI, @pandemixxx has confirmed decoding seems to be working fine on a real device. Waiting on feedback for sending.

seancaulfield pushed a commit to seancaulfield/IRremoteESP8266 that referenced this pull request Sep 30, 2018
RAWTICK is deprecated. However some users may have used this #define in their code, so adding an alias to the new name will help some user projects compile.

Ref: crankyoldgit#535
@crankyoldgit
Copy link
Owner Author

@pandemixxx has confirmed sendPanasonicAC() works as expected too! Huzzah!

src/ir_Panasonic.cpp Outdated Show resolved Hide resolved
crankyoldgit and others added 3 commits October 1, 2018 17:58
* Make the class remember the set temp, so Fan mode doesn't overwrite it.
* Adjust unit tests to cover that.
* Update unit test Makefile to include protocol header files for dependencies.

Ref: #525 (comment)
@crankyoldgit crankyoldgit added the Hacktoberfest Hacktoberfest participation label Oct 2, 2018
@crankyoldgit crankyoldgit merged commit 17d0364 into master Oct 2, 2018
crankyoldgit added a commit that referenced this pull request Oct 2, 2018
**[Bug Fixes]**
- Correct the byte used for Samsung AC Swing. (#529)
- Fix not sending Samsung A/C messages in IRMQTTServer. (#529)

**[Features]**
- Experimental support for Electra A/C messages. (#528)
- Samsung A/C fixes & improvements (#529)
- IRMQTTServer v0.6.0 (#530)

**[Misc]**
- Change required WifiManager lib version to v0.14
- Add alias for RAWTICK to kRawTick. (#535)
- Update sendLutron() status. (#515)
- Remove leftover debug message in IRrecvDumpV2 (#526)
crankyoldgit added a commit that referenced this pull request Oct 2, 2018
**[Bug Fixes]**
- Correct the byte used for Samsung AC Swing. (#529)
- Fix not sending Samsung A/C messages in IRMQTTServer. (#529)

**[Features]**
- Experimental support for Electra A/C messages. (#528)
- Experimental support for Panasonic A/C messages. (#535)
- Samsung A/C fixes & improvements (#529)
- IRMQTTServer v0.6.0 (#530)

**[Misc]**
- Change required WifiManager lib version to v0.14
- Add alias for RAWTICK to kRawTick. (#535)
- Update sendLutron() status. (#515)
- Remove leftover debug message in IRrecvDumpV2 (#526)
@crankyoldgit crankyoldgit mentioned this pull request Oct 2, 2018
crankyoldgit added a commit that referenced this pull request Oct 2, 2018
**[Bug Fixes]**
- Correct the byte used for Samsung AC Swing. (#529)
- Fix not sending Samsung A/C messages in IRMQTTServer. (#529)

**[Features]**
- Experimental support for Electra A/C messages. (#528)
- Experimental support for Panasonic A/C messages. (#535)
- Samsung A/C fixes & improvements (#529)
- IRMQTTServer v0.6.0 (#530)

**[Misc]**
- Change required WifiManager lib version to v0.14
- Add alias for RAWTICK to kRawTick. (#535)
- Update sendLutron() status. (#515)
- Remove leftover debug message in IRrecvDumpV2 (#526)
@crankyoldgit crankyoldgit deleted the panasonic_ac branch October 13, 2018 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Hacktoberfest Hacktoberfest participation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants