remove "plain MAP" mode (#1398)

* consumers

* enums, input, config

* java

* default config cleanup
This commit is contained in:
Matthew Kennedy 2020-05-05 12:04:55 -07:00 committed by GitHub
parent 573d6d2d55
commit 10903c4659
8 changed files with 2 additions and 65 deletions

View File

@ -20,62 +20,12 @@
EXTERN_CONFIG; EXTERN_CONFIG;
static const fuel_table_t default_139qmb_fuel_table = {
{ 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000 }, { 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000 }, { 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000 }, { 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000 }, { 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000 },
{ 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000 }, { 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000 }, { 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000 }, { 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000 }, { 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000 }, { 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000 }, { 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000 }, { 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000 }, { 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000 }, { 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000 }, { 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000, 6.000000, 6.000000, 6.000000, 6.000000, 6.000000,
6.000000 }
};
static void setDefault139qmbMaps(DECLARE_CONFIG_PARAMETER_SIGNATURE) { static void setDefault139qmbMaps(DECLARE_CONFIG_PARAMETER_SIGNATURE) {
setFuelLoadBin(1.2, 4.4 PASS_CONFIG_PARAMETER_SUFFIX); setFuelLoadBin(1.2, 4.4 PASS_CONFIG_PARAMETER_SUFFIX);
setFuelRpmBin(1000, 11000 PASS_CONFIG_PARAMETER_SUFFIX); setFuelRpmBin(1000, 11000 PASS_CONFIG_PARAMETER_SUFFIX);
setTimingLoadBin(1.2, 4.4 PASS_CONFIG_PARAMETER_SUFFIX); setTimingLoadBin(1.2, 4.4 PASS_CONFIG_PARAMETER_SUFFIX);
setTimingRpmBin(1000, 11000 PASS_CONFIG_PARAMETER_SUFFIX); setTimingRpmBin(1000, 11000 PASS_CONFIG_PARAMETER_SUFFIX);
copyFuelTable(default_139qmb_fuel_table, config->fuelTable);
//copyTimingTable(default_139qmb_timing_table, config->ignitionTable);
} }
void setGy6139qmbDefaultEngineConfiguration(DECLARE_CONFIG_PARAMETER_SIGNATURE) { void setGy6139qmbDefaultEngineConfiguration(DECLARE_CONFIG_PARAMETER_SIGNATURE) {
@ -87,7 +37,6 @@ void setGy6139qmbDefaultEngineConfiguration(DECLARE_CONFIG_PARAMETER_SIGNATURE)
engineConfiguration->cranking.rpm = 1100; engineConfiguration->cranking.rpm = 1100;
setTargetRpmCurve(2000 PASS_CONFIG_PARAMETER_SUFFIX); setTargetRpmCurve(2000 PASS_CONFIG_PARAMETER_SUFFIX);
engineConfiguration->analogInputDividerCoefficient = 1; engineConfiguration->analogInputDividerCoefficient = 1;
engineConfiguration->fuelAlgorithm = LM_MAP;
engineConfiguration->globalTriggerAngleOffset = 45; engineConfiguration->globalTriggerAngleOffset = 45;
engineConfiguration->sensorChartMode = SC_MAP; engineConfiguration->sensorChartMode = SC_MAP;
engineConfiguration->specs.displacement = 0.072; // 72cc engineConfiguration->specs.displacement = 0.072; // 72cc

View File

@ -368,8 +368,7 @@ float getInitialAdvance(int rpm, float map, float advanceMax) {
* this method builds a good-enough base timing advance map bases on a number of heuristics * this method builds a good-enough base timing advance map bases on a number of heuristics
*/ */
void buildTimingMap(float advanceMax DECLARE_CONFIG_PARAMETER_SUFFIX) { void buildTimingMap(float advanceMax DECLARE_CONFIG_PARAMETER_SUFFIX) {
if (engineConfiguration->fuelAlgorithm != LM_SPEED_DENSITY && if (engineConfiguration->fuelAlgorithm != LM_SPEED_DENSITY) {
engineConfiguration->fuelAlgorithm != LM_MAP) {
warning(CUSTOM_WRONG_ALGORITHM, "wrong algorithm for MAP-based timing"); warning(CUSTOM_WRONG_ALGORITHM, "wrong algorithm for MAP-based timing");
return; return;
} }

View File

@ -660,8 +660,6 @@ case Force_4_bytes_size_engine_load_mode:
return "Force_4_bytes_size_engine_load_mode"; return "Force_4_bytes_size_engine_load_mode";
case LM_ALPHA_N: case LM_ALPHA_N:
return "LM_ALPHA_N"; return "LM_ALPHA_N";
case LM_MAP:
return "LM_MAP";
case LM_PLAIN_MAF: case LM_PLAIN_MAF:
return "LM_PLAIN_MAF"; return "LM_PLAIN_MAF";
case LM_REAL_MAF: case LM_REAL_MAF:

View File

@ -405,10 +405,6 @@ typedef enum {
* Throttle Position Sensor value is used as engine load. http://en.wikipedia.org/wiki/Throttle_position_sensor * Throttle Position Sensor value is used as engine load. http://en.wikipedia.org/wiki/Throttle_position_sensor
*/ */
LM_ALPHA_N = 1, LM_ALPHA_N = 1,
/**
* raw Manifold Absolute Pressure sensor value is used as engine load http://en.wikipedia.org/wiki/MAP_sensor
*/
LM_MAP = 2,
/** /**
* Speed Density algorithm - Engine Load is a function of MAP, VE and target AFR * Speed Density algorithm - Engine Load is a function of MAP, VE and target AFR
* http://articles.sae.org/8539/ * http://articles.sae.org/8539/

View File

@ -65,9 +65,6 @@ float getEngineLoadT(DECLARE_ENGINE_PARAMETER_SIGNATURE) {
} }
return getMafVoltage(PASS_ENGINE_PARAMETER_SIGNATURE); return getMafVoltage(PASS_ENGINE_PARAMETER_SIGNATURE);
case LM_SPEED_DENSITY: case LM_SPEED_DENSITY:
// SD engine load is used for timing lookup but not for fuel calculation,
// so fall thru to the MAP case.
case LM_MAP:
return getMap(PASS_ENGINE_PARAMETER_SIGNATURE); return getMap(PASS_ENGINE_PARAMETER_SIGNATURE);
case LM_ALPHA_N: case LM_ALPHA_N:
return Sensor::get(SensorType::Tps1).value_or(0); return Sensor::get(SensorType::Tps1).value_or(0);

View File

@ -447,7 +447,7 @@ int sensorSnifferRpmThreshold;+Disable sensor sniffer above this rpm;"RPM",
int rpmHardLimit;set rpm_hard_limit X;"rpm", 1, 0, 0, 20000.0, 2 int rpmHardLimit;set rpm_hard_limit X;"rpm", 1, 0, 0, 20000.0, 2
#define engine_load_mode_e_enum "MAF", "Alpha-N/TPS", "MAP", "SPEED DENSITY", "MAF Air Charge" #define engine_load_mode_e_enum "MAF", "Alpha-N/TPS", "INVALID", "SPEED DENSITY", "MAF Air Charge"
custom engine_load_mode_e 4 bits, U32, @OFFSET@, [0:7], @@engine_load_mode_e_enum@@ custom engine_load_mode_e 4 bits, U32, @OFFSET@, [0:7], @@engine_load_mode_e_enum@@

View File

@ -366,7 +366,6 @@ fileVersion = { @@TS_FILE_VERSION@@ }
; todo: generate this section programatically ; todo: generate this section programatically
LM_PLAIN_MAF = {0}, LM_PLAIN_MAF = {0},
LM_ALPHA_N = {1}, LM_ALPHA_N = {1},
LM_MAP = {2},
LM_SPEED_DENSITY = {3}, LM_SPEED_DENSITY = {3},
LM_REAL_MAF = {4} LM_REAL_MAF = {4}

View File

@ -94,7 +94,6 @@ public class FormulasPane {
enum engine_load_mode_e { enum engine_load_mode_e {
LM_PLAIN_MAF("Plain MAF"), LM_PLAIN_MAF("Plain MAF"),
LM_ALPHA_N("Alpha-N/TPS"), LM_ALPHA_N("Alpha-N/TPS"),
LM_MAP("Plain MAP"),
LM_SPEED_DENSITY("Speed Density/MAP"), LM_SPEED_DENSITY("Speed Density/MAP"),
LM_REAL_MAF("MAF"); LM_REAL_MAF("MAF");