rusefi
/
custom-board-bundle-sample-fork-of-rusefi
mirror of https://github.com/rusefi/custom-board-bundle-sample-fork-of-rusefi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Set sensible default for dwell by voltage multiplier table fix #4067

This commit is contained in:
rusefillc 2022-04-13 15:21:31 -04:00
parent 0baa5a3e7a
commit d2205a7fd6
3 changed files with 19 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
firmware/controllers/algo/defaults/default_ignition.cpp
View File

@ -98,6 +98,9 @@ void setDefaultIgnition() {
// Dwell table
setConstantDwell(4);
setLinearCurve(engineConfiguration->dwellVoltageCorrVoltBins, 8, 12, 0.1);
setLinearCurve(engineConfiguration->dwellVoltageCorrValues, 1, 1, 1);
// Multispark
setDefaultMultisparkParameters();

26
firmware/controllers/engine_controller.cpp
View File

@ -600,13 +600,11 @@ bool validateConfig() {
ensureArrayIsAscending("Ignition IAT corr RPM", config->ignitionIatCorrRpmBins);
}
if (config->mapEstimateTpsBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Map estimate TPS", config->mapEstimateTpsBins);
}
ensureArrayIsAscendingOrDefault("Map estimate TPS", config->mapEstimateTpsBins);
ensureArrayIsAscendingOrDefault("Map estimate RPM", config->mapEstimateRpmBins);
if (config->mapEstimateRpmBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Map estimate RPM", config->mapEstimateRpmBins);
}
ensureArrayIsAscendingOrDefault("Dwell Correction Voltage", engineConfiguration->dwellVoltageCorrVoltBins);
ensureArrayIsAscendingOrDefault("Dwell Correction Values", engineConfiguration->dwellVoltageCorrValues);
ensureArrayIsAscending("MAF decoding", config->mafDecodingBins);
@ -618,18 +616,10 @@ bool validateConfig() {
// Idle tables
ensureArrayIsAscending("Idle target RPM", engineConfiguration->cltIdleRpmBins);
ensureArrayIsAscending("Idle warmup mult", config->cltIdleCorrBins);
if (engineConfiguration->iacCoastingBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Idle coasting position", engineConfiguration->iacCoastingBins);
}
if (config->idleVeRpmBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Idle VE RPM", config->idleVeRpmBins);
}
if (config->idleVeLoadBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Idle VE Load", config->idleVeLoadBins);
}
if (config->idleAdvanceBins[1] != 0) { // only validate map if not all zeroes default
ensureArrayIsAscending("Idle timing", config->idleAdvanceBins);
}
ensureArrayIsAscendingOrDefault("Idle coasting position", engineConfiguration->iacCoastingBins);
ensureArrayIsAscendingOrDefault("Idle VE RPM", config->idleVeRpmBins);
ensureArrayIsAscendingOrDefault("Idle VE Load", config->idleVeLoadBins);
ensureArrayIsAscendingOrDefault("Idle timing", config->idleAdvanceBins);
for (size_t index = 0; index < efi::size(engineConfiguration->vrThreshold); index++) {
auto& cfg = engineConfiguration->vrThreshold[index];

8
firmware/util/math/interpolation.h
View File

@ -37,6 +37,14 @@ void ensureArrayIsAscending(const char* msg, const TValue (&values)[TSize]) {
}
}
template<typename TValue, int TSize>
void ensureArrayIsAscendingOrDefault(const char* msg, const TValue (&values)[TSize]) {
if (values[1] == 0) {
return; // looks like default empty array, do not check
}
ensureArrayIsAscending(msg, values);
}
namespace priv {
struct BinResult
{