From 7b611810df4af42e130e09ef592922cdd8198c89 Mon Sep 17 00:00:00 2001
From: Robin Mueller <robin.mueller.m@gmail.com>
Date: Thu, 6 Jul 2023 20:13:33 +0200
Subject: [PATCH] there are actually 8 heaters

---
 CHANGELOG.md                             | 2 ++
 mission/controller/ThermalController.cpp | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 08f904a9..a74c99fd 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -27,6 +27,8 @@ will consitute of a breaking change warranting a new major release:
 
 - TMP1075: Set dataset invalid on shutdown explicitely
 - Small fixes for TMP1075 FDIR: Use strange and missed reply counters.
+- TCS controller: Last heater (S-band heater) was skipped for transition completion
+  checks.
 
 # [v6.0.0] 2023-07-02
 
diff --git a/mission/controller/ThermalController.cpp b/mission/controller/ThermalController.cpp
index fca71acc..3f6b2995 100644
--- a/mission/controller/ThermalController.cpp
+++ b/mission/controller/ThermalController.cpp
@@ -1751,7 +1751,7 @@ void ThermalController::resetSensorsArray() {
 }
 
 void ThermalController::heaterTransitionControl(const HeaterSwitchStates& currentHeaterStates) {
-  for (unsigned i = 0; i < 7; i++) {
+  for (unsigned i = 0; i < heater::Switch::NUMBER_OF_SWITCHES; i++) {
     if (heaterStates[i].switchTransition) {
       if (currentHeaterStates[i] == heaterStates[i].target) {
         heaterStates[i].switchTransition = false;