From f14c2b66399aff65ac9b32a5e6e0b19edbb64d0e Mon Sep 17 00:00:00 2001 From: Rhuan Costa Date: Sat, 3 Feb 2018 22:51:35 -0200 Subject: [PATCH] Fixed hop time not checked if temperature dropped below setpoint Hop time was not being checked in case the temperature dropped below the setpoint, even when into the offset margin. Now it's being checked every iteration after the timer starts. --- modules/base_plugins/brew_steps/__init__.py | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/modules/base_plugins/brew_steps/__init__.py b/modules/base_plugins/brew_steps/__init__.py index fdd4962..4ebe832 100644 --- a/modules/base_plugins/brew_steps/__init__.py +++ b/modules/base_plugins/brew_steps/__init__.py @@ -217,12 +217,15 @@ class BoilStep(StepBase): # Check if Timer is Running if self.is_timer_finished() is None: self.start_timer(int(self.timer) * 60) - else: - self.check_hop_timer(1, self.hop_1) - self.check_hop_timer(2, self.hop_2) - self.check_hop_timer(3, self.hop_3) - self.check_hop_timer(4, self.hop_4) - self.check_hop_timer(5, self.hop_5) + + # Check if hop time reached + if self.is_timer_finished() == False: + self.check_hop_timer(1, self.hop_1) + self.check_hop_timer(2, self.hop_2) + self.check_hop_timer(3, self.hop_3) + self.check_hop_timer(4, self.hop_4) + self.check_hop_timer(5, self.hop_5) + # Check if timer finished and go to next step if self.is_timer_finished() == True: self.notify("Boil Step Completed!", "Starting the next step", timeout=None)