From 611b2812d3f1f874e41bf102a62348da19d92513 Mon Sep 17 00:00:00 2001 From: Arthur Date: Fri, 21 Apr 2023 09:59:50 +0800 Subject: [PATCH] FIX: do not redo slicing after changing support params This reverts part of the commit d4490af, because redo slicing is unacceptable after changing support params. Known issue: Bottom surface type depends on support parmas, that's why we add the steps dependency of slicing against support params in commit d4490af. With this commit, bottom surface type may not be right with the initial params, and the user can't do anything to make it better. Change-Id: Iff079c4d1205b8a07468be6396990fe87634a3fe (cherry picked from commit c663a844629c07dd40973832df4e4d1e1827235a) --- src/libslic3r/PrintObject.cpp | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/libslic3r/PrintObject.cpp b/src/libslic3r/PrintObject.cpp index bd93015522..50f3fac5a4 100644 --- a/src/libslic3r/PrintObject.cpp +++ b/src/libslic3r/PrintObject.cpp @@ -727,14 +727,7 @@ bool PrintObject::invalidate_state_by_config_options( || opt_key == "support_top_z_distance" || opt_key == "support_bottom_z_distance" || opt_key == "xy_hole_compensation" - || opt_key == "xy_contour_compensation" - //BBS: [Arthur] the following params affect bottomBridge surface type detection - || opt_key == "support_type" - || opt_key == "bridge_no_support" - || opt_key == "max_bridge_length" - || opt_key == "support_interface_top_layers" - || opt_key == "support_critical_regions_only" - ) { + || opt_key == "xy_contour_compensation") { steps.emplace_back(posSlice); } else if (opt_key == "enable_support") { steps.emplace_back(posSupportMaterial);