diff options
| author | Kevin O'Connor <kevin@koconnor.net> | 2018-09-28 14:50:42 -0400 |
|---|---|---|
| committer | Kevin O'Connor <kevin@koconnor.net> | 2018-09-30 10:40:44 -0400 |
| commit | 864918b1f980b3160299d68bd8a24f35f1e2a60a (patch) | |
| tree | aad9bd9fa254f59cda082af068779d23eaee8ce2 /klippy | |
| parent | 47b6144ff69c34862883f13d45a95e50097be4e3 (diff) | |
| download | kutter-864918b1f980b3160299d68bd8a24f35f1e2a60a.tar.gz kutter-864918b1f980b3160299d68bd8a24f35f1e2a60a.tar.xz kutter-864918b1f980b3160299d68bd8a24f35f1e2a60a.zip | |
verify_heater: Don't report a heater error if printer is shutdown
Don't log a heater verification error if that error is due to the
printer being shutdown for some other reason. Those spurious error
reports can be confusing.
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'klippy')
| -rw-r--r-- | klippy/extras/verify_heater.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/klippy/extras/verify_heater.py b/klippy/extras/verify_heater.py index ba7add6c..7ac4d920 100644 --- a/klippy/extras/verify_heater.py +++ b/klippy/extras/verify_heater.py @@ -26,6 +26,7 @@ class HeaterCheck: self.met_target = False self.last_target = self.goal_temp = self.error = 0. self.fault_systime = self.printer.get_reactor().NEVER + self.check_timer = None def printer_state(self, state): if state == 'connect': if self.printer.get_start_args().get('debugoutput') is not None: @@ -35,7 +36,11 @@ class HeaterCheck: self.heater = pheater.lookup_heater(self.heater_name) logging.info("Starting heater checks for %s", self.heater_name) reactor = self.printer.get_reactor() - reactor.register_timer(self.check_event, reactor.NOW) + self.check_timer = reactor.register_timer(self.check_event, + reactor.NOW) + elif state == 'shutdown' and self.check_timer is not None: + reactor = self.printer.get_reactor() + reactor.update_timer(self.check_timer, reactor.NEVER) def check_event(self, eventtime): temp, target = self.heater.get_temp(eventtime) if temp >= target - self.hysteresis: |