Trigger setup in TS is highly confusing: hide operation mode from users? #4031
This commit is contained in:
parent
2bc1215f86
commit
a43aec4e60
|
@ -28,6 +28,8 @@ void setGy6139qmbDefaultEngineConfiguration() {
|
|||
engineConfiguration->specs.displacement = 0.072; // 72cc
|
||||
engineConfiguration->specs.cylindersCount = 1;
|
||||
setOperationMode(engineConfiguration, FOUR_STROKE_CRANK_SENSOR);
|
||||
// this is related to 'setDefaultBaseEngine' having 'skippedWheelOnCam = true' which is a weird fact by itself
|
||||
engineConfiguration->skippedWheelOnCam = false;
|
||||
|
||||
engineConfiguration->specs.firingOrder = FO_1;
|
||||
|
||||
|
|
|
@ -55,6 +55,8 @@ void setTestCrankEngineConfiguration() {
|
|||
engineConfiguration->trigger.type = TT_ONE;
|
||||
|
||||
setOperationMode(engineConfiguration, FOUR_STROKE_CRANK_SENSOR);
|
||||
// this is related to 'setDefaultBaseEngine' having 'skippedWheelOnCam = true' which is a weird fact by itself
|
||||
engineConfiguration->skippedWheelOnCam = false;
|
||||
}
|
||||
|
||||
void setTestVVTEngineConfiguration() {
|
||||
|
|
|
@ -34,7 +34,9 @@ void setDefaultBaseEngine() {
|
|||
|
||||
// Trigger
|
||||
engineConfiguration->trigger.type = TT_TOOTHED_WHEEL_60_2;
|
||||
// huh WAT? our default is skipped on CAM?! and *many* our tests are relying on that?!
|
||||
setOperationMode(engineConfiguration, FOUR_STROKE_CAM_SENSOR);
|
||||
engineConfiguration->skippedWheelOnCam = true;
|
||||
engineConfiguration->useOnlyRisingEdgeForTrigger = false;
|
||||
|
||||
engineConfiguration->globalTriggerAngleOffset = 0;
|
||||
|
|
Loading…
Reference in New Issue