-
Notifications
You must be signed in to change notification settings - Fork 836
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
Thank you for sharing this project. Thank you for your work. #814
Comments
The state of my A/C remote control operation is ON, and the state of my A/C air conditioner is MODE is ON. In the source code, POWER refers to ON/OFF distinguishing code, not MODE. |
Hi, Thanks for the report. I'm confused as to what you mean by:
What is TTL in this context? https://en.wikipedia.org/wiki/TTL |
The current problem is that the remote control operation of green air conditioner is completely correctly fed back. The only problem is that the power state feedback is off, not the correct on state. |
=====================When I press the "power" button ON================ =======================When I press the "power" button OFF=================== |
Brand: green Remote control model: YBOFB2 |
SmartIRRepeater |
SmartIRRepeater |
HI @1043717432 , previous data indicated the 3rd byte ( I've created a branch that has the fix for your particular circumstances, while leaving things compatible with existing systems .. I hope. Can you please test the branch of the code in https://github.com/crankyoldgit/IRremoteESP8266/tree/Issue814 ? I'll need you to confirm the ESP can power on/off your GREE a/c using the class method (e.g. via the "Aircon" page in the IRMQTTServer example code.) This all could just be your A/C's codes are subtly different from everyone elses, but fingers crossed this will keep everyone happy. |
Thank you. I'll test it right away and give you feedback. |
If you clone/pull/download that branch and compile using it, IRrecvDumpV2 and IRMQTTServer should now detect the Power on/off correctly, and hopefully be able to control (construct and send new messages) the power functions of your A/C. |
No idea where you are getting this output from. |
putty com |
@crankyoldgit Thank you. The 64 different original codes I got by using various device learning buttons are all in ON state. Using your test file for compilation, the problem still exists. The key states ON and OFF cannot be distinguished clearly.
|
Putty is just a terminal program. I meant, what program is running on the ESP. It is NOT |
I don't understand what you are saying above.
Again, I'm not sure what you are saying here. The tests I added were based on the data you supplied, and due to the changes I made, it should now detect correctly. If it doesn't, then I think you may not be building the binary using the updated library.
I have no idea what this is from. It certainly isn't from something in our library. |
No. I meant please try to send an "on" command via the "Aircon" menu/page. Not the home page. |
The test failed. Under the Aircon menu, Gree is used to send'on'. The air conditioner has a'beep', but on turns off. And there was no response to send'OFF'. |
* Add set/getModel(). e.g. YAW1F (1) [default] & YBOFB2 (2) * Automatic detection of models only works with messages with power on. * set/getPower() now changes behaviour with different models. * Report model type in the .toString() function. * Update common a/c routines and unit tests. * Unit test coverage for different power/model operation. Fixes #814
I've updated https://github.com/crankyoldgit/IRremoteESP8266/tree/Issue814 to support different models of GREE a/c.
|
Thank you for your kindness. It's perfect. Air conditioning brand: green Remote control model: YBOFB/YAPOF3 |
Thanks for confirming it is working as expected. |
Wrong. The remote control forgot to bring it. I'll try it again tomorrow. |
It is correct to send on to open A/C on a web page. However, when remote control opens A/C, the web page displays OFF. |
Again, can you please capture & document the same settings/sequences you want via your IR remote, and capture it via I'm pretty sure the Kelvinator protocol is correct (as it is an aircon I have) but your's may be subtly different in some way. |
Thank you. The Kelvinator protocol is correct, and it's perfect for Gree A/C operations. Swing (V) and light are perfect. Remote control model: YAPOF3 perfect But for: YBOF2 currently operates Swing (V), IRMQ TServer, web Aircon status Swing (V) is always off when remotely controlling any operation. |
Can you please be more verbose in your explanations of the issues you are having? I'm finding it very difficult to understand what your problems are. Google Translate is probably not helping matters, but the more descriptive & detailed you can be the more context I have to work on to understand you and thus, help you. Again, can you please capture & document the same settings/sequences you want via your IR remote, and capture it via IRrecvDumpV2? Without data, it is hard to see what is going wrong. The more information you give me, the faster & better the response from me will be. |
I'm sorry, I'm Chinese, and my English expression is not good. I'll test it again and give you feedback. Thank you. |
I'm not faulting your English, as it is far far better than my Chinese/Mandarin/Cantonese/etc. ;-) I'm just trying to save frustration and improve our communication level and speed. |
* Add set/getModel(). e.g. YAW1F (1) [default] & YBOFB2 (2) * Automatic detection of models only works with messages with power on. * set/getPower() now changes behaviour with different models. * Report model type in the .toString() function. * Update common a/c routines and unit tests. * Unit test coverage for different power/model operation. Fixes #814
Chasing this up. |
===========When the ON button is pressed,============== Encoding : GREE uint16_t rawData[139] = {8996, 4476, 650, 1648, 650, 554, 648, 552, 648, 1648, 650, 1650, 648, 554, 648, 554, 648, 556, 646, 554, 648, 1650, 648, 554, 648, 1650, 652, 552, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 556, 646, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 1650, 648, 554, 648, 1650, 648, 554, 648, 552, 648, 1648, 648, 554, 648, 19986, 650, 556, 646, 554, 648, 554, 650, 552, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 650, 554, 648, 554, 648, 1648, 650, 554, 646, 554, 648, 554, 650, 554, 648, 554, 648, 554, 650, 552, 648, 554, 648, 554, 648, 554, 650, 552, 648, 554, 648, 554, 648, 552, 650, 1648, 650, 1650, 650, 1648, 650, 1648, 648}; // GREE Encoding : GREE uint16_t rawData[139] = {8996, 4476, 650, 1648, 650, 554, 648, 554, 650, 1648, 648, 1650, 648, 554, 648, 554, 648, 554, 648, 554, 648, 1650, 648, 554, 648, 1650, 648, 554, 648, 554, 648, 556, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 552, 650, 1648, 650, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 1648, 648, 554, 648, 1648, 650, 554, 648, 554, 648, 1648, 650, 554, 650, 19984, 650, 552, 650, 552, 650, 554, 648, 554, 650, 552, 650, 552, 648, 554, 648, 552, 650, 554, 650, 552, 648, 554, 648, 554, 648, 554, 648, 1648, 650, 552, 650, 554, 650, 552, 648, 554, 648, 554, 648, 552, 650, 552, 650, 554, 650, 552, 648, 552, 650, 552, 650, 552, 650, 552, 650, 552, 650, 1648, 650, 1648, 652, 1646, 650, 1648, 650}; // GREE Encoding : GREE uint16_t rawData[139] = {8998, 4474, 650, 1648, 650, 552, 648, 554, 648, 1648, 650, 1648, 648, 554, 648, 556, 648, 554, 648, 556, 646, 1650, 648, 554, 648, 1650, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 646, 556, 646, 554, 648, 556, 646, 556, 646, 556, 646, 558, 646, 554, 648, 554, 646, 556, 646, 1650, 648, 556, 646, 1650, 648, 554, 648, 554, 648, 1650, 648, 556, 646, 19988, 650, 554, 648, 556, 646, 554, 648, 554, 648, 554, 646, 554, 648, 554, 648, 552, 648, 556, 646, 554, 648, 554, 648, 552, 648, 554, 648, 1650, 650, 554, 648, 552, 650, 554, 648, 552, 650, 554, 648, 552, 650, 552, 650, 552, 648, 554, 648, 552, 650, 552, 648, 554, 648, 552, 650, 552, 650, 1648, 650, 1646, 650, 1648, 650, 1646, 650}; // GREE Encoding : GREE uint16_t rawData[139] = {8968, 4504, 622, 1676, 622, 580, 620, 582, 644, 1652, 622, 1676, 622, 580, 620, 582, 622, 1676, 622, 582, 620, 1676, 622, 582, 620, 1678, 620, 582, 620, 580, 622, 580, 622, 580, 622, 580, 622, 580, 620, 582, 622, 580, 622, 580, 622, 580, 622, 580, 620, 582, 646, 556, 620, 582, 622, 580, 620, 582, 622, 1674, 624, 580, 622, 1676, 624, 580, 620, 580, 646, 1652, 622, 580, 622, 20012, 624, 580, 646, 554, 646, 556, 646, 556, 622, 580, 646, 556, 622, 580, 622, 580, 622, 580, 622, 580, 646, 556, 648, 554, 622, 580, 622, 1674, 646, 556, 622, 580, 648, 554, 646, 556, 622, 580, 622, 580, 622, 580, 646, 554, 648, 554, 648, 554, 622, 580, 646, 554, 646, 556, 622, 580, 622, 1674, 622, 1676, 622, 1676, 624, 1672, 624}; // GREE Encoding : GREE uint16_t rawData[139] = {8996, 4476, 650, 1648, 650, 556, 646, 554, 650, 1648, 650, 1650, 648, 554, 648, 554, 648, 554, 648, 554, 648, 1648, 648, 554, 648, 1650, 650, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 648, 554, 646, 556, 648, 554, 648, 554, 648, 554, 648, 556, 646, 556, 648, 554, 646, 556, 648, 556, 646, 1650, 646, 558, 646, 1650, 648, 556, 646, 556, 646, 1650, 648, 556, 646, 19990, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 646, 556, 648, 554, 646, 554, 648, 554, 646, 1650, 650, 554, 646, 556, 646, 556, 646, 554, 648, 554, 648, 556, 646, 554, 648, 556, 646, 556, 648, 554, 648, 556, 648, 556, 646, 556, 648, 554, 646, 1650, 650, 1650, 648, 1648, 650, 1650, 648}; // GREE Encoding : GREE uint16_t rawData[139] = {8970, 4504, 622, 1676, 622, 580, 622, 580, 622, 580, 620, 1676, 622, 582, 620, 582, 620, 582, 620, 582, 620, 1676, 624, 580, 622, 1676, 624, 580, 622, 582, 620, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 580, 622, 582, 620, 582, 620, 582, 620, 580, 622, 1674, 622, 582, 620, 1678, 620, 580, 622, 582, 620, 1674, 624, 582, 620, 20012, 622, 582, 620, 582, 622, 580, 620, 580, 622, 582, 620, 580, 622, 582, 620, 580, 622, 580, 622, 582, 620, 580, 622, 580, 622, 582, 620, 1676, 622, 582, 620, 582, 622, 580, 622, 582, 622, 580, 620, 582, 620, 582, 620, 580, 622, 582, 620, 580, 622, 580, 622, 580, 622, 580, 622, 582, 620, 1676, 622, 1676, 622, 1676, 622, 580, 622}; // GREE |
I think I found the bug/mistake. I had Light & Turbo swapped. See: 96c0611 I've updated https://github.com/crankyoldgit/IRremoteESP8266/tree/Issue814, so please download and recompile everything based on that, and let me know how it goes. |
ok,Thank you.! |
··· \Arduino\libraries\IRremoteESP8266-Issue814\src\IRac.cpp:32:25: fatal error: gtest/gtest.h: No such file or directory #include "gtest/gtest.h"
compilation terminated. exit status 1 Or compile IRremoteESP 8266-Issue814 test? |
I have no idea what you have done or are doing. Clearly what ever it is is not right. You should just be downloading it the same as you were doing previously/earlier in this issue. |
Thank you. You can compile it. But the sweeper still can't get the right feedback! |
What do you mean by "Sweeper"? Do you mean the "Swing". As in vertical control of where the air flows? Or do you mean something else? Did the Light/LED thing work at all? |
Or did you mean "Sleep"? |
Just chasing this up. |
* Add set/getModel(). e.g. YAW1F (1) [default] & YBOFB2 (2) * Automatic detection of models only works with messages with power on. * `set/getPower()` now changes behaviour with different models. * Report model type in the .toString() function. * Update common a/c routines and unit tests. * Unit test coverage for different power/model operation. * [BUG] Fix parameter ordering for Gree in common a/c code. For #814
Another chase up. I still don't know what you mean by "sweeper". |
FYI, the code changes thus far have been included in the v2.6.4 release of the library. |
Another ping to chase answers for my questions. |
Closing this due to lack of response and that we seem to have resolved everything else. |
Thank you for sharing this project. Thank you for your work.
When A/C uses TTL monitoring, it is found that the remote control a/c opens the switch on and the TTL status shows off.
Ask for help
Timestamp : 002188.163
Library : v2.6.3
Encoding : GREE
Code : 59072050012000C0 (64 bits)
Mesg Desc.: Power: Off, Mode: 1 (COOL), Temp: 23C, Fan: 1, Turbo: Off, IFeel: Off, WiFi: Off, XFan: Off, Light: On, Sleep: Off, Swing Vertical Mode: Auto, Swing Vertical Pos: 1 (Auto)
Raw Timing[139]:
+ 8928, - 4450, + 642, - 1642, + 640, - 554, + 638, - 554,
+ 640, - 1642, + 666, - 1616, + 666, - 530, + 640, - 1640,
+ 640, - 554, + 640, - 1642, + 640, - 1642, + 640, - 1642,
+ 668, - 528, + 664, - 530, + 662, - 530, + 666, - 528,
+ 640, - 554, + 638, - 554, + 664, - 528, + 640, - 554,
+ 666, - 528, + 664, - 530, + 664, - 1616, + 640, - 554,
+ 638, - 556, + 640, - 554, + 666, - 528, + 664, - 528,
+ 640, - 554, + 664, - 1616, + 666, - 528, + 638, - 1642,
+ 668, - 528, + 666, - 528, + 640, - 1642, + 666, - 528,
+ 640, - 19850, + 644, - 1640, + 642, - 554, + 638, - 554,
+ 640, - 554, + 640, - 554, + 640, - 554, + 640, - 554,
+ 640, - 556, + 638, - 554, + 638, - 556, + 638, - 554,
+ 640, - 554, + 666, - 530, + 640, - 1640, + 640, - 554,
+ 666, - 528, + 666, - 528, + 640, - 554, + 666, - 528,
+ 638, - 554, + 640, - 554, + 664, - 530, + 666, - 528,
+ 640, - 554, + 640, - 554, + 640, - 554, + 640, - 554,
+ 664, - 530, + 640, - 554, + 638, - 554, + 640, - 1642,
+ 666, - 1616, + 666
uint16_t rawData[139] = {8928, 4450, 642, 1642, 640, 554, 638, 554, 640, 1642, 666, 1616, 666, 530, 640, 1640, 640, 554, 640, 1642, 640, 1642, 640, 1642, 668, 528, 664, 530, 662, 530, 666, 528, 640, 554, 638, 554, 664, 528, 640, 554, 666, 528, 664, 530, 664, 1616, 640, 554, 638, 556, 640, 554, 666, 528, 664, 528, 640, 554, 664, 1616, 666, 528, 638, 1642, 668, 528, 666, 528, 640, 1642, 666, 528, 640, 19850, 644, 1640, 642, 554, 638, 554, 640, 554, 640, 554, 640, 554, 640, 554, 640, 556, 638, 554, 638, 556, 638, 554, 640, 554, 666, 530, 640, 1640, 640, 554, 666, 528, 666, 528, 640, 554, 666, 528, 638, 554, 640, 554, 664, 530, 666, 528, 640, 554, 640, 554, 640, 554, 640, 554, 664, 530, 640, 554, 638, 554, 640, 1642, 666, 1616, 666}; // GREE
uint8_t state[8] = {0x59, 0x07, 0x20, 0x50, 0x01, 0x20, 0x00, 0xC0};
Mesg Desc.: Power: Off, The correct display is ON
The text was updated successfully, but these errors were encountered: