-
Notifications
You must be signed in to change notification settings - Fork 839
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
Adding support for A/C Trotec PAC 3550 Pro #1563
Comments
Please try out the code in branch: https://github.com/crankyoldgit/IRremoteESP8266/tree/Trotec3550 and let me know how it goes for you. It's only basic support (simple send & receive) of a X-Ref: #1507 . |
Mhh, I´m not excat sure how I should test it? Send the TurnOnTrotecAC.ino in e.g. Wemos D1 with IR-LED and let it run ? |
A simplistic approach to testing it:
Or .. replace the copy of the library inside Tasmota's source with the one from this branch, and and rebuild Tasmota. Note: This doesn't give you the ability to set temperatures like other protocols ... yet. |
Meanwhile I downloaded ../tree/Trotec3550.zip and replaced older IRremoteESP libraries and load TurnOnTrotecAC to a Wemos D1 and a IR LED via NPN Transistor. I see IR LED via Handy Video Mode working, and YTR IR Bridge receives something, but the A/C is not turning on. |
As for Tasmota, you'll need to compile the "full IR" version with that branch for it to work. (if you want help using/building Tasmota, you'll need to talk to them. Not here) |
Now I´m confused, I thought all of this will helps to bring in Model Pac 3550, here as a second device in Trotec section. |
With IRrecevDump2 I get this with command from original remote (18°C,cool,f3,swing=on) and this when I push off: |
Sadly, the settings bit locations, and protocol timings are so different from the existing
You will be able to use it sort of like that soon. Except |
Excellent. Thanks for that. That means the basic decoding for the protocol is working as expected. I can now move on to adding detailed control, like adjusting settings. I still need you to check you can transmit with |
* Add detailed & Common A/C API support for: - Power - Temp (excluding Fahrenheit) - SwingV - Mode - Fan Speed - Checksums * Add and update Unit tests. For #1563
I've just added a commit to that branch that adds detailed support based on the data/spreadsheet you've provided. There are still aspects of your spreadsheet and analysis that are not clear or incomplete. Please download and test it. This version should give you the support you want via Tasmota, if you build against this library. Let me know what works, doesn't work, or is missing etc. Note: We shouldn't need |
So positive tested in IRsendDemo.ino 👍 I have added the lines in excel for °F, if something else is missing please let me know. Mhh, I never made my own build of Tasmota via ArduinoIDE, I thought so far it is too complex to include all right libs. |
So, finally with ATOM I made my first own tasmota build with replaced your experimental code above. AND it works fine :-) 10:42:48.419 RSL: RESULT = {"IRHVAC":{"Vendor":"TROTEC_3550","Model":1,"Mode":"Auto","Power":"On","Celsius":"On","Temp":18,"FanSpeed":"Medium","SwingV":"Off","SwingH":"Off","Quiet":"Off","Turbo":"Off","Econo":"Off","Light":"Off","Filter":"Off","Clean":"Off","Beep":"Off","Sleep":-1}} Have not checked all parameter, but on off temp mode are working fine. THANKS for you great support. |
Excellent. I'll take a look tomorrow.
yay!
Looking forward to the feedback.
Thanks for confirming. I'll note that it works in the source. |
* Add proper support for DegC and DegF * Update `sendTrotec3550()` to STABLE. For #1563
I've added proper Fahrenheit support in the latest commit to that branch. |
So I have tested again, and it works fine. Just what I learned is that when I like to change single parameter like Temp 18->19 I have to give "Power On" parameter too, otherwise it went off by default. But I think so it is right implemented, not playing around with bit 58.. as the status if A/C was already On or not is not known. |
I'm not sure I understand what you are saying. Can you please give data/examples? If you're only supplying one parameter in each JSON/IRHVAC tasmota object/message, then it may be only applying that one parameter, thus, yes, you would need to tell it to power on. But I don't know how Tasmota handles climate states etc. That's something you will need to talk to them about. What is bit 58? I don't understand. Your spreadsheet lists it as Also, what is the maximum value for the timer? is it 8 hrs? |
Oh ... FYI, the first Temp/ |
What I want to say, all lines 43-50 (with U) do not start the a/c however bit15 is set to 1, I assume it has to do with bit 58 if this is also "1" the a/c will ignore bit15. I collected the values when remote control display in on just changing temp + or -. |
Ah ... Yes, I see now. I'll leave that bit (58) off/alone.
FYI, you got the number of bits for the Timer wrong. You can't fit |
The branch now has support for the Timer function. Please test if you can etc. |
So °F setting is working, but I´m struggeling with Timer setting, is this the "Sleep" command in IRhvac ? I played with it but does not work. |
No, it's controlled by the Thanks for confirming the Fahrenheit settings/control works. |
_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)
_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)
## _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)
FYI, the changes mentioned above have been included in the just released v2.7.20 of the library. |
Great, thanks for implementation and work on this together. |
Comming from discussion board, here is my new request for adding support A/C Trotec PAC 3550 Pro.
For analyses I used : https://github.com/ToniA/Raw-IR-decoder-for-Arduino with Nano3. This was for me very helpful as it gave header info and a nice 0/1 bit structure makes it easier for me to analyse single bits. See in this sheet, may it is helpful for others too.
https://docs.google.com/spreadsheets/d/1j4H1TDJ2QRFLPnls_sxkTZ3c9922TWxqIxwyGWwBIn0/edit?usp=sharing
I can help further if other code records are needed. I think last 5% of analytics are necessary.
My workaround in absense of impleted protocol is working with Tasmota IR raw send command like :
Irsend raw,0,12024,5096,550,484,1910,010101010010001100000000000001010000000000000000001100011000100000110110
Thanks
The text was updated successfully, but these errors were encountered: