From 4fd19a5531186baaaa9c30d1fe01f72ecdd9a184 Mon Sep 17 00:00:00 2001 From: Clifford Garwood Date: Mon, 25 May 2026 05:13:44 -0400 Subject: [PATCH] fix: drop duplicate filament-extruder reshape in Print::apply Upstream #13360 moved `m_ori_full_print_config = ...` and `update_values_to_printer_extruders_for_multiple_filaments(...)` into a new `if (!extruder_applied)` block. The 3-way merge with our IMEX `physical_extruder_map` derivation block (sitting between the old and new positions) silently kept both copies, running the per-filament reshape twice and miscounting filaments for the multi-temperature compatibility check. Removed the obsolete second copy at the old position. Resolves `p1s_multicolor.3mf` regression test failure. --- src/libslic3r/PrintApply.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/libslic3r/PrintApply.cpp b/src/libslic3r/PrintApply.cpp index 789076a43e..213a20ac55 100644 --- a/src/libslic3r/PrintApply.cpp +++ b/src/libslic3r/PrintApply.cpp @@ -1187,8 +1187,6 @@ Print::ApplyStatus Print::apply(const Model &model, DynamicPrintConfig new_full_ pem->values = effective_physical_extruder_map(pem, pei).values; } - m_ori_full_print_config = new_full_config; - new_full_config.update_values_to_printer_extruders_for_multiple_filaments(new_full_config, filament_options_with_variant, "filament_self_index", "filament_extruder_variant"); auto opt_filament_map = new_full_config.option("filament_map"); std::vector filament_maps = opt_filament_map ? opt_filament_map->values : std::vector();