techie66 / EngineDataLogger

Data logger and master controller for Raspberry Pi based ECU that communicates with various external MCUs
GNU General Public License v3.0
5 stars 0 forks source link

RPM gap #5

Closed techie66 closed 3 years ago

techie66 commented 3 years ago

Not sure if this belongs here or in libIgnitech, but the exported RPM has an odd gap in it. between 3325-3575 (250 rpm) the recorded rpm is lower than actual by 750.

techie66 commented 3 years ago

so far, testing indicates that the program registers RPM in that range correctly. I'll have to get some data dumps. This might be a firmware bug with the Ignitech.

techie66 commented 3 years ago

Serial dumps collected. The firmware sends checksum-correct packets that contain bogus data. https://github.com/techie66/libIgnitech/commit/2a6b783fcb30869fcbe898ec6e7730b40b9b19ca works to identify these inconsistencies and set IGN_BAD. 00998e7b683cd78c8c982c80279bc4396373a630 works to only use good data.

techie66 commented 3 years ago

More testing of libIgnitech uncovered a serious bug in tty termios settings that resulted in certain control characters getting translated.