From 579fbbc8cee8250eaa4cea61bc663121f9022de3 Mon Sep 17 00:00:00 2001 From: "Crom (Thibaut CHARLES)" Date: Tue, 19 Mar 2024 19:25:19 +0100 Subject: [PATCH 1/2] fix: Allow extrusion if min_extrude_temp is below zero Signed-off-by: Thibaut CHARLES --- src/mixins/toolhead.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/mixins/toolhead.ts b/src/mixins/toolhead.ts index ae5afa4378..f09ebf7ccb 100644 --- a/src/mixins/toolhead.ts +++ b/src/mixins/toolhead.ts @@ -21,8 +21,6 @@ export default class ToolheadMixin extends Vue { return ( activeExtruder !== undefined && - activeExtruder.temperature >= 0 && - activeExtruder.min_extrude_temp >= 0 && activeExtruder.temperature >= activeExtruder.min_extrude_temp ) } From 09c44137eb9e686bf0ff73d7b9a23de82df03661 Mon Sep 17 00:00:00 2001 From: Pedro Lamas Date: Wed, 20 Mar 2024 16:48:33 +0000 Subject: [PATCH 2/2] fix: check can_extrude property Klipper added `can_extrude` at least 2 years ago, so we should use this property if available and fallback to current behavior otherwise. --- src/mixins/toolhead.ts | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/mixins/toolhead.ts b/src/mixins/toolhead.ts index f09ebf7ccb..e01176bc2a 100644 --- a/src/mixins/toolhead.ts +++ b/src/mixins/toolhead.ts @@ -20,8 +20,13 @@ export default class ToolheadMixin extends Vue { const activeExtruder = this.activeExtruder return ( - activeExtruder !== undefined && - activeExtruder.temperature >= activeExtruder.min_extrude_temp + activeExtruder?.can_extrude ?? + ( + activeExtruder !== undefined && + activeExtruder.temperature >= 0 && + activeExtruder.min_extrude_temp >= 0 && + activeExtruder.temperature >= activeExtruder.min_extrude_temp + ) ) }