From 4bb802362babdb4321dcca62ec2841b0b1319d8f Mon Sep 17 00:00:00 2001
From: Robin Mueller <muellerr@irs.uni-stuttgart.de>
Date: Thu, 9 Feb 2023 17:32:40 +0100
Subject: [PATCH] disable thermal ctrl loop, throws too many useless events

---
 bsp_hosted/scheduling.cpp                |  9 ++--
 mission/controller/ThermalController.cpp | 52 +++++++++++++-----------
 mission/controller/ThermalController.h   |  1 +
 3 files changed, 33 insertions(+), 29 deletions(-)

diff --git a/bsp_hosted/scheduling.cpp b/bsp_hosted/scheduling.cpp
index cb97a27f..843b2892 100644
--- a/bsp_hosted/scheduling.cpp
+++ b/bsp_hosted/scheduling.cpp
@@ -154,11 +154,10 @@ void scheduling::initTasks() {
     scheduling::printAddObjectError("Core controller dummy", objects::CORE_CONTROLLER);
   }
 
-  // Disabled until more stable
-  // result = thermalTask->addComponent(objects::THERMAL_CONTROLLER);
-  // if (result != returnvalue::OK) {
-  //   scheduling::printAddObjectError("THERMAL_CONTROLLER", objects::THERMAL_CONTROLLER);
-  // }
+  result = thermalTask->addComponent(objects::THERMAL_CONTROLLER);
+  if (result != returnvalue::OK) {
+    scheduling::printAddObjectError("THERMAL_CONTROLLER", objects::THERMAL_CONTROLLER);
+  }
 
   FixedTimeslotTaskIF* pstTask = factory->createFixedTimeslotTask(
       "DUMMY_PST", 75, PeriodicTaskIF::MINIMUM_STACK_SIZE * 4, 0.5, missedDeadlineFunc);
diff --git a/mission/controller/ThermalController.cpp b/mission/controller/ThermalController.cpp
index 9ff3b7d8..947119ad 100644
--- a/mission/controller/ThermalController.cpp
+++ b/mission/controller/ThermalController.cpp
@@ -109,30 +109,7 @@ void ThermalController::performControlOperation() {
     deviceTemperatures.commit();
   }
 
-  ctrlCameraBody();
-  ctrlAcsBoard();
-  ctrlMgt();
-  ctrlRw();
-  ctrlStr();
-  ctrlIfBoard();
-  ctrlAcsBoard();
-  ctrlObc();
-  ctrlObcIfBoard();
-  ctrlSBandTransceiver();
-  ctrlPcduP60Board();
-  ctrlPcduAcu();
-  ctrlPcduPdu();
-  ctrlPlPcduBoard();
-  ctrlPlocMissionBoard();
-  ctrlPlocProcessingBoard();
-  ctrlDac();
-
-  ctrlDro();
-  ctrlX8();
-  ctrlHpa();
-  ctrlTx();
-  ctrlMpa();
-  ctrlScexBoard();
+  //performThermalModuleCtrl();
 }
 
 ReturnValue_t ThermalController::initializeLocalDataPool(localpool::DataPool& localDataPoolMap,
@@ -1338,6 +1315,33 @@ void ThermalController::ctrlHpa() {
   }
 }
 
+void ThermalController::performThermalModuleCtrl() {
+  ctrlCameraBody();
+  ctrlAcsBoard();
+  ctrlMgt();
+  ctrlRw();
+  ctrlStr();
+  ctrlIfBoard();
+  ctrlAcsBoard();
+  ctrlObc();
+  ctrlObcIfBoard();
+  ctrlSBandTransceiver();
+  ctrlPcduP60Board();
+  ctrlPcduAcu();
+  ctrlPcduPdu();
+  ctrlPlPcduBoard();
+  ctrlPlocMissionBoard();
+  ctrlPlocProcessingBoard();
+  ctrlDac();
+
+  ctrlDro();
+  ctrlX8();
+  ctrlHpa();
+  ctrlTx();
+  ctrlMpa();
+  ctrlScexBoard();
+}
+
 void ThermalController::ctrlScexBoard() {
   sensors[0].first = sensorTemperatures.sensor_scex.isValid();
   sensors[0].second = sensorTemperatures.sensor_scex.value;
diff --git a/mission/controller/ThermalController.h b/mission/controller/ThermalController.h
index 79ef4a95..03c7954c 100644
--- a/mission/controller/ThermalController.h
+++ b/mission/controller/ThermalController.h
@@ -44,6 +44,7 @@ class ThermalController : public ExtendedControllerBase {
   ReturnValue_t initialize() override;
 
  protected:
+  void performThermalModuleCtrl();
   ReturnValue_t handleCommandMessage(CommandMessage* message) override;
   void performControlOperation() override;
   ReturnValue_t initializeLocalDataPool(localpool::DataPool& localDataPoolMap,