-
-
Notifications
You must be signed in to change notification settings - Fork 237
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
GM SENT ETB bench test in NJ #4688
Comments
Good output:
|
Definitely good output from fuel pressure sensor:
|
So this looks like "another" CRC. J7216 has defines "New" and "Legacy" CRC calculation methods. |
New CRC calculation compatible with GM ETB is implemented in #4692 |
@dron0gus do you have CRC auto-detection or do you want a new configuration setting for CRC mode choice? |
idle position "small" 12639457AA
ETB held CLOSED
etb held OPEN
|
larger 12617792 is also happy idle position
forced closed
force open
|
|
|
lua
|
Ford part https://rusefi.com/forum/viewtopic.php?p=46584#p46584 idle state
Note 13% CRC error rate? fully closed causes restats
fully open
|
Time to implement CRC check on slow channels... |
LUA code:
|
|
SENT ETB starts to make some noise #4735 |
master as is works perfectly with the following magic:
that set of Lua sensors works with the current logic as in master |
Full script (please ignore CAN bus, please note timeout/tickrate weirdness)
|
Oooops I've broken GM, just fixed it
|
I think we are mission accomplished here see https://rusefi.com/forum/viewtopic.php?p=44982#p44982 |
Trying to get GM ETB with MRE in NJ
at the moment use master, MRE 0.6.0 with C54 removed
The text was updated successfully, but these errors were encountered: