From c482a62123575f5949add2be106c5d45e8d5df38 Mon Sep 17 00:00:00 2001 From: Ioannis Giannakas <59056762+igiannakas@users.noreply.github.com> Date: Sat, 18 May 2024 15:38:42 +0100 Subject: [PATCH] Updated arc fitting pop up to clarify that this option should not be enabled for Klipper machines (#5352) * Updated arc fitting pop up * Merge branch 'main' into Arc-fitting-updated-pop-up-for-Klipper-machines --- src/libslic3r/PrintConfig.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/libslic3r/PrintConfig.cpp b/src/libslic3r/PrintConfig.cpp index 03aac8f29c..29eeba5ae6 100644 --- a/src/libslic3r/PrintConfig.cpp +++ b/src/libslic3r/PrintConfig.cpp @@ -2453,7 +2453,11 @@ def = this->add("filament_loading_speed", coFloats); def = this->add("enable_arc_fitting", coBool); def->label = L("Arc fitting"); def->tooltip = L("Enable this to get a G-code file which has G2 and G3 moves. " - "And the fitting tolerance is same with resolution"); + "The fitting tolerance is same as the resolution. \n\n" + "Note: For klipper machines, this option is recomended to be disabled. Klipper does not benefit from " + "arc commands as these are split again into line segments by the firmware. This results in a reduction " + "in surface quality as line segments are converted to arcs by the slicer and then back to line segments " + "by the firmware."); def->mode = comAdvanced; def->set_default_value(new ConfigOptionBool(0)); // BBS