From bd55f5ecf42f8b5a956f050ac78dc1d63c7a3918 Mon Sep 17 00:00:00 2001 From: Rijk van Manen Date: Thu, 26 Jan 2023 11:20:28 +0100 Subject: [PATCH 1/3] Add new setting for the narrow skin region width Small Top/Bottom regions are filled with walls instead of the normal Top/Bottom pattern. This helps to avoids jerky motions. CURA-10201 --- resources/definitions/fdmprinter.def.json | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index 89d928eb7b..d5817def57 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1649,6 +1649,19 @@ "limit_to_extruder": "top_bottom_extruder_nr", "settable_per_mesh": true }, + "small_skin_width": + { + "label": "Small Top/Bottom width", + "description": "Small Top/Bottom regions are filled with walls instead of the normal Top/Bottom pattern. This helps to avoids jerky motions.", + "value": "skin_line_width * 2", + "default_value": 1, + "minimum_value": "0", + "maximum_value_warning": "skin_line_width * 10", + "type": "float", + "enabled": "(top_layers > 0 or bottom_layers > 0) and top_bottom_pattern != 'concentric'", + "limit_to_extruder": "top_bottom_extruder_nr", + "settable_per_mesh": true + }, "skin_no_small_gaps_heuristic": { "label": "No Skin in Z Gaps", From 8410f8418405cb3a1d9f6e710ce4bd04fe202b16 Mon Sep 17 00:00:00 2001 From: Rijk van Manen Date: Thu, 26 Jan 2023 11:31:27 +0100 Subject: [PATCH 2/3] add new setting to expert CURA-10201 --- resources/setting_visibility/expert.cfg | 1 + 1 file changed, 1 insertion(+) diff --git a/resources/setting_visibility/expert.cfg b/resources/setting_visibility/expert.cfg index 92196391ba..101fdc2d63 100644 --- a/resources/setting_visibility/expert.cfg +++ b/resources/setting_visibility/expert.cfg @@ -58,6 +58,7 @@ top_bottom_pattern_0 skin_monotonic connect_skin_polygons skin_angles +small_skin_width skin_no_small_gaps_heuristic skin_outline_count ironing_enabled From 837171ac5cab98db4ff1b6441dd14f68dcd43e8f Mon Sep 17 00:00:00 2001 From: Casper Lamboo Date: Mon, 20 Mar 2023 17:27:07 +0100 Subject: [PATCH 3/3] Apply suggestions from code review --- resources/definitions/fdmprinter.def.json | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/resources/definitions/fdmprinter.def.json b/resources/definitions/fdmprinter.def.json index d5817def57..bd7ff6c177 100644 --- a/resources/definitions/fdmprinter.def.json +++ b/resources/definitions/fdmprinter.def.json @@ -1651,8 +1651,8 @@ }, "small_skin_width": { - "label": "Small Top/Bottom width", - "description": "Small Top/Bottom regions are filled with walls instead of the normal Top/Bottom pattern. This helps to avoids jerky motions.", + "label": "Small Top/Bottom Width", + "description": "Small top/bottom regions are filled with walls instead of the default top/bottom pattern. This helps to avoids jerky motions.", "value": "skin_line_width * 2", "default_value": 1, "minimum_value": "0", @@ -1660,7 +1660,8 @@ "type": "float", "enabled": "(top_layers > 0 or bottom_layers > 0) and top_bottom_pattern != 'concentric'", "limit_to_extruder": "top_bottom_extruder_nr", - "settable_per_mesh": true + "settable_per_mesh": true, + "unit": "mm" }, "skin_no_small_gaps_heuristic": {