From 6a8c344bc952f51b4534ba35f4c3a4c769660c65 Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 29 Mar 2022 12:30:24 +0200 Subject: [PATCH 1/6] Introduce setting wall_transition_filter_deviation --- resources/definitions/fdmprinter.def.json | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index 95abd8df59..3a8f88f084 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1150,6 +1150,18 @@ "minimum_value_warning": "math.cos(wall_transition_angle / 180 * math.pi) * wall_line_width_x", "maximum_value_warning": "10 * math.cos(wall_transition_angle / 180 * math.pi) * wall_line_width_x" }, + "wall_transition_filter_deviation": + { + "label": "Wall Transition Filter Deviation", + "description": "Prevent Wall Transition Filtering from inducing errors on the extrusion width larger than this value.", + "type": "float", + "unit": "mm", + "default_value": 0.2, + "value": ".5 * wall_line_width_x", + "minimum_value": "0", + "minimum_value_warning": ".25 * wall_line_width_x", + "maximum_value_warning": "wall_line_width_x" + }, "wall_0_wipe_dist": { "label": "Outer Wall Wipe Distance", From ddbd91951f4ace160df34c2fba0929a2a31807f7 Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 29 Mar 2022 12:31:13 +0200 Subject: [PATCH 2/6] Rename Filter Distance We're not filtering distances, we're filtering stuff by this distance, so the word order should be reversed --- resources/definitions/fdmprinter.def.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index 3a8f88f084..16e82b2ecc 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1140,7 +1140,7 @@ }, "wall_transition_filter_distance": { - "label": "Wall Transition Distance Filter", + "label": "Wall Transition Filter Distance", "description": "If it would be transitioning back and forth between different numbers of walls in quick succession, don't transition at all. Remove transitions if they are closer together than this distance.", "type": "float", "unit": "mm", From 2f2f3bb5281b203787351635ca589c812da54bf2 Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 29 Mar 2022 18:17:06 +0200 Subject: [PATCH 3/6] change filter deviation defaults as discussed with Rijk from PP&M --- resources/definitions/fdmprinter.def.json | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index 16e82b2ecc..fde8e0875e 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1156,11 +1156,11 @@ "description": "Prevent Wall Transition Filtering from inducing errors on the extrusion width larger than this value.", "type": "float", "unit": "mm", - "default_value": 0.2, - "value": ".5 * wall_line_width_x", + "default_value": 0.1, + "value": ".25 * machine_nozzle_size", "minimum_value": "0", - "minimum_value_warning": ".25 * wall_line_width_x", - "maximum_value_warning": "wall_line_width_x" + "minimum_value_warning": ".01", + "maximum_value_warning": "machine_nozzle_size" }, "wall_0_wipe_dist": { From d3e0b04eeda2402a962fc393ef7441b17366f139 Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 5 Apr 2022 13:57:56 +0200 Subject: [PATCH 4/6] update filter margin name and description --- resources/definitions/fdmprinter.def.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index fde8e0875e..0ee23dc799 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1152,8 +1152,8 @@ }, "wall_transition_filter_deviation": { - "label": "Wall Transition Filter Deviation", - "description": "Prevent Wall Transition Filtering from inducing errors on the extrusion width larger than this value.", + "label": "Wall Transitioning Filter Margin", + "description": "Prevent transitioning back and forth between one extra wall and one less. This margin extends the range of line widths which follow to [Minimum Wall Line Width - Margin, 2 * Minimum Wall Line Width + Margin]. Increasing this margin reduces the number of transitions, which reduces the number of extrusion starts/stops and travel time. However, large line width variation can lead to under- or overextrusion problems.", "type": "float", "unit": "mm", "default_value": 0.1, From de29eb7d53caa82b0e2567e79865c684caca7b6c Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 5 Apr 2022 13:59:01 +0200 Subject: [PATCH 5/6] change name to transitionING filter This is not a property of a transition, but of when the transitioning algorithm makes or leaves out transitions --- resources/definitions/fdmprinter.def.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index 0ee23dc799..df0089c033 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1140,7 +1140,7 @@ }, "wall_transition_filter_distance": { - "label": "Wall Transition Filter Distance", + "label": "Wall Transitioning Filter Distance", "description": "If it would be transitioning back and forth between different numbers of walls in quick succession, don't transition at all. Remove transitions if they are closer together than this distance.", "type": "float", "unit": "mm", From 8098c208601327b9ecd4bc0b2b0f700d0ad2dcd1 Mon Sep 17 00:00:00 2001 From: Tim Kuipers Date: Tue, 5 Apr 2022 14:00:42 +0200 Subject: [PATCH 6/6] disable wall_transition_filter_distance We don't want to filter based on the length of the filtered region, but based on the induced extra line width variation. This diables the setting for the frontend, but doesn't fully remove it yet. TODO --- resources/definitions/fdmprinter.def.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index df0089c033..efbd8db67c 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1144,8 +1144,8 @@ "description": "If it would be transitioning back and forth between different numbers of walls in quick succession, don't transition at all. Remove transitions if they are closer together than this distance.", "type": "float", "unit": "mm", - "default_value": 1.4, - "value": "4 * math.cos(wall_transition_angle / 180 * math.pi) * wall_line_width_x", + "enabled": false, + "default_value": 100, "minimum_value": "wall_transition_length", "minimum_value_warning": "math.cos(wall_transition_angle / 180 * math.pi) * wall_line_width_x", "maximum_value_warning": "10 * math.cos(wall_transition_angle / 180 * math.pi) * wall_line_width_x"