Minor ERG mode fix.

doudar · Apr 28, 2024 · e66162a · e66162a
1 parent e86e376
commit e66162a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,7 +29,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Increased the default incline multiplier to 5.
 - Added more robust activity monitoring and reboot every 30 minutes if there is no activity.
 - Updated all references of SmartSkin2K to SmartSpin2k for consistency.
-- Fixed bug where BT scanner "Loading" wouldn't disappear if "NONE" and "NONE" were selected. 
+- Fixed bug where BT scanner "Loading" wouldn't disappear if "NONE" and "NONE" were selected.
+- Fixed Bug where ERG setpoint state wasn't going to the positive control loop correctly.  
 
 ### Hardware
 - added Yesoul S3.

diff --git a/platformio.ini b/platformio.ini
@@ -51,7 +51,7 @@ check_flags =
     --suppress=*:*/.pio/*
     --suppress=*:*/lib/NimBLE-Arduino/*
     --suppress=unmatchedSuppression
-    --suppress=missingIncludeSystem
+    --suppress=missingIncludeSystemthrough
 check_severity = medium, high
 check_skip_packages = true
 

diff --git a/src/ERG_Mode.cpp b/src/ERG_Mode.cpp
@@ -572,7 +572,7 @@ void ErgMode::computeErg() {
   }
 
   // SetPoint changed
-  if (abs(this->setPoint - newWatts.getTarget() > 20)) {
+  if (abs(this->setPoint - newWatts.getTarget()) > 20) {
     _setPointChangeState(newCadence, newWatts);
     return;
   }