pimatic / RFControl

433mhz rf sniffer and receiver
GNU General Public License v3.0
117 stars 34 forks source link

verification does not allow packets #23

Open Links2004 opened 6 years ago

Links2004 commented 6 years ago

Hi,

when running the examples I am not able to see my RF remote. after some debugging I found out that the verification do not allow the packets. currently the verification process is a big mystery to me.

logic scope of one packet:

image

I modified the class to allow to dump the full buffer:

void RFControl::getRealRaw(unsigned int **buffer, unsigned int* timings_size) {
    *buffer = &timings[0];
    *timings_size = MAX_RECORDINGS;
}

the dumped values are:

3624, 1056, 980, 452, 480, 952, 472, 468, 492, 468, 952, 944, 960, 468, 472, 968,
460, 488, 456, 484, 948, 972, 456, 476, 956, 952, 936, 976, 444, 496, 936, 984,
916, 984, 920, 516, 444, 972, 924, 980, 928, 968, 932, 976, 452, 492, 456, 496,
924, 984, 924, 984, 444, 516, 416, 512, 928, 500, 444, 504, 448, 984, 920, 504,
448, 528, 420, 55216, 3612, 1080, 956, 460, 484, 956, 480, 456, 492, 468, 956, 960,
948, 468, 472, 952, 476, 488, 456, 476, 944, 980, 444, 480, 948, 956, 940, 972,
464, 484, 936, 968, 924, 984, 932, 488, 460, 976, 916, 988, 920, 984, 924, 972,
448, 516, 432, 516, 904, 984, 932, 984, 436, 516, 428, 516, 924, 492, 444, 532,
432, 984, 908, 512, 440, 520, 432, 55216, 3616, 1076, 952, 468, 476, 952, 480, 468,
480, 472, 948, 960, 948, 468, 472, 960, 468, 484, 460, 480, 944, 976, 448, 488,
948, 948, 948, 984, 436, 492, 936, 976, 924, 992, 904, 512, 448, 972, 828, 1100,
912, 960, 948, 956, 456, 500, 456, 504, 924, 964, 936, 992, 440, 516, 424, 516,
912, 496, 312, 656, 436, 992, 904, 524, 432, 524, 424, 30048, 108, 63924, 108, 144,
124, 268, 124, 144, 168, 164, 120, 140, 180, 160, 120, 148, 144, 148, 120, 148,
180, 144, 136, 288, 124, 1096, 100, 232, 144, 144, 108, 700, 204, 148, 1296, 1008,
152, 528, 120, 132, 236, 144, 108, 1196, 228, 192, 112, 144, 100, 984, 1164, 132,
112, 968, 648, 132, 112, 140, 208, 152, 1480, 1392, 1944, 1380, 1476, 1864, 1948, 900,
1960, 1860, 1692, 136, 112, 1376, 1016, 528, 192, 516, 184, 984, 300, 180, 132, 156,
140, 976, 124, 144, 188, 616, 308, 1476, 124, 352, 136, 216, 732, 1296, 184, 804,
392, 552, 408, 516, 216, 54948, 3604, 1076, 972, 452, 480, 960, 480, 456, 480, 496,
936, 948, 948, 468, 480, 960, 460, 508, 448, 468, 944, 964, 464, 476, 960, 968,
924, 988, 436, 504, 936, 968, 920, 988, 912, 508, 452, 980, 336, 1560, 748, 1148,
928, 972, 456, 524, 428, 524, 892, 984, 576, 1356, 416, 516, 432, 516, 924, 508,
136, 808, 440, 992, 292, 1136, 432, 528, 292, 55360, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 

the raw data looks good to me.

side note: The data encoding is like:

  __ __       __    __ __    __
 |     |     |  |  |     |  |  |
_|     |__ __|  |__|     |__|  |__.....
 |  0  |  0  |  1  |  0  |  1  |
31warren commented 5 years ago

I am trying to get RF Control to work on an UNO or Mega and have tried your modification - still getting nothing displayed in Serial Monitor. The hardware/sensors work ok with other libraries e.g. RC_Switch.

Any suggestions?? Also, do I need to add anything other than the above modification plus addition to the header file and reference the new function in the ino file. Thanks