some strange pid limits

(cherry picked from commit bd45ebe88a)
This commit is contained in:
Matthew Kennedy 2023-05-05 17:39:16 -07:00 committed by rusefillc
parent 670ce4a1f5
commit 9b84d2a460
3 changed files with 5 additions and 4 deletions

View File

@ -29,7 +29,8 @@ static void commonNA8() {
engineConfiguration->idle_derivativeFilterLoss = 0.08;
engineConfiguration->idle_antiwindupFreq = 0.03;
engineConfiguration->idleRpmPid.dFactor = 0.002;
engineConfiguration->idleRpmPid.minValue = 76;
engineConfiguration->idleRpmPid.minValue = -20;
engineConfiguration->idleRpmPid.maxValue = 20;
engineConfiguration->idlerpmpid_iTermMin = -15;
engineConfiguration->idlerpmpid_iTermMax = 30;
engineConfiguration->acIdleExtraOffset = 15;

View File

@ -279,7 +279,7 @@ static void setCommonMazdaNB() {
engineConfiguration->idle_antiwindupFreq = 0.03;
engineConfiguration->idleRpmPid.dFactor = 0.002;
engineConfiguration->idleRpmPid.minValue = -8;
engineConfiguration->idleRpmPid.minValue = 76;
engineConfiguration->idleRpmPid.maxValue = 10;
engineConfiguration->idlerpmpid_iTermMin = -15;
engineConfiguration->idlerpmpid_iTermMax = 30;

View File

@ -530,8 +530,8 @@ static void setDefaultEngineConfiguration() {
engineConfiguration->idleRpmPid.pFactor = 0.05;
engineConfiguration->idleRpmPid.iFactor = 0.002;
engineConfiguration->idleRpmPid.minValue = 0;
engineConfiguration->idleRpmPid.maxValue = 99;
engineConfiguration->idleRpmPid.minValue = -20;
engineConfiguration->idleRpmPid.maxValue = 20;
/**
* between variation between different sensor and weather and fabrication tolerance
* five percent looks like a safer default