From 15ca7cb6fa0fc30901777f564d2e52a6b816bd76 Mon Sep 17 00:00:00 2001
From: Rob Hallam <0504004h@student.gla.ac.uk>
Date: Tue, 3 Sep 2024 13:46:22 +0100
Subject: [PATCH] fix: negative total durations should not be possible

---
 pipeline/adjusters.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/pipeline/adjusters.py b/pipeline/adjusters.py
index 97afeb3..ec97595 100644
--- a/pipeline/adjusters.py
+++ b/pipeline/adjusters.py
@@ -55,6 +55,10 @@ class TargetTimeAdjuster(Adjuster):
             target_time_max = time + (time * margin / 100)
             target_time_min = time - (time * margin / 100)
 
+        # ensure we don't have negative times
+        if type(target_time_min) is float and target_time_min < 0:
+            target_time_min = 0.0
+
         return (target_time_min, target_time_max)
 
     def _features_total_time(self, features: list) -> float: