-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Disable DFCO when clutch is applied #487
Conversation
@@ -71,6 +71,8 @@ enum class SensorType : unsigned char { | |||
|
|||
BarometricPressure, | |||
|
|||
ClutchUp, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Initially thought I needed to plumb it through before I saw that the digital inputs were getting set in engine.cpp. This can/should be removed
@@ -109,6 +109,7 @@ static void setDefaultFuelCutParameters() { | |||
engineConfiguration->coastingFuelCutTps = 2; | |||
engineConfiguration->coastingFuelCutMap = 30; | |||
engineConfiguration->coastingFuelCutClt = 60; | |||
engineConfiguration->disableFuelCutOnClutch = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By the way, you don't have to set any defaults that have a bit value of 0. The configuration is zeroed before reset, so everything defaults to 0/false/etc.
Allow clutchUp switch to be a gating term for fuel cut activation. Enables feature requested in #22