Fix issue with starting the workout second time

app/__init__.py

@@ -333,6 +333,7 @@ def on_system_control(event: dict) -> None:
             if sandglass_thread is not None and sandglass_thread.is_alive():
                 led_controller.stop_sandglass_thread()
                 sandglass_thread.join()
+                sandglass_thread = None
 
             if event["led_toggle"] is True:
                 sandglass_thread = threading.Thread(

app/const.py

@@ -1,10 +1,13 @@
 """Constants for the Stair Challenge app."""
 # ruff: noqa: E501
+import threading
 from typing import Final
 
 MQTT_SENSOR: Final = "sensor"
 MQTT_WORKOUT: Final = "workout"
 
+THREAD_STOP_EVENT = threading.Event()
+
 # MQTT topics - Subscribed
 MQTT_TEST_TOPIC: Final = "test"
 MQTT_TRIGGER_TOPIC: Final = f"{MQTT_SENSOR}/+/trigger"

app/led_controller.py

@@ -1,12 +1,11 @@
 """LED strip module."""
 import secrets
-import threading
 import time
 from enum import Enum
 
 from rpi_ws281x import Color, PixelStrip
 
-thread_stop_event = threading.Event()
+from app.const import THREAD_STOP_EVENT
 
 
 class Colors:
@@ -282,6 +281,9 @@ def sandglass(self, duration: int, color: Color) -> None:
             duration (int): Duration of the sandglass animation in seconds.
             color (Color): Color object with RGB values
         """
+        # Reset the stop event
+        THREAD_STOP_EVENT.clear()
+
         num_leds = self.strip.numPixels()
         self.set_color(color)
 
@@ -294,9 +296,9 @@ def sandglass(self, duration: int, color: Color) -> None:
         for i in range(num_leds):
             remaining_time = duration - (i * interval_seconds)
 
-            if thread_stop_event.is_set():
+            if THREAD_STOP_EVENT.is_set():
                 print("Force stopping sandglass animation")
-                thread_stop_event.clear()
+                THREAD_STOP_EVENT.clear()
                 break
 
             # Turn off the current LED
@@ -318,4 +320,8 @@ def turn_off(self) -> None:
 
     def stop_sandglass_thread(self) -> None:
         """Stop the sandglass thread."""
-        thread_stop_event.set()
+        THREAD_STOP_EVENT.set()
+
+    def reset_stop_event(self) -> None:
+        """Reset the stop event."""
+        THREAD_STOP_EVENT.clear()