rusefi / rusefi

rusefi - GPL internal combustion engine control unit
http://rusefi.com
Other
779 stars 243 forks source link

BRAND_NEW_UNIT flag and fatal error to facilitate initial setup #4420

Open rusefillc opened 2 years ago

rusefillc commented 2 years ago

problem statement: initial setup contributing factor: warnings are invisible

solution: fatal errors have great visibility with clear text nuace: we do not want to fatal while in the corner at the track

proposed solution:

image

rusefillc commented 2 years ago

@BeerMoneyMotorsports @stefanst @ElDominio opinions?

mck1117 commented 2 years ago

special handing of BRAND_NEW_UNIT: fatal error if SD selected AND one second since boot AND map value out of 100kpa range

Having seen cars get limped in off-track after hardware failures/crashing/a spin/whatever (and an ignition cycle!), I'm not a fan of this logic at all. I've also rebooted the car while moving, in gear, making vacuum. My cars also start in far less than a second.

mck1117 commented 2 years ago

It seems like this is not worth adding additional new complicated and probably annoying feature instead of writing an accurate and useful "before first start checklist" document. I am sure several of us would be happy to contribute content to such a document.

rusefillc commented 2 years ago

@mck1117 I am looking at https://github.com/rusefi/rusefi_documentation/issues/227 and it tells me everything I need to know about managing documentation content in the open source setup

more importantly I believe that average users would not read any documentation, including our existing https://github.com/rusefi/rusefi/wiki/First-Engine-Start which has no content relevant for this issue here

rusefillc commented 2 years ago

Having seen cars get limped in off-track after hardware failures/crashing/a spin/whatever (and an ignition cycle!), I'm not a fan of this logic at all. I've also rebooted the car while moving, in gear, making vacuum. My cars also start in far less than a second.

@mck1117 please clarify if I was unclear with "special handing of BRAND_NEW_UNIT: fatal error after 120 seconds of running to force user to turn off BRAND_NEW_UNIT flag manually" or if you did not read that far?

ElDominio commented 2 years ago

Sadly, I don't think this is fixable. Even the best-of-best ECUs always say in the manual RTFM, the program annoys you with RTFM, and no one does RTFM.

I think the most useful way to do something like this is to put an onboard piezo buzzer that doesn't shut up until manually disabled lol

rusefillc commented 2 years ago

image