Merge pull request #566 from doudar/remove-temp

Removed temperature checks

CHANGELOG.md

@@ -42,6 +42,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Removed extra logging when loading table.
 - Prevent table returns from going in the wrong direction.
 - Fixed bug with stepper speed not updating.
+- Removed driver temp checking. It's not accurate on the ESP32. 
 
 
 ### Hardware

README.md

@@ -59,9 +59,11 @@ Everything you need to know is in the [SS2K Wiki](https://github.com/doudar/Smar
 ## Contributing
 1. Install Microsoft Visual Code. https://code.visualstudio.com/
 2. Install PlatformIO into Visual Code. https://platformio.org/platformio-ide
-3. Install Pre-commit. https://pre-commit.com
-4. Install pre-push git hooks: `pre-commit install --hook-type pre-push`
-5. Open this project in PlatformIO.
+3. Install Python https://www.python.org/downloads/
+4. Install Pre-commit. https://pre-commit.com `py -m pip install pre-commit`
+5. Follow any install warnings to add pre-commit to your path. 
+6. Install pre-push git hooks: `pre-commit install --hook-type pre-push`
+7. Open this project in PlatformIO.
 
 ## Important announcement for existing users
 If you experience an issue after upgrading firmware, please follow these instructions in the Wiki. Flashing instructions are available [HERE.](https://github.com/doudar/SmartSpin2k/wiki/Loading-Software)

src/Main.cpp

@@ -275,7 +275,9 @@ void SS2K::maintenanceLoop(void *pvParameters) {
 
     // Things to do every 20 loops
     if (loopCounter > 20) {
-      ss2k->checkDriverTemperature();
+     // Removed driver temp checking. This really doesn't do anything benificial anyway, because the thermistors in the ESP32 are not accurate.
+     // The Driver itsself will throttle automatically if the temp is too high. 
+     // ss2k->checkDriverTemperature();
 
 #ifdef DEBUG_STACK
       Serial.printf("Step Task: %d \n", uxTaskGetStackHighWaterMark(moveStepperTask));