Keep the Segment-Based Voronoi Reproducers

The gapfill, weirdness and NaN reproducers go through Orca's repairing VoronoiDiagram once they use it instead of raw boost, so they are converted rather than deleted. Only the two point-input reproducers, which the wrapper neither checks nor repairs, are removed.
Include Model.hpp in the fff_print Test Helpers
2026-10-19 13:41:03 +00:00 · 2026-10-19 16:58:02 +08:00 · 2026-10-19 15:45:35 +08:00 · 2026-10-19 15:40:06 +08:00 · 2026-10-19 13:48:53 +08:00 · 2026-10-19 13:30:36 +08:00
42 changed files with 66 additions and 1408 deletions
@@ -10657,6 +10657,9 @@ msgstr ""
 msgid "Send Failed"
 msgstr ""
 msgid "No available printers"
 msgstr ""
 msgid "The name length exceeds the limit."
 msgstr ""
@@ -756,7 +756,14 @@ public:
    Preset&         get_edited_preset()         { return m_edited_preset; }
    const Preset&   get_edited_preset() const   { return m_edited_preset; }
-    const Preset& get_selected_preset_base() const { return *get_preset_base(m_presets[m_idx_selected]); }
+    // The root of the selected preset's inherits chain, or the selected preset itself when
    // the chain names a preset that is not loaded (e.g. a vendor that is not installed).
    const Preset& get_selected_preset_base() const
    {
        const Preset &selected = m_presets[m_idx_selected];
        const Preset *base     = get_preset_base(selected);
        return base ? *base : selected;
    }
    // Return the last saved preset.
 //  const Preset&   get_saved_preset() const { return m_saved_preset; }
@@ -3151,7 +3151,7 @@ void CreatePrinterPresetDialog::set_current_visible_printer()
    m_printer_name_to_preset.clear();
    for (const Preset &printer_preset : printer_presets) {
        if (!printer_preset.is_visible) continue;
-        if (preset_bundle->printers.get_preset_base(printer_preset)->name != printer_preset.name) continue;
+        if (preset_bundle->printers.get_preset_base(printer_preset) != &printer_preset) continue;
        if (auto printer_model = dynamic_cast<ConfigOptionString*>(const_cast<Preset&>(printer_preset).config.option("printer_model", false))) {
            if (m_printer_name_to_preset.find(printer_model->value) == m_printer_name_to_preset.end()) {
                printer_choice.push_back(from_u8(printer_model->value));
@@ -4962,7 +4962,7 @@ void CreatePresetForPrinterDialog::get_visible_printer_and_compatible_filament_p
                    if (filament_preset.is_default || !filament_preset.is_compatible || filament_preset.is_project_embedded) continue;
                    ConfigOptionStrings *filament_types;
                    const Preset *       filament_preset_base = m_preset_bundle->filaments.get_preset_base(filament_preset);
-                    if (filament_preset_base == &filament_preset) {
+                    if (filament_preset_base == &filament_preset || filament_preset_base == nullptr) {
                        filament_types = dynamic_cast<ConfigOptionStrings *>(const_cast<Preset *>(&filament_preset)->config.option("filament_type"));
                    } else {
                        filament_types = dynamic_cast<ConfigOptionStrings *>(const_cast<Preset *>(filament_preset_base)->config.option("filament_type"));
@@ -4087,7 +4087,7 @@ void SelectMachineDialog::update_user_printer()
    {
        m_printer_last_select = "";
        update_select_layout(nullptr);
-        m_printer_box->GetPrinterComboBox()->SetTextLabel("");
+        m_printer_box->GetPrinterComboBox()->SetValue(_L("No available printers"));
    }
    BOOST_LOG_TRIVIAL(info) << __FUNCTION__ << "for send task, current printer id =  " << m_printer_last_select << std::endl;
@@ -1229,7 +1229,7 @@ void SendToPrinterDialog::update_user_printer()
    }
    else {
        m_printer_last_select = "";
-        m_comboBox_printer->SetTextLabel("");
+        m_comboBox_printer->SetValue(_L("No available printers"));
    }
    BOOST_LOG_TRIVIAL(info) << __FUNCTION__ << "for send task, current printer id =  " << m_printer_last_select << std::endl;
@@ -287,7 +287,7 @@ void ComboBox::DoDeleteOneItem(unsigned int pos)
 void ComboBox::ForceDropdownOpen()
 {
-    if (!IsEnabled())
+    if (!IsEnabled() || items.empty())
        return;
    if (!drop_down) {
@@ -398,7 +398,7 @@ void ComboBox::mouseDown(wxMouseEvent &event)
    SetFocus();
    if (drop_down) {
        drop.Hide();
-    } else if (drop.HasDismissLongTime()) {
+    } else if (!items.empty() && drop.HasDismissLongTime()) { // DropDown paints nothing for an empty list
        drop.autoPosition();
        drop_down = true;
        drop.Popup(&drop);
@@ -426,7 +426,7 @@ void ComboBox::keyDown(wxKeyEvent& event)
        case WXK_SPACE:
            if (drop_down) {
                drop.DismissAndNotify();
-            } else if (drop.HasDismissLongTime()) {
+            } else if (!items.empty() && drop.HasDismissLongTime()) {
                drop.autoPosition();
                drop_down = true;
                drop.Popup();
@@ -584,6 +584,19 @@ bool PythonInterpreter::initialize()
        }
        BOOST_LOG_TRIVIAL(debug) << "Program name set successfully";
        // Keep bytecode out of the Python home. The shipped .pyc files are stale against their
        // sources, so a writable install would rewrite them inside the app (breaking a signed
        // macOS bundle's seal) and a read-only one would recompile them on every launch.
        const std::string pycache_prefix = data_dir() + "/python/pycache";
 #ifdef _WIN32
        status = PyConfig_SetString(&config, &config.pycache_prefix, boost::nowide::widen(pycache_prefix).c_str());
 #else
        status = PyConfig_SetBytesString(&config, &config.pycache_prefix, pycache_prefix.c_str());
 #endif
        if (PyStatus_Exception(status))
            BOOST_LOG_TRIVIAL(warning) << "Failed to set the Python bytecode cache to " << pycache_prefix << ": "
                                       << (status.err_msg ? status.err_msg : "unknown error");
        // Use pybind11's scoped_interpreter which properly initializes
        // pybind11 internals for multi-threaded use — raw Py_InitializeFromConfig
        // does not set up thread state tracking that pybind11 requires.
@@ -1 +0,0 @@
 solid STL generated by MeshLab␍  facet normal  0.000000e+00 -0.000000e+00 -1.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  0.000000e+00␍      vertex   2.000000e+01  0.000000e+00  0.000000e+00␍      vertex   0.000000e+00  0.000000e+00  0.000000e+00␍    endloop␍  endfacet␍  facet normal -0.000000e+00  0.000000e+00 -1.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  0.000000e+00␍      vertex   0.000000e+00  0.000000e+00  0.000000e+00␍      vertex   0.000000e+00  2.000000e+01  0.000000e+00␍    endloop␍  endfacet␍  facet normal  0.000000e+00  0.000000e+00  1.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  2.000000e+01␍      vertex   0.000000e+00  2.000000e+01  2.000000e+01␍      vertex   0.000000e+00  0.000000e+00  2.000000e+01␍    endloop␍  endfacet␍  facet normal  0.000000e+00  0.000000e+00  1.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  2.000000e+01␍      vertex   0.000000e+00  0.000000e+00  2.000000e+01␍      vertex   2.000000e+01  0.000000e+00  2.000000e+01␍    endloop␍  endfacet␍  facet normal  1.000000e+00  0.000000e+00 -0.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  0.000000e+00␍      vertex   2.000000e+01  2.000000e+01  2.000000e+01␍      vertex   2.000000e+01  0.000000e+00  2.000000e+01␍    endloop␍  endfacet␍  facet normal  1.000000e+00  0.000000e+00  0.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  0.000000e+00␍      vertex   2.000000e+01  0.000000e+00  2.000000e+01␍      vertex   2.000000e+01  0.000000e+00  0.000000e+00␍    endloop␍  endfacet␍  facet normal  0.000000e+00 -1.000000e+00  0.000000e+00␍    outer loop␍      vertex   2.000000e+01  0.000000e+00  0.000000e+00␍      vertex   2.000000e+01  0.000000e+00  2.000000e+01␍      vertex   0.000000e+00  0.000000e+00  2.000000e+01␍    endloop␍  endfacet␍  facet normal  0.000000e+00 -1.000000e+00  0.000000e+00␍    outer loop␍      vertex   2.000000e+01  0.000000e+00  0.000000e+00␍      vertex   0.000000e+00  0.000000e+00  2.000000e+01␍      vertex   0.000000e+00  0.000000e+00  0.000000e+00␍    endloop␍  endfacet␍  facet normal -1.000000e+00  0.000000e+00  0.000000e+00␍    outer loop␍      vertex   0.000000e+00  0.000000e+00  0.000000e+00␍      vertex   0.000000e+00  0.000000e+00  2.000000e+01␍      vertex   0.000000e+00  2.000000e+01  2.000000e+01␍    endloop␍  endfacet␍  facet normal -1.000000e+00  0.000000e+00  0.000000e+00␍    outer loop␍      vertex   0.000000e+00  0.000000e+00  0.000000e+00␍      vertex   0.000000e+00  2.000000e+01  2.000000e+01␍      vertex   0.000000e+00  2.000000e+01  0.000000e+00␍    endloop␍  endfacet␍  facet normal  0.000000e+00  1.000000e+00  0.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  2.000000e+01␍      vertex   2.000000e+01  2.000000e+01  0.000000e+00␍      vertex   0.000000e+00  2.000000e+01  0.000000e+00␍    endloop␍  endfacet␍  facet normal  0.000000e+00  1.000000e+00  0.000000e+00␍    outer loop␍      vertex   2.000000e+01  2.000000e+01  2.000000e+01␍      vertex   0.000000e+00  2.000000e+01  0.000000e+00␍      vertex   0.000000e+00  2.000000e+01  2.000000e+01␍    endloop␍  endfacet␍endsolid vcg␍
@@ -44,7 +44,6 @@
 #include "libslic3r/Layer.hpp"
 #include "libslic3r/Print.hpp"
 #include "libslic3r/PrintConfig.hpp"
 #include "libslic3r/SVG.hpp"
 #include "libslic3r/libslic3r.h"
 #include "test_helpers.hpp"
@@ -53,15 +52,6 @@ using namespace Slic3r;
 bool test_if_solid_surface_filled(const ExPolygon& expolygon, double flow_spacing, double angle = 0, double density = 1.0);
 #if 0
 TEST_CASE("Adjusted solid distance", "[Fill]") {
    int surface_width = 250;
    int distance = Slic3r::Flow::solid_spacing(surface_width, 47);
    REQUIRE(distance == Catch::Approx(50));
    REQUIRE(surface_width % distance == 0);
 }
 #endif
 TEST_CASE("Pattern path length", "[Fill]") {
    std::unique_ptr<Slic3r::Fill> filler(Slic3r::Fill::new_from_type("rectilinear"));
    filler->angle = float(-(PI)/2.0);
@@ -120,15 +110,6 @@ TEST_CASE("Pattern path length", "[Fill]") {
                filler->angle = float(angle);
                ExPolygon e(test_square, test_hole);
                Slic3r::Polylines paths = test(e);
 #if 0
 				{
 					BoundingBox bbox = get_extents(e);
 					SVG svg("c:\\data\\temp\\square_with_holes.svg", bbox);
 					svg.draw(e);
 					svg.draw(paths);
 					svg.Close();
 				}
 #endif
                REQUIRE((paths.size() >= 1 && paths.size() <= 3));
                // paths don't cross hole
                REQUIRE(diff_pl(paths, offset(e, float(SCALED_EPSILON*10))).size() == 0);
@@ -175,25 +156,6 @@ TEST_CASE("Pattern path length", "[Fill]") {
        }
    }
    #if 0   // Disabled temporarily due to precision issues on the Mac VM
    SECTION("Solid surface fill") {
        Slic3r::Points points {
            Point::new_scale(6883102, 9598327.01296997),
            Point::new_scale(6883102, 20327272.01297),
            Point::new_scale(3116896, 20327272.01297),
            Point::new_scale(3116896, 9598327.01296997) 
        };
        Slic3r::ExPolygon expolygon(points);
        REQUIRE(test_if_solid_surface_filled(expolygon, 0.55) == true);
        for (size_t i = 0; i <= 20; ++i)
        {
            expolygon.scale(1.05);
            REQUIRE(test_if_solid_surface_filled(expolygon, 0.55) == true);
        }
    }
    #endif
    SECTION("Solid surface fill") {
        Slic3r::Points points {
                Slic3r::Point(59515297,5422499),Slic3r::Point(59531249,5578697),Slic3r::Point(59695801,6123186),
@@ -226,245 +188,6 @@ TEST_CASE("Pattern path length", "[Fill]") {
    }
 }
 /*
 {
    my $collection = Slic3r::Polyline::Collection->new(
            Slic3r::Polyline->new([0,15], [0,18], [0,20]),
            Slic3r::Polyline->new([0,10], [0,8], [0,5]),
            );
    is_deeply
        [ map $_->[Y], map @$_, @{$collection->chained_path_from(Slic3r::Point->new(0,30), 0)} ],
        [20, 18, 15, 10, 8, 5],
        'chained path';
 }
 {
    my $collection = Slic3r::Polyline::Collection->new(
            Slic3r::Polyline->new([4,0], [10,0], [15,0]),
            Slic3r::Polyline->new([10,5], [15,5], [20,5]),
            );
    is_deeply
        [ map $_->[X], map @$_, @{$collection->chained_path_from(Slic3r::Point->new(30,0), 0)} ],
        [reverse 4, 10, 15, 10, 15, 20],
        'chained path';
 }
 {
    my $collection = Slic3r::ExtrusionPath::Collection->new(
            map Slic3r::ExtrusionPath->new(polyline => $_, role => 0, mm3_per_mm => 1),
            Slic3r::Polyline->new([0,15], [0,18], [0,20]),
            Slic3r::Polyline->new([0,10], [0,8], [0,5]),
            );
    is_deeply
        [ map $_->[Y], map @{$_->polyline}, @{$collection->chained_path_from(Slic3r::Point->new(0,30), 0)} ],
        [20, 18, 15, 10, 8, 5],
        'chained path';
 }
 {
    my $collection = Slic3r::ExtrusionPath::Collection->new(
            map Slic3r::ExtrusionPath->new(polyline => $_, role => 0, mm3_per_mm => 1),
            Slic3r::Polyline->new([15,0], [10,0], [4,0]),
            Slic3r::Polyline->new([10,5], [15,5], [20,5]),
            );
    is_deeply
        [ map $_->[X], map @{$_->polyline}, @{$collection->chained_path_from(Slic3r::Point->new(30,0), 0)} ],
        [reverse 4, 10, 15, 10, 15, 20],
        'chained path';
 }
 for my $pattern (qw(rectilinear honeycomb hilbertcurve concentric)) {
    my $config = Slic3r::Config->new_from_defaults;
    $config->set('fill_pattern', $pattern);
    $config->set('external_fill_pattern', $pattern);
    $config->set('perimeters', 1);
    $config->set('skirts', 0);
    $config->set('fill_density', 20);
    $config->set('layer_height', 0.05);
    $config->set('perimeter_extruder', 1);
    $config->set('infill_extruder', 2);
    my $print = Slic3r::Test::init_print('20mm_cube', config => $config, scale => 2);
    ok my $gcode = Slic3r::Test::gcode($print), "successful $pattern infill generation";
    my $tool = undef;
    my @perimeter_points = my @infill_points = ();
    Slic3r::GCode::Reader->new->parse($gcode, sub {
            my ($self, $cmd, $args, $info) = @_;
            if ($cmd =~ /^T(\d+)/) {
            $tool = $1;
            } elsif ($cmd eq 'G1' && $info->{extruding} && $info->{dist_XY} > 0) {
            if ($tool == $config->perimeter_extruder-1) {
            push @perimeter_points, Slic3r::Point->new_scale($args->{X}, $args->{Y});
            } elsif ($tool == $config->infill_extruder-1) {
            push @infill_points, Slic3r::Point->new_scale($args->{X}, $args->{Y});
            }
            }
            });
    my $convex_hull = convex_hull(\@perimeter_points);
    ok !(defined first { !$convex_hull->contains_point($_) } @infill_points), "infill does not exceed perimeters ($pattern)";
 }
 {
    my $config = Slic3r::Config->new_from_defaults;
    $config->set('infill_only_where_needed', 1);
    $config->set('bottom_solid_layers', 0);
    $config->set('infill_extruder', 2);
    $config->set('infill_extrusion_width', 0.5);
    $config->set('fill_density', 40);
    $config->set('cooling', 0);                 # for preventing speeds from being altered
        $config->set('first_layer_speed', '100%');  # for preventing speeds from being altered
        my $test = sub {
            my $print = Slic3r::Test::init_print('pyramid', config => $config);
            my $tool = undef;
            my @infill_extrusions = ();  # array of polylines
                Slic3r::GCode::Reader->new->parse(Slic3r::Test::gcode($print), sub {
                        my ($self, $cmd, $args, $info) = @_;
                        if ($cmd =~ /^T(\d+)/) {
                        $tool = $1;
                        } elsif ($cmd eq 'G1' && $info->{extruding} && $info->{dist_XY} > 0) {
                        if ($tool == $config->infill_extruder-1) {
                        push @infill_extrusions, Slic3r::Line->new_scale(
                                [ $self->X, $self->Y ],
                                [ $info->{new_X}, $info->{new_Y} ],
                                );
                        }
                        }
                        });
            return 0 if !@infill_extrusions;  # prevent calling convex_hull() with no points
                my $convex_hull = convex_hull([ map $_->pp, map @$_, @infill_extrusions ]);
            return unscale unscale sum(map $_->area, @{offset([$convex_hull], scale(+$config->infill_extrusion_width/2))});
        };
    my $tolerance = 5;  # mm^2
        $config->set('solid_infill_below_area', 0);
    ok $test->() < $tolerance,
       'no infill is generated when using infill_only_where_needed on a pyramid';
    $config->set('solid_infill_below_area', 70);
    ok abs($test->() - $config->solid_infill_below_area) < $tolerance,
       'infill is only generated under the forced solid shells';
 }
 {
    my $config = Slic3r::Config->new_from_defaults;
    $config->set('skirts', 0);
    $config->set('perimeters', 1);
    $config->set('fill_density', 0);
    $config->set('top_solid_layers', 0);
    $config->set('bottom_solid_layers', 0);
    $config->set('solid_infill_below_area', 20000000);
    $config->set('solid_infill_every_layers', 2);
    $config->set('perimeter_speed', 99);
    $config->set('external_perimeter_speed', 99);
    $config->set('cooling', 0);
    $config->set('first_layer_speed', '100%');
    my $print = Slic3r::Test::init_print('20mm_cube', config => $config);
    my %layers_with_extrusion = ();
    Slic3r::GCode::Reader->new->parse(Slic3r::Test::gcode($print), sub {
            my ($self, $cmd, $args, $info) = @_;
            if ($cmd eq 'G1' && $info->{dist_XY} > 0 && $info->{extruding}) {
            if (($args->{F} // $self->F) != $config->perimeter_speed*60) {
            $layers_with_extrusion{$self->Z} = ($args->{F} // $self->F);
            }
            }
            });
    ok !%layers_with_extrusion,
       "solid_infill_below_area and solid_infill_every_layers are ignored when fill_density is 0";
 }
 {
    my $config = Slic3r::Config->new_from_defaults;
    $config->set('skirts', 0);
    $config->set('perimeters', 3);
    $config->set('fill_density', 0);
    $config->set('layer_height', 0.2);
    $config->set('first_layer_height', 0.2);
    $config->set('nozzle_diameter', [0.35]);
    $config->set('infill_extruder', 2);
    $config->set('solid_infill_extruder', 2);
    $config->set('infill_extrusion_width', 0.52);
    $config->set('solid_infill_extrusion_width', 0.52);
    $config->set('first_layer_extrusion_width', 0);
    my $print = Slic3r::Test::init_print('A', config => $config);
    my %infill = ();  # Z => [ Line, Line ... ]
        my $tool = undef;
    Slic3r::GCode::Reader->new->parse(Slic3r::Test::gcode($print), sub {
            my ($self, $cmd, $args, $info) = @_;
            if ($cmd =~ /^T(\d+)/) {
            $tool = $1;
            } elsif ($cmd eq 'G1' && $info->{extruding} && $info->{dist_XY} > 0) {
            if ($tool == $config->infill_extruder-1) {
            my $z = 1 * $self->Z;
            $infill{$z} ||= [];
            push @{$infill{$z}}, Slic3r::Line->new_scale(
                    [ $self->X, $self->Y ],
                    [ $info->{new_X}, $info->{new_Y} ],
                    );
            }
            }
            });
    my $grow_d = scale($config->infill_extrusion_width)/2;
    my $layer0_infill = union([ map @{$_->grow($grow_d)}, @{ $infill{0.2} } ]);
    my $layer1_infill = union([ map @{$_->grow($grow_d)}, @{ $infill{0.4} } ]);
    my $diff = diff($layer0_infill, $layer1_infill);
    $diff = offset2_ex($diff, -$grow_d, +$grow_d);
    $diff = [ grep { $_->area > 2*(($grow_d*2)**2) } @$diff ];
    is scalar(@$diff), 0, 'no missing parts in solid shell when fill_density is 0';
 }
 {
    # GH: #2697
    my $config = Slic3r::Config->new_from_defaults;
    $config->set('perimeter_extrusion_width', 0.72);
    $config->set('top_infill_extrusion_width', 0.1);
    $config->set('infill_extruder', 2);         # in order to distinguish infill
        $config->set('solid_infill_extruder', 2);   # in order to distinguish infill
        my $print = Slic3r::Test::init_print('20mm_cube', config => $config);
    my %infill = ();  # Z => [ Line, Line ... ]
        my %other  = ();  # Z => [ Line, Line ... ]
        my $tool = undef;
    Slic3r::GCode::Reader->new->parse(Slic3r::Test::gcode($print), sub {
            my ($self, $cmd, $args, $info) = @_;
            if ($cmd =~ /^T(\d+)/) {
            $tool = $1;
            } elsif ($cmd eq 'G1' && $info->{extruding} && $info->{dist_XY} > 0) {
            my $z = 1 * $self->Z;
            my $line = Slic3r::Line->new_scale(
                    [ $self->X, $self->Y ],
                    [ $info->{new_X}, $info->{new_Y} ],
                    );
            if ($tool == $config->infill_extruder-1) {
            $infill{$z} //= [];
            push @{$infill{$z}}, $line;
            } else {
            $other{$z} //= [];
            push @{$other{$z}}, $line;
            }
            }
            });
    my $top_z = max(keys %infill);
    my $top_infill_grow_d = scale($config->top_infill_extrusion_width)/2;
    my $top_infill = union([ map @{$_->grow($top_infill_grow_d)}, @{ $infill{$top_z} } ]);
    my $perimeters_grow_d = scale($config->perimeter_extrusion_width)/2;
    my $perimeters = union([ map @{$_->grow($perimeters_grow_d)}, @{ $other{$top_z} } ]);
    my $covered = union_ex([ @$top_infill, @$perimeters ]);
    my @holes = map @{$_->holes}, @$covered;
    ok sum(map unscale unscale $_->area*-1, @holes) < 1, 'no gaps between top solid infill and perimeters';
 }
 */
 bool test_if_solid_surface_filled(const ExPolygon& expolygon, double flow_spacing, double angle, double density)
 {
    std::unique_ptr<Slic3r::Fill> filler(Slic3r::Fill::new_from_type("rectilinear"));
@@ -498,18 +221,6 @@ bool test_if_solid_surface_filled(const ExPolygon& expolygon, double flow_spacin
    const double scaled_flow_spacing = std::pow(scale_(flow_spacing), 2);
    uncovered.erase(std::remove_if(uncovered.begin(), uncovered.end(), [scaled_flow_spacing](const ExPolygon& poly) { return poly.area() < scaled_flow_spacing; }), uncovered.end());
 #if 0
 	if (! uncovered.empty()) {
 		BoundingBox bbox = get_extents(expolygon.contour);
 		bbox.merge(get_extents(uncovered));
 		bbox.merge(get_extents(grown_paths));
 		SVG svg("c:\\data\\temp\\test_if_solid_surface_filled.svg", bbox);
 		svg.draw(expolygon);
 		svg.draw(uncovered, "red");
 		svg.Close();
 	}
 #endif
    return uncovered.empty(); // solid surface is fully filled
 }
@@ -58,23 +58,6 @@ SCENARIO("Flow math for non-bridges", "[Flow]") {
            }
        }
    }
 #if 0
    /// Check for an edge case in the maths where the spacing could be 0; original
    /// math is 0.99. Slic3r issue #4654
    GIVEN("Input spacing of 0.414159 and a total width of 2") {
        double in_spacing = 0.414159;
        double total_width = 2.0;
        auto flow = Flow::new_from_spacing(1.0, 0.4, 0.3);
        WHEN("solid_spacing() is called") {
            double result = flow.solid_spacing(total_width, in_spacing);
            THEN("Yielded spacing is greater than 0") {
                REQUIRE(result > 0);
            }
        }
    }
 #endif    
 }
 /// Spacing, width calculation for bridge extrusions
@@ -142,21 +142,6 @@ SCENARIO("z_hop lifts the nozzle when a lift is requested", "[GCodeWriter]") {
    }
 }
 SCENARIO("Origin manipulation", "[GCodeWriter]") {
 	Slic3r::GCode gcodegen;
 	WHEN("set_origin to (10,0)") {
    	gcodegen.set_origin(Vec2d(10,0));
    	REQUIRE(gcodegen.origin() == Vec2d(10, 0));
    }
 	WHEN("set_origin to (10,0) and translate by (5, 5)") {
 		gcodegen.set_origin(Vec2d(10,0));
 		gcodegen.set_origin(gcodegen.origin() + Vec2d(5, 5));
 		THEN("origin returns reference to point") {
    		REQUIRE(gcodegen.origin() == Vec2d(15,5));
    	}
    }
 }
 TEST_CASE("A cached config slot is looked up again whenever its key changes", "[GCodeWriter]")
 {
    GCode::ConfigIndexCache cache;
@@ -5,6 +5,7 @@
 #include "libslic3r/GCodeReader.hpp"
 #include "libslic3r/Config.hpp"
 #include "libslic3r/Print.hpp"
 #include "libslic3r/Model.hpp"
 #include <algorithm>
 #include <cstdlib>
@@ -502,27 +503,3 @@ std::vector<std::string> role_sequence(const std::string &gcode, const std::vect
 }
 } } // namespace Slic3r::Test
 #include <catch2/catch_all.hpp>
 #include "libslic3r/Arrange.hpp"
 #include "libslic3r/Model.hpp"
 SCENARIO("init_print functionality", "[test_helpers]") {
 	GIVEN("A default config") {
 		Slic3r::DynamicPrintConfig config = Slic3r::DynamicPrintConfig::full_print_config();
 		WHEN("init_print is called with a single mesh.") {
 			Slic3r::Model model;
 			Slic3r::Print print;
 			Slic3r::Test::init_print({ Slic3r::Test::cube(20) }, print, model, config);
 			THEN("One mesh/printobject is in the resulting Print object.") {
 				REQUIRE(print.objects().size() == 1);
 			}
 			THEN("print.process() doesn't crash.") {
 				REQUIRE_NOTHROW(print.process());
 			}
 			THEN("Export gcode functions outputs text.") {
 				REQUIRE(! Slic3r::Test::gcode(print).empty());
 			}
 		}
 	}
 }
@@ -327,24 +327,6 @@ TEST_CASE("A printer with IMEX disabled resolves the mode placeholders to nothin
    CHECK_FALSE(has_line(gcode, kQuadScript));
 }
 // The placeholders are set unconditionally, ahead of the is_imex check, so they are defined for
 // every printer. A single-extruder preset that mentions {imex_mode} must expand it to an empty
 // string rather than fail the export on an unknown variable.
 TEST_CASE("IMEX mode placeholders are defined on an ordinary single-extruder printer",
          "[ImexModeGcode][IMEX]")
 {
    const std::string gcode = slice({ cube(20) }, {
        { "skirt_loops",         "0" },
        { "brim_type",           "no_brim" },
        { "gcode_flavor",        "klipper" },
        { "machine_start_gcode", kReportStartGcode },
    });
    CHECK(has_line(gcode, ";IMEX_MODE:"));
    CHECK(has_line(gcode, ";IMEX_MODE_INDEX:0"));
    CHECK(has_line(gcode, ";IMEX_MODE_GCODE:"));
 }
 // In a parallel mode no tool changes occur, so every active carriage has to be addressed
 // explicitly: once before the print for pressure advance, and again at the second layer for the
 // drop from the initial-layer temperature. Nothing else in the suite covers either emission, and
@@ -66,21 +66,6 @@ void count_mixed(ToolOrdering &to, size_t &groups, size_t &resolutions)
 } // namespace
 TEST_CASE("enable_mixed_color_sublayer reaches the Print config", "[MixedFilament]")
 {
    Print print;
    Model model;
    init_print({cube(20)}, print, model, mixed_config(true));
    // The option lives in PrintConfig; if it did not survive Print::apply the slicer would
    // silently fall back to the whole-layer path.
    CHECK(print.config().enable_mixed_color_sublayer.value == true);
    REQUIRE(print.config().filament_is_mixed.values.size() == 3);
    CHECK(print.config().filament_is_mixed.values[2] == true);
    REQUIRE(print.config().filament_mixed_components.values.size() == 3);
    CHECK(print.config().filament_mixed_components.values[2] == "1,2");
 }
 TEST_CASE("Mixed filament splits layers into sub-layers when the option is on", "[MixedFilament]")
 {
    Print print;
@@ -840,32 +840,6 @@ static void all_regions_on_filament(DynamicPrintConfig &config, int filament_1ba
        config.set_deserialize_strict({ { key, std::to_string(filament_1based) } });
 }
 // IMEX parallel modes emit per-carriage temperatures from a branch that is mutually
 // exclusive with the standard per-extruder path, and that branch skipped the head the
 // print's own toolpaths run on. That head therefore never received its 1st->2nd layer
 // transition and held nozzle_temperature_initial_layer for the whole job.
 TEST_CASE("Parallel-mode IMEX prints transition the printing head to its second-layer temperature",
          "[MultiFilament][IMEX]")
 {
    DynamicPrintConfig config = multifilament_config(7);
    imex_7x4_printer(config);
    all_regions_on_filament(config, 1); // filament 1 => logical slot 0 => physical head 0
    // The multi-extruder normalization collapses per-filament temperature vectors to a
    // single value, so heads are told apart by their tool qualifier, not by temperature.
    config.set_deserialize_strict({
        { "imex_parallel_mode",                "copy" },
        { "nozzle_temperature_initial_layer",  "200" },
        { "nozzle_temperature",                "240" },
    });
    const std::string gcode = slice({ cube(20) }, config);
    // Head 0 runs the print's own toolpaths and must step 200 -> 240 at the second layer.
    CHECK(gcode.find("M104 S240 T0") != std::string::npos);
    // Head 1 is the copy carriage; it already worked and must keep working.
    CHECK(gcode.find("M104 S240 T1") != std::string::npos);
 }
 // IMEX supplements is_extruder_used for the secondary carriages a parallel mode drives.
 // `primary` drives exactly one tool, so the supplement must not run: routing every region
 // to filament 6 puts the initial tool on physical head 2, while the mode's only declared
@@ -54,8 +54,6 @@
 #include <cstdlib>
 #include <sstream>
 #include <limits>
 #include <fstream>
 #include <iterator>
 #include <memory>
 #include <string_view>
 #include <utility>
@@ -794,25 +792,6 @@ TEST_CASE("Print statistics in custom G-code match the final print statistics",
    }
 }
 TEST_CASE("export_gcode writes G-code without a result pointer", "[Print][export_gcode]")
 {
    Print print;
    Model model;
    Slic3r::Test::init_print({cube(20)}, print, model);
    print.process();
    SECTION("non-BBL printer") {}
    SECTION("BBL printer") { print.is_BBL_printer() = true; }
    ScopedTemporaryFile temp(".gcode");
    REQUIRE_NOTHROW(print.export_gcode(temp.string(), nullptr, nullptr));
    std::ifstream in(temp.string());
    const std::string gcode((std::istreambuf_iterator<char>(in)), std::istreambuf_iterator<char>());
    REQUIRE_FALSE(gcode.empty());
 }
 TEST_CASE("Exporting a sliced print again gives the same G-code", "[Print][export_gcode][Regression]")
 {
    const int instances = GENERATE(1, 3);
@@ -1062,34 +1041,6 @@ TEST_CASE("Belt printers keep the part fan off within the band above the belt",
    CHECK(clear_fan_off == 0);
 }
 // Organic supports under an overhang on a belt printer reach below the object's first layer,
 // where the virtual belt raft layers sit at negative Z. The lowest of them used to get a
 // negative height and abort slicing with a negative flow error.
 TEST_CASE("Belt printers slice organic tree supports that reach the belt", "[Print][belt][Support]")
 {
    DynamicPrintConfig config = DynamicPrintConfig::full_print_config();
    config.set_deserialize_strict({
        { "belt_printer",               1 },
        { "belt_slice_rotation",        "x" },
        { "belt_slice_rotation_angle",  45 },
        { "gcode_remap_x",              "rev_x" },
        { "gcode_remap_y",              "pos_z" },
        { "gcode_remap_z",              "pos_y" },
        { "layer_height",               0.2 },
        { "initial_layer_print_height", 0.2 },
        { "skirt_loops",                0 },
        { "z_hop",                      0 },
        { "enable_support",             1 },
        { "support_type",               "tree(auto)" },
        { "support_style",              "organic" },
        { "machine_start_gcode",        "T[initial_tool]\n" },
        { "layer_change_gcode",         "G92 E0\n" },
    });
    std::string gcode;
    REQUIRE_NOTHROW(gcode = slice({ TestMesh::overhang }, config));
    CHECK(! gcode.empty());
 }
 TEST_CASE("Slicing errors are reported per object with the object's name", "[Print]")
 {
    Print print;
@@ -837,30 +837,6 @@ TEST_CASE("Belt brim on an object layer precedes its perimeters, once", "[SkirtB
    CHECK(role_passes(gc, "brim") <= bands);
 }
 // B - single extruder (filament id 1).  Every band must survive the 1-based -> 0-based
 // filament-id conversion the apron path performs: a wrong conversion drops all single-extruder
 // bands, so the pass count would collapse.  The expected count is derived from the sliced
 // layers, not a ratio.
 TEST_CASE("Belt brim on a single extruder emits every band once", "[SkirtBrim][belt]")
 {
    DynamicPrintConfig config = belt_brim_config();
    config.set_deserialize_strict({
        { "brim_type",       "outer_only" },
        { "brim_width",      4 },
        { "brim_object_gap", 0 },
    });
    Print print;
    Model model;
    init_print({ cube(20) }, print, model, config);
    const std::string gc = gcode(print);
    const PrintObject &object = *print.objects().front();
    const int expected = nonempty_belt_brim_layers(object) + belt_brim_apron_bands(object);
    REQUIRE(expected > 0);
    CHECK(role_layers(gc, "brim") == expected);
    CHECK(belt_tools_for_role(gc, "brim") == std::set<int>{ 0 });   // filament 1 -> tool 0
 }
 // Number of brim segments the belt brim generator produced for `object`: the lattice
 // lines of every per-layer band plus the apron prologue.  Each segment is written as one
 // extruding move, so this is what a G-code count has to match.  A pass count cannot see a
@@ -27,14 +27,6 @@ TEST_CASE("slicing_pipeline_plugin option exists and defaults empty", "[slicing_
 #include "libslic3r/Print.hpp"
 TEST_CASE("slicing pipeline hook setter is a no-op-safe injection", "[slicing_pipeline]") {
    int calls = 0;
    Slic3r::Print::set_slicing_pipeline_hook_fn(
        [&](Slic3r::Print&, const Slic3r::PrintObject*, Slic3r::SlicingPipelineStepPlugin){ ++calls; });
    Slic3r::Print::set_slicing_pipeline_hook_fn(nullptr); // reset — must be legal
    CHECK(calls == 0);
 }
 #include <vector>
 #include <algorithm>
 using namespace Slic3r::Test;
@@ -10,8 +10,6 @@
 #include <cstdlib>
 #include <cstddef>
 #include <cmath>
 #include <future>
 #include <chrono>
 #include <vector>
 #include <math.h>
 #include "libslic3r/ExPolygon.hpp"
@@ -59,12 +57,6 @@ SCENARIO("Basic mesh statistics", "[TriangleMesh]") {
            REQUIRE(cube.facets_count() == facets.size());
        }
 #if 0
        THEN( "Number of normals is equal to the number of facets.") {
            REQUIRE(cube.normals().size() == facets.size());
        }
 #endif
        THEN( "center() returns the center of the object.") {
            REQUIRE(cube.center() == Vec3d(10.0,10.0,10.0));
        }
@@ -265,20 +257,6 @@ SCENARIO("Split functionality", "[TriangleMesh]") {
    }
 }
 SCENARIO("Mesh merge functions", "[TriangleMesh]") {
    GIVEN( "Two 20mm cubes, each with one corner on the origin") {
 		auto cube = make_cube();
 		TriangleMesh cube2(cube);
        WHEN( "The two meshes are merged") {
            cube.merge(cube2);
            THEN( "There are twice as many facets in the merged mesh as the original.") {
                REQUIRE(cube.facets_count() == 2 * cube2.facets_count());
            }
        }
    }
 }
 SCENARIO("Cut behavior", "[TriangleMesh]") {
    GIVEN( "A 20mm cube with one corner on the origin") {
 		auto cube = make_cube();
@@ -306,53 +284,3 @@ SCENARIO("Cut behavior", "[TriangleMesh]") {
        }
    }
 }
 #ifdef TEST_PERFORMANCE
 TEST_CASE("Large mesh slices within the time budget (#4486)", "[TriangleMesh][Regression]") {
    TriangleMesh mesh;
    DynamicPrintConfig config = Slic3r::DynamicPrintConfig::full_print_config();
    mesh.ReadSTLFile(std::string(testfile_dir) + "test_trianglemesh/4486/100_000.stl");
    config.set("layer_height", 500);
    config.set("first_layer_height", 250);
    config.set("nozzle_diameter", 500);
    Slic3r::Print print;
    Slic3r::Model model;
    Slic3r::Test::init_print({mesh}, print, model, config);
    print.status_cb = [] (int ln, const std::string& msg) { Slic3r::Log::info("Print") << ln << " " << msg << "\n";};
    std::future<void> fut = std::async([&print] () { print.process(); });
    std::chrono::milliseconds span {120000};
    bool timedout {false};
    if(fut.wait_for(span) == std::future_status::timeout) {
        timedout = true;
    }
    REQUIRE(timedout == false);
 }
 #endif // TEST_PERFORMANCE
 #ifdef BUILD_PROFILE
 TEST_CASE("Large mesh slicing profile (#4486)", "[TriangleMesh][Profile]") {
    TriangleMesh mesh;
    DynamicPrintConfig config = Slic3r::DynamicPrintConfig::full_print_config();
    mesh.ReadSTLFile(std::string(testfile_dir) + "test_trianglemesh/4486/10_000.stl");
    config.set("layer_height", 500);
    config.set("first_layer_height", 250);
    config.set("nozzle_diameter", 500);
    config.set("fill_density", "5%");
    Slic3r::Print print;
    Slic3r::Model model;
    Slic3r::Test::init_print({mesh}, print, model, config);
    print.status_cb = [] (int ln, const std::string& msg) { Slic3r::Log::info("Print") << ln << " " << msg << "\n";};
    print.process();
    REQUIRE(true);
 }
 #endif //BUILD_PROFILE
@@ -348,21 +348,6 @@ TEST_CASE("Inactive inward wipe settings preserve the exported trajectory", "[Wi
    CHECK_FALSE(trajectories_differ(regular_paths, inward_paths));
 }
 TEST_CASE("Inward wipe changes the exported trajectory when outer wall width is Auto", "[Wipe][Regression]")
 {
    const char *wall_generator = GENERATE("classic", "arachne");
    INFO("wall generator: " << wall_generator);
    const std::vector<Vec2d> regular = wipe_destinations(
        slice({make_cube(10., 10., 1.)}, wipe_config(wall_generator, false)));
    const std::vector<Vec2d> inward = wipe_destinations(
        slice({make_cube(10., 10., 1.)}, wipe_config(wall_generator, true)));
    REQUIRE_FALSE(regular.empty());
    REQUIRE_FALSE(inward.empty());
    REQUIRE(trajectories_differ(regular, inward));
 }
 TEST_CASE("Inward wipe recognizes an external wall starting on an overhang", "[Wipe][Regression]")
 {
    const char *wall_generator = GENERATE("classic", "arachne");
@@ -1647,69 +1647,6 @@ SCENARIO("Legacy 3MF without published metadata loads unchanged", "[3mf]") {
    }
 }
 // Locks the serialization contract of the orca_published_material_keys metadata: the per-entry JSON
 // must survive a store_bbs_3mf -> load_bbs_3mf round-trip verbatim, exactly like orca_published_keys.
 SCENARIO("Published 3MF round-trips the published_material_keys metadata", "[3mf]") {
    GIVEN("a model carrying published material keys metadata") {
        Model model;
        std::string src_file = std::string(TEST_DATA_DIR) + "/test_3mf/Prusa.stl";
        REQUIRE(load_stl(src_file.c_str(), &model));
        model.add_default_instances();
        const std::string material_keys_json =
            R"([{"material":{"filament_type":"PLA","filament_vendor":"Generic","filament_id":"GFL99"},"slot":0,"keys":["filament_retraction_length","filament_z_hop"]}])";
        model.model_info = std::make_shared<ModelInfo>();
        model.model_info->metadata_items[ORCA_PUBLISHED_MATERIAL_TAG] = material_keys_json;
        ScopedTemporaryDir backup_dir("orca_pub_mat");
        model.set_backup_path(backup_dir.string());
        WHEN("stored to and reloaded from a .3mf") {
            ScopedTemporaryFile temp(".3mf");
            const std::string test_file = temp.string();
            DynamicPrintConfig config = DynamicPrintConfig::full_print_config();
            StoreParams store_params;
            store_params.path    = test_file.c_str();
            store_params.model   = &model;
            store_params.config  = &config;
            store_params.strategy = SaveStrategy::Zip64 | SaveStrategy::Silence;
            REQUIRE(store_bbs_3mf(store_params));
            Model dst_model;
            DynamicPrintConfig dst_config;
            ConfigSubstitutionContext ctxt{ ForwardCompatibilitySubstitutionRule::Enable };
            PlateDataPtrs        dst_plates;
            std::vector<Preset*> project_presets;
            bool   is_bbl_3mf = false, is_orca_3mf = false;
            Semver file_version;
            bool loaded = load_bbs_3mf(test_file.c_str(), &dst_config, &ctxt, &dst_model, &dst_plates,
                                       &project_presets, &is_bbl_3mf, &is_orca_3mf, &file_version, nullptr,
                                       LoadStrategy::LoadModel | LoadStrategy::LoadConfig);
            THEN("the published material keys metadata round-trips unchanged") {
                REQUIRE(loaded);
                REQUIRE(dst_model.model_info != nullptr);
                REQUIRE(dst_model.model_info->metadata_items[ORCA_PUBLISHED_MATERIAL_TAG] == material_keys_json);
                // The value must parse back to one material entry carrying the nested identity
                // object, the author slot ordinal and the key list.
                nlohmann::json entries = nlohmann::json::parse(material_keys_json);
                REQUIRE(entries.is_array());
                REQUIRE(entries.size() == 1);
                REQUIRE(entries[0]["material"]["filament_type"] == "PLA");
                REQUIRE(entries[0]["material"]["filament_vendor"] == "Generic");
                REQUIRE(entries[0]["material"]["filament_id"] == "GFL99");
                REQUIRE(entries[0]["slot"] == 0);
                REQUIRE(entries[0]["keys"].is_array());
                REQUIRE(entries[0]["keys"].size() == 2);
                REQUIRE(entries[0]["keys"][0] == "filament_retraction_length");
            }
            release_PlateData_list(dst_plates);
        }
    }
 }
 SCENARIO("Minimal published 3MF omits project config, preset dumps and slicer tags", "[3mf]") {
    GIVEN("a multi-instance model carrying published metadata and a published_config payload") {
        Model model;
@@ -159,15 +159,6 @@ TEST_CASE("overlap_area detects overlap and ignores touching edges", "[Arrange]"
    }
 }
 TEST_CASE("Arrange places every item on the physical bed", "[Arrange]")
 {
    ArrangePolygons items = squares(5, 20.);
    arrange(items, bed(200, 200), quiet_params(scaled(1.)));
    for (const ArrangePolygon &ap : items)
        REQUIRE(ap.bed_idx == 0);
 }
 TEST_CASE("Arranged items stay within the bed", "[Arrange]")
 {
    ArrangePolygons items = squares(6, 30.);
@@ -2061,49 +2061,6 @@ TEST_CASE("mass properties: invalid body index", "[CadDocument]")
    REQUIRE_FALSE(mp3.valid);
 }
 TEST_CASE("serialize_recipe roundtrip with two bodies", "[CadDocument]")
 {
    using Catch::Matchers::WithinRel;
    CadDocument doc;
    // Body 1: rectangle sketch + extrude + fillet
    int sk1 = doc.add_sketch(SketchShape::Rectangle, SketchPlane::XY(),
                             20, 20, 10, "Rect1");
    REQUIRE(sk1 >= 0);
    doc.add_extrude(sk1, 10.0, false, BooleanMode::New, "Extrude1");
    doc.add_fillet(2.0, FaceGroup::All, "Fillet1");
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    // Body 2: circle sketch + extrude (separate New body)
    SketchEntity circ;
    circ.type   = SketchEntity::Type::Circle;
    circ.center = Vec2d(0, 0);
    circ.radius = 8.0;
    int sk2 = doc.add_sketch_entities({circ}, SketchPlane::XZ(), "Circle2");
    doc.add_extrude(sk2, 6.0, false, BooleanMode::New, "Extrude2");
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    REQUIRE(doc.bodies.size() >= 2);
    std::vector<double> orig_vols;
    for (const auto& b : doc.bodies)
        orig_vols.push_back(double(SketchEngine::tessellate(b.shape).volume()));
    auto blob = doc.serialize_recipe();
    REQUIRE_FALSE(blob.empty());
    CadDocument doc2;
    REQUIRE(doc2.deserialize_recipe(blob));
    REQUIRE(doc2.error.empty());
    REQUIRE(doc2.bodies.size() == doc.bodies.size());
    for (size_t i = 0; i < doc.bodies.size(); ++i) {
        double v2 = double(SketchEngine::tessellate(doc2.bodies[i].shape).volume());
        REQUIRE_THAT(v2, WithinRel(orig_vols[i], 1e-6));
    }
 }
 TEST_CASE("deserialize_recipe rejects future version with error", "[CadDocument]")
 {
    CadDocument doc;
@@ -2147,41 +2104,6 @@ TEST_CASE("deserialize_recipe handles empty blob without throwing", "[CadDocumen
    REQUIRE_FALSE(doc.error.empty());
 }
 TEST_CASE("deserialize_recipe error is non-empty on every failure path", "[CadDocument]")
 {
    CadDocument doc;
    auto reset = [&]() { doc = CadDocument{}; };
    // too new
    {
        std::ostringstream oss;
        { cereal::BinaryOutputArchive ar(oss); uint32_t v = 999; ar(v); }
        reset();
        REQUIRE_FALSE(doc.deserialize_recipe(oss.str()));
        REQUIRE_FALSE(doc.error.empty());
    }
    // too old
    {
        std::ostringstream oss;
        { cereal::BinaryOutputArchive ar(oss); uint32_t v = 1; ar(v); }
        reset();
        REQUIRE_FALSE(doc.deserialize_recipe(oss.str()));
        REQUIRE_FALSE(doc.error.empty());
    }
    // truncated / garbage
    {
        reset();
        REQUIRE_FALSE(doc.deserialize_recipe("not a valid blob \x00\x01\x02"));
        REQUIRE_FALSE(doc.error.empty());
    }
    // empty
    {
        reset();
        REQUIRE_FALSE(doc.deserialize_recipe(""));
        REQUIRE_FALSE(doc.error.empty());
    }
 }
 TEST_CASE("a v4 project still opens", "[CadDocument][recipe]")
 {
    // The regression guard for everyone who already has projects: a pre-framing v4
@@ -4209,25 +4131,6 @@ static CadDocument make_golden_doc_v1()
    return doc;
 }
 TEST_CASE("regenerate golden recipe fixture", "[.regen]")
 {
    CadDocument doc = make_golden_doc_v1();
    // ponytail: serialize_recipe() only needs features, recompute is unnecessary
    // for a fixture that exercises the serialization format.
    auto blob = doc.serialize_recipe();
    REQUIRE_FALSE(blob.empty());
    // Version-free on purpose: this writes whatever today's format is, so a version bump does
    // not have to remember to edit it. The version-stamped fixtures beside it are frozen
    // evidence from older builds and are never regenerated.
    std::string path = std::string(TEST_DATA_DIR) + "/cad_recipe_current.bin";
    std::ofstream ofs(path, std::ios::binary);
    REQUIRE(ofs.is_open());
    ofs.write(blob.data(), static_cast<std::streamsize>(blob.size()));
    ofs.close();
    SUCCEED("Fixture written to " << path);
 }
 // The previous format's real blob, kept on disk deliberately. It is the only thing that can
 // prove the version gate does its job: a v3 recipe carries FEWER fields per feature than this
 // build reads, so without the bump to v4 it would have passed the gate and had two ints read
@@ -5692,45 +5595,6 @@ TEST_CASE("surface-loft round-trip serialize/deserialize", "[CadDocument][surfac
 // --- Mate tests (M8a) ---
 TEST_CASE("fastened mate with zero offset/angle", "[CadDocument][mate]")
 {
    using Catch::Matchers::WithinAbs;
    CadDocument doc;
    int sk_box = doc.add_sketch(SketchShape::Rectangle, SketchPlane::XY(), 10, 10, 0, "Box");
    doc.add_extrude(sk_box, 5.0, false, BooleanMode::New, "BoxExt");
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    int cs_fixed = doc.add_coordsys(CoordSysType::PointWorld, Vec3d(5, 5, 5), "CS_Fixed");
    doc.features[cs_fixed].coordsys_body = 0;
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    int sk_cyl = doc.add_sketch(SketchShape::Circle, SketchPlane::XY(), 0, 0, 3, "Cyl");
    doc.add_extrude(sk_cyl, 10.0, false, BooleanMode::New, "CylExt");
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    REQUIRE(doc.bodies.size() == 2);
    int cs_moving = doc.add_coordsys(CoordSysType::PointWorld, Vec3d(0, 0, 5), "CS_Moving");
    doc.features[cs_moving].coordsys_body = 1;
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    int mi = doc.add_mate(0, cs_fixed, cs_moving, 0.0, 0.0, false, "Mate");
    REQUIRE(mi >= 0);
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    GProp_GProps props;
    BRepGProp::VolumeProperties(doc.bodies[1].shape, props);
    gp_Pnt com = props.CentreOfMass();
    REQUIRE_THAT(double(com.X()), WithinAbs(5.0, 1e-4));
    REQUIRE_THAT(double(com.Y()), WithinAbs(5.0, 1e-4));
    REQUIRE_THAT(double(com.Z()), WithinAbs(5.0, 1e-4));
 }
 TEST_CASE("fastened mate with offset", "[CadDocument][mate]")
 {
    using Catch::Matchers::WithinAbs;
@@ -5927,30 +5791,6 @@ TEST_CASE("mate round-trip serialization", "[CadDocument][mate]")
    REQUIRE(mf->mate_flip   == true);
 }
 TEST_CASE("version 2 blob is rejected", "[CadDocument][mate]")
 {
    CadDocument doc;
    int sk = doc.add_sketch(SketchShape::Rectangle, SketchPlane::XY(), 10, 10, 0, "Box");
    doc.add_extrude(sk, 5.0, false, BooleanMode::New, "E");
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    std::ostringstream oss;
    {
        cereal::BinaryOutputArchive ar(oss);
        uint32_t fake_v = 2;
        ar(fake_v);
        ar(doc.features);
        ar(doc.variables);
    }
    std::string blob = oss.str();
    CadDocument fresh;
    REQUIRE_FALSE(fresh.deserialize_recipe(blob));
    REQUIRE_CONTAINS(fresh.error, "older version");
 }
 TEST_CASE("mate error: out of range connectors", "[CadDocument][mate]")
 {
@@ -7693,17 +7533,6 @@ TEST_CASE("two holes are both subtracted", "[CadDocument][sketchwire]")
    REQUIRE_THAT(double(doc.display_mesh.volume()), Catch::Matchers::WithinRel(expected, 0.01));
 }
 TEST_CASE("a lone circle still extrudes exactly as before", "[CadDocument][sketchwire]")
 {
    std::vector<SketchEntity> ents = { circle_entity({0, 0}, 8.0) };
    CadDocument doc = plate_doc(ents, 5.0);
    REQUIRE(doc.recompute());
    REQUIRE(doc.error.empty());
    const double expected = M_PI * 64.0 * 5.0;
    REQUIRE_THAT(double(doc.display_mesh.volume()), Catch::Matchers::WithinRel(expected, 0.01));
 }
 TEST_CASE("a closed polygon still extrudes exactly as before", "[CadDocument][sketchwire]")
 {
    std::vector<SketchEntity> ents = rect_entities(20, 20);
@@ -155,51 +155,6 @@ SCENARIO("Various Clipper operations - xs/t/11_clipper.t", "[ClipperUtils]") {
            REQUIRE(result.front().length() == Catch::Approx(subject.length()));
        }
    }
 #if 0
    {
        # Clipper does not preserve polyline orientation
        my $polyline = Slic3r::Polyline->new([50, 150], [300, 150]);
        my $result = Slic3r::Geometry::Clipper::intersection_pl([$polyline], [$square]);
        is scalar(@$result), 1, 'intersection_pl - correct number of result lines';
        is_deeply $result->[0]->pp, [[100, 150], [200, 150]], 'clipped line orientation is preserved';
    }
    {
        # Clipper does not preserve polyline orientation
        my $polyline = Slic3r::Polyline->new([300, 150], [50, 150]);
        my $result = Slic3r::Geometry::Clipper::intersection_pl([$polyline], [$square]);
        is scalar(@$result), 1, 'intersection_pl - correct number of result lines';
        is_deeply $result->[0]->pp, [[200, 150], [100, 150]], 'clipped line orientation is preserved';
    }
    {
        # Disabled until Clipper bug #127 is fixed
        my $subject = [
            Slic3r::Polyline->new([-90000000, -100000000], [-90000000, 100000000]), # vertical
                Slic3r::Polyline->new([-100000000, -10000000], [100000000, -10000000]), # horizontal
                Slic3r::Polyline->new([-100000000, 0], [100000000, 0]), # horizontal
                Slic3r::Polyline->new([-100000000, 10000000], [100000000, 10000000]), # horizontal
        ];
        my $clip = Slic3r::Polygon->new(# a circular, convex, polygon
            [99452190, 10452846], [97814760, 20791169], [95105652, 30901699], [91354546, 40673664], [86602540, 50000000],
            [80901699, 58778525], [74314483, 66913061], [66913061, 74314483], [58778525, 80901699], [50000000, 86602540],
            [40673664, 91354546], [30901699, 95105652], [20791169, 97814760], [10452846, 99452190], [0, 100000000],
            [-10452846, 99452190], [-20791169, 97814760], [-30901699, 95105652], [-40673664, 91354546],
            [-50000000, 86602540], [-58778525, 80901699], [-66913061, 74314483], [-74314483, 66913061],
            [-80901699, 58778525], [-86602540, 50000000], [-91354546, 40673664], [-95105652, 30901699],
            [-97814760, 20791169], [-99452190, 10452846], [-100000000, 0], [-99452190, -10452846],
            [-97814760, -20791169], [-95105652, -30901699], [-91354546, -40673664], [-86602540, -50000000],
            [-80901699, -58778525], [-74314483, -66913061], [-66913061, -74314483], [-58778525, -80901699],
            [-50000000, -86602540], [-40673664, -91354546], [-30901699, -95105652], [-20791169, -97814760],
            [-10452846, -99452190], [0, -100000000], [10452846, -99452190], [20791169, -97814760],
            [30901699, -95105652], [40673664, -91354546], [50000000, -86602540], [58778525, -80901699],
            [66913061, -74314483], [74314483, -66913061], [80901699, -58778525], [86602540, -50000000],
            [91354546, -40673664], [95105652, -30901699], [97814760, -20791169], [99452190, -10452846], [100000000, 0]
            );
        my $result = Slic3r::Geometry::Clipper::intersection_pl($subject, [$clip]);
        is scalar(@$result), scalar(@$subject), 'intersection_pl - expected number of polylines';
        is sum(map scalar(@$_), @$result), scalar(@$subject) * 2, 'intersection_pl - expected number of points in polylines';
    }
 #endif
 }
 SCENARIO("Various Clipper operations - t/clipper.t", "[ClipperUtils]") {
@@ -643,45 +643,6 @@ SCENARIO("update_non_diff_values_to_base_config keeps a project's changed values
    }
 }
 // SCENARIO("DynamicPrintConfig JSON serialization", "[Config]") {
 //     WHEN("DynamicPrintConfig is serialized and deserialized") {
 // 	auto now = std::chrono::high_resolution_clock::now();
 // 	auto timestamp = now.time_since_epoch().count();
 // 	std::stringstream ss;
 // 	ss << "catch_test_serialization_" << timestamp << ".json";
 // 	std::string filename = (fs::temp_directory_path() / ss.str()).string();
 // TODO: Finish making a unit test for JSON serialization
 //         FullPrintConfig full_print_config;
 //         DynamicPrintConfig cfg;
 //         cfg.apply(full_print_config, false);
 //         std::string serialized;
 //         try {
 //             std::ostringstream ss;
 //             cereal::BinaryOutputArchive oarchive(ss);
 //             oarchive(cfg);
 //             serialized = ss.str();
 //         } catch (const std::runtime_error & /* e */) {
 //             // e.what();
 //         }
 // 	CAPTURE(serialized.length());
 //         THEN("Config object contains ini file options.") {
 //             DynamicPrintConfig cfg2;
 //             try {
 //                 std::stringstream ss(serialized);
 //                 cereal::BinaryInputArchive iarchive(ss);
 //                 iarchive(cfg2);
 //             } catch (const std::runtime_error & /* e */) {
 //                 // e.what();
 //             }
 // 	    CAPTURE(cfg.diff_report(cfg2));
 //             REQUIRE(cfg == cfg2);
 //         }
 //     }
 // }
 TEST_CASE("save_to_json round-trips plugin capability references as strings", "[Config][plugins]") {
    ScopedTemporaryFile tmp(".json");
    const std::vector<std::string> refs = {
@@ -17,10 +17,6 @@ using namespace Slic3r;
 // #define TESTS_EXPORT_SVGS
 namespace Slic3r {
 	Points mittered_offset_path_scaled(const Points& contour, const std::vector<float>& deltas, double miter_limit);
 }
 static ExPolygon spirograph_gear_1mm()
 {
 	ExPolygon out;
@@ -479,33 +475,6 @@ SCENARIO("Elephant foot compensation", "[ElephantFoot]") {
        }
 	}
 #if 0
 	GIVEN("Varying inner offset") {
 		ExPolygon input = spirograph_gear_1mm().simplify(SCALED_EPSILON).front();
 		ExPolygon output;
 		std::vector<float> deltas(input.contour.points.size(), scale_(1.));
                // mittered_offset_path_scaled_points is commented out somewhere above
 		output.contour.points = Slic3r::mittered_offset_path_scaled_points(input.contour.points, deltas, 2.);
 #ifdef TESTS_EXPORT_SVGS
 		{
 			SVG svg(debug_out_path("elephant_foot_compensation_0.svg").c_str(), get_extents(output));
 			svg.draw(input, "blue");
 			svg.draw_outline(output, "black", coord_t(scale_(0.01)));
 		}
 #endif /* TESTS_EXPORT_SVGS */
 		for (size_t i = 0; i <= deltas.size() / 2; ++ i)
 			deltas[i] = deltas[deltas.size() - i - 1] = scale_(1.) * double(i) / (0.5 * double(deltas.size()));
 		output.contour.points = Slic3r::mittered_offset_path_scaled_points(input.contour.points, deltas, 2.);
 #ifdef TESTS_EXPORT_SVGS
 		{
 			SVG svg(debug_out_path("elephant_foot_compensation_varying.svg").c_str(), get_extents(output));
 			svg.draw(input, "blue");
 			svg.draw_outline(output, "black", coord_t(scale_(0.01)));
 		}
 #endif /* TESTS_EXPORT_SVGS */
 	}
 #endif
    GIVEN("Rectangle with a narrow part sticking out") {				
 		// Rectangle
 		ExPolygon expoly;
@@ -208,20 +208,3 @@ TEST_CASE("Octagram spiral smooth factor controls corner curvature", "[FillPlane
    REQUIRE(path_length(half_smooth) < path_length(sharp));
    REQUIRE(invalid_factor == sharp);
 }
 TEST_CASE("Hilbert curve smooth factor controls corner curvature", "[FillPlanePath]")
 {
    const Points sharp       = TestableHilbertCurve().generate_points(0.005);
    const Points half_smooth = TestableHilbertCurve().generate_points(0.005, 0.5);
    const Points full_smooth = TestableHilbertCurve().generate_points(0.005, 1.);
    const Points invalid_factor = TestableHilbertCurve().generate_points(
        0.005, std::numeric_limits<double>::quiet_NaN());
    REQUIRE(full_smooth.front() == half_smooth.front());
    REQUIRE(full_smooth.back() == half_smooth.back());
    REQUIRE(path_length(full_smooth) < path_length(half_smooth));
    REQUIRE(invalid_factor == sharp);
    for (size_t i = 1; i < full_smooth.size(); ++i)
        REQUIRE((full_smooth[i] - full_smooth[i - 1]).squaredNorm() > 0);
 }
@@ -18,8 +18,6 @@
 #include "libslic3r/ClipperUtils.hpp"
 #include "libslic3r/ShortestPath.hpp"
 //#include <random>
 //#include "libnest2d/tools/benchmark.h"
 #include "libslic3r/SVG.hpp"
 #include "../libnest2d/printer_parts.hpp"
@@ -605,73 +603,6 @@ TEST_CASE("Convex polygon intersection on two overlapping squares", "[Geometry][
    REQUIRE(is_inters == true);
 }
 //// Only for benchmarking
 //static Polygon gen_convex_poly(std::mt19937_64 &rg, size_t point_cnt)
 //{
 //    std::uniform_int_distribution<coord_t> dist(0, 100);
 //    Polygon out;
 //    out.points.reserve(point_cnt);
 //    coord_t tr = dist(rg) * 2 / SCALING_FACTOR;
 //    for (size_t i = 0; i < point_cnt; ++i)
 //        out.points.emplace_back(tr + dist(rg) / SCALING_FACTOR,
 //                                tr + dist(rg) / SCALING_FACTOR);
 //    return Geometry::convex_hull(out.points);
 //}
 //TEST_CASE("Convex polygon intersection test on random polygons", "[Geometry]") {
 //    constexpr size_t TEST_CNT = 1000;
 //    constexpr size_t POINT_CNT = 1000;
 //    auto seed = std::random_device{}();
 ////    unsigned long seed = 2525634386;
 //    std::mt19937_64 rg{seed};
 //    Benchmark bench;
 //    auto tests = reserve_vector<std::pair<Polygon, Polygon>>(TEST_CNT);
 //    auto results = reserve_vector<bool>(TEST_CNT);
 //    auto expects = reserve_vector<bool>(TEST_CNT);
 //    for (size_t i = 0; i < TEST_CNT; ++i) {
 //        tests.emplace_back(gen_convex_poly(rg, POINT_CNT), gen_convex_poly(rg, POINT_CNT));
 //    }
 //    bench.start();
 //    for (const auto &test : tests)
 //        results.emplace_back(Geometry::convex_polygons_intersect(test.first, test.second));
 //    bench.stop();
 //    std::cout << "Test time: " << bench.getElapsedSec() << std::endl;
 //    bench.start();
 //    for (const auto &test : tests)
 //        expects.emplace_back(!intersection(test.first, test.second).empty());
 //    bench.stop();
 //    std::cout << "Clipper time: " << bench.getElapsedSec() << std::endl;
 //    REQUIRE(results.size() == expects.size());
 //    auto seedstr = std::to_string(seed);
 //    for (size_t i = 0; i < results.size(); ++i) {
 //        // std::cout << expects[i] << " ";
 //        if (results[i] != expects[i]) {
 //            SVG svg{std::string("fail_seed") + seedstr + "_" + std::to_string(i) + ".svg"};
 //            svg.draw(tests[i].first, "blue");
 //            svg.draw(tests[i].second, "green");
 //            svg.Close();
 //            // std::cout << std::endl;
 //        }
 //        REQUIRE(results[i] == expects[i]);
 //    }
 //    std::cout << std::endl;
 //}
 struct Pair
 {
    size_t first, second;
@@ -71,15 +71,6 @@ TEST_CASE("effective_physical_extruder_map - degenerate nozzle count still yield
    REQUIRE(effective_physical_extruder_map(nullptr, 0).values == std::vector<int>{0});
 }
 TEST_CASE("effective_physical_extruder_map - IDEX ghost-color regression guard", "[IMEX]") {
    // Dual extruder, no authored map. Before the GUI fix this produced a black ghost on T1
    // because first_filament_for_physical_head({0}, 1) == -1.
    auto default_pem = make_pem({0});
    auto pem         = effective_physical_extruder_map(&default_pem, 2);
    REQUIRE(first_filament_for_physical_head(pem, 0) == 0);
    REQUIRE(first_filament_for_physical_head(pem, 1) == 1); // no longer -1
 }
 TEST_CASE("imex_pem_tool_for - non-parallel mode returns -1", "[IMEX]") {
    // Empty mode string (not in IMEX) and "primary" (IMEX present but not parallel) both
    // short-circuit to bare-firmware PA/temp emission.
@@ -1058,12 +1049,6 @@ TEST_CASE("compute_imex_slice_offset - flag on + copy mode returns primary zone
    REQUIRE_THAT(z.y(), WithinAbs(40.0, 1e-9));
 }
 TEST_CASE("compute_imex_slice_offset - flag on + mirror mode returns primary zone center", "[IMEX]") {
    auto z = compute_imex_slice_offset(true, "mirror", kPrimaryZoneCenteredAt60_40);
    REQUIRE_THAT(z.x(), WithinAbs(60.0, 1e-9));
    REQUIRE_THAT(z.y(), WithinAbs(40.0, 1e-9));
 }
 TEST_CASE("compute_imex_slice_offset - user-defined non-primary mode names trigger shift", "[IMEX]") {
    // Mode names are user-defined; anything not "primary" or "" should activate.
    auto z = compute_imex_slice_offset(true, "iq-copy", kPrimaryZoneCenteredAt60_40);
@@ -6,7 +6,6 @@
 #include "libslic3r/Point.hpp"
 #include "libslic3r/libslic3r.h"
 #include <limits>
 #include <random>
 #include <catch2/catch_all.hpp>
 #include <vector>
 #include <string>
@@ -117,68 +116,6 @@ TEST_CASE("Split two watertight meshes", "[its_split][its]") {
 }
 #include <libslic3r/QuadricEdgeCollapse.hpp>
 static float triangle_area(const Vec3f &v0, const Vec3f &v1, const Vec3f &v2)
 {
    Vec3f ab = v1 - v0;
    Vec3f ac = v2 - v0;
    return ab.cross(ac).norm() / 2.f;
 }
 static float triangle_area(const stl_triangle_vertex_indices &triangle_indices, const std::vector<Vec3f> &vertices)
 {
    return triangle_area(vertices[triangle_indices[0]],
                         vertices[triangle_indices[1]],
                         vertices[triangle_indices[2]]);
 }
 #if 0
 // clang complains about unused functions
 static std::mt19937 create_random_generator() {
    std::random_device rd;
    std::mt19937 gen(rd());
    return gen;
 }
 #endif
 std::vector<Vec3f> its_sample_surface(const indexed_triangle_set &its,
                                      double        sample_per_mm2,
                                      std::mt19937 random_generator) // = create_random_generator())
 {
    std::vector<Vec3f> samples;
    std::uniform_real_distribution<float> rand01(0.f, 1.f);
    for (const auto &triangle_indices : its.indices) {
        float area = triangle_area(triangle_indices, its.vertices);
        float countf;
        float fractional = std::modf(area * sample_per_mm2, &countf);
        int count = static_cast<int>(countf);
        float generate = rand01(random_generator);
        if (generate < fractional) ++count;
        if (count == 0) continue;
        const Vec3f &v0 = its.vertices[triangle_indices[0]];
        const Vec3f &v1 = its.vertices[triangle_indices[1]];
        const Vec3f &v2 = its.vertices[triangle_indices[2]];
        for (int c = 0; c < count; c++) {
            // barycentric coordinate
            Vec3f b;
            b[0] = rand01(random_generator);
            b[1] = rand01(random_generator);
            if ((b[0] + b[1]) > 1.f) {
                b[0] = 1.f - b[0];
                b[1] = 1.f - b[1];
            }
            b[2] = 1.f - b[0] - b[1];
            Vec3f pos;
            for (int i = 0; i < 3; i++) {
                pos[i] = b[0] * v0[i] + b[1] * v1[i] + b[2] * v2[i];
            }
            samples.push_back(pos);
        }        
    }
    return samples;
 }
 #include "libslic3r/AABBTreeIndirect.hpp"
@@ -261,16 +261,6 @@ TEST_CASE("snake_core produces valid paths on ring", "[Snake]") {
    REQUIRE(is_permutation(p2, centers.size()));
 }
 // --- Core Strategy Tests: Random Layout ---
 TEST_CASE("snake_core produces valid paths on random input", "[Snake]") {
    Points centers = make_random_16();
    auto p2 = snake_core(centers);
    REQUIRE(is_permutation(p2, centers.size()));
 }
 // --- Quality Comparison Tests ---
 TEST_CASE("snake has no crossings on random input", "[Snake]") {
@@ -730,25 +730,6 @@ TEST_CASE("Neighboring vertices distinguish repeated anchors on different lobes"
    CHECK(edges == std::vector<size_t>{0, 2, 3, 4, 6, 7});
 }
 TEST_CASE("Modifier holes subtract coverage while separate components add intervals", "[PreciseSeam][SegmentExtraction]")
 {
    const Polygon perimeter = rectangle(0, 0, 20, 20);
    ExPolygon area(rectangle(1, -3, 10, 3));
    area.holes.push_back(rectangle(4, -1, 7, 1));
    area.holes.back().reverse();
    const auto result = PreciseSeam::extract_perimeter_segments(PreciseSeam::PreparedPerimeter(perimeter),
        PreciseSeam::prepare_modifier_regions({area, ExPolygon(rectangle(14, -3, 18, 3))}), ModelVolumeType::PRECISE_SEAM_CENTER);
    check_provenance(perimeter, result);
    REQUIRE(result.segments.size() == 3);
    const std::vector<Point> starts{mm(1, 0), mm(7, 0), mm(14, 0)};
    const std::vector<Point> ends{mm(4, 0), mm(10, 0), mm(18, 0)};
    for (size_t i = 0; i < 3; ++i) {
        CHECK(result.segments[i].polyline.points.front() == starts[i]);
        CHECK(result.segments[i].polyline.points.back() == ends[i]);
    }
    CHECK_FALSE(result.full_containment);
 }
 TEST_CASE("Full coverage is distinct from an empty or point-only intersection", "[PreciseSeam][SegmentExtraction]")
 {
    const bool reverse = GENERATE(false, true);
@@ -1219,6 +1219,24 @@ TEST_CASE("find_preset2 auto-matches removed Generic vendor profiles to the libr
    CHECK(bundle.filaments.find_preset2("BrandX Generic PETG", /*auto_match=*/true) == nullptr);
 }
 TEST_CASE("The selected preset is its own base when its parent is not loaded", "[Preset]")
 {
    PresetCollection printers(Preset::TYPE_PRINTER, Preset::printer_options(),
                              static_cast<const PrintRegionConfig &>(FullPrintConfig::defaults()));
    add_inmemory_preset(printers, "Printer");
    add_inmemory_preset(printers, "User Printer", "Printer");
    add_inmemory_preset(printers, "Orphan Printer", "Uninstalled Vendor Printer");
    printers.select_preset_by_name("User Printer", true);
    REQUIRE(printers.get_selected_preset_name() == "User Printer");
    CHECK(printers.get_selected_preset_base().name == "Printer");
    printers.select_preset_by_name("Orphan Printer", true);
    REQUIRE(printers.get_selected_preset_name() == "Orphan Printer");
    REQUIRE(printers.get_preset_base(printers.get_selected_preset()) == nullptr);
    CHECK(printers.get_selected_preset_base().name == "Orphan Printer");
 }
 TEST_CASE("Renamed parent is normalized into a loaded preset's inherits", "[Preset][Rename]")
 {
    ScopedTemporaryDir   temp_dir;
@@ -239,30 +239,6 @@ TEST_CASE("DistanceX on two Points binds real roles with non-negative prefill",
    REQUIRE(p.defs[0].eb == 0);
 }
 TEST_CASE("Horizontal on a Point rejects with NeedALine", "[SketchConstraints]")
 {
    std::vector<SketchEntity> ents = { mk_point(1, 2) };
    ConstraintPlan p = plan_entity_constraint(ents, 0, -1, -1, SketchConstraintType::Horizontal);
    REQUIRE(p.kind == ConstraintPlan::Kind::Reject);
    REQUIRE(p.reason == ConstraintReject::NeedALine);
 }
 TEST_CASE("Angle on two Circles rejects with NeedTwoLines", "[SketchConstraints]")
 {
    std::vector<SketchEntity> ents = { mk_circle(0, 0, 1), mk_circle(5, 0, 1) };
    ConstraintPlan p = plan_entity_constraint(ents, 0, 1, -1, SketchConstraintType::Angle);
    REQUIRE(p.kind == ConstraintPlan::Kind::Reject);
    REQUIRE(p.reason == ConstraintReject::NeedTwoLines);
 }
 TEST_CASE("Parallel on a Line + Circle rejects with NeedTwoLines (new guard)", "[SketchConstraints]")
 {
    std::vector<SketchEntity> ents = { mk_line(0, 0, 1, 0), mk_circle(5, 0, 1) };
    ConstraintPlan p = plan_entity_constraint(ents, 0, 1, -1, SketchConstraintType::Parallel);
    REQUIRE(p.kind == ConstraintPlan::Kind::Reject);
    REQUIRE(p.reason == ConstraintReject::NeedTwoLines);
 }
 TEST_CASE("Equal on two Circles promotes to EqualRadius", "[SketchConstraints]")
 {
    std::vector<SketchEntity> ents = { mk_circle(0, 0, 1), mk_circle(5, 0, 2) };
@@ -44,17 +44,6 @@ SCENARIO("Reading an STL file", "[stl]") {
 				REQUIRE(is_approx(model.objects.front()->volumes.front()->mesh().size(), Vec3d(20, 20, 20)));
 			}
 		}
 #if 0
 		// ASCII STLs ending with just carriage returns are not supported. These were used by the old Macs, while the Unix based MacOS uses LFs as any other Unix.
 		WHEN("line endings CR") {
 			Slic3r::Model model;
 			THEN("load should succeed") {
 				REQUIRE(Slic3r::load_stl(stl_path("ASCII/20mmbox-CR.stl").c_str(), &model));
 				REQUIRE(is_approx(model.objects.front()->volumes.front()->mesh().size(), Vec3d(20, 20, 20)));
 			}
 		}
 #endif
 		WHEN("nonstandard STL file (text after ending tags, invalid normals, for example infinities)") {
 			Slic3r::Model model;
 			THEN("load should succeed") {
@@ -3285,33 +3285,6 @@ TEST_CASE("a ModelUV layer falls back to Triplanar when the volume carries no UV
        CHECK_THAT((no_channel.vertices[i] - as_triplanar.vertices[i]).norm(), WithinAbs(0.f, 1e-5f));
 }
 TEST_CASE("a bake with colour off on every layer reports no colour at all", "[TextureDisplacement]")
 {
    const indexed_triangle_set cube = its_make_cube(10., 10., 10.);
    TextureDisplacementLayer layer;
    layer.slot          = 0;
    layer.depth_mm      = 1.0f;
    layer.tiling_scale  = 5.0f;
    layer.image_data    = make_flat_rgb_png(200, 40, 10); // has colour to give...
    layer.color_enabled = false;                          // ...but is not asked for it
    TextureDisplacementFacetsData facets{};
    facets[0] = paint_whole_mesh(cube);
    std::vector<uint8_t> triangle_color;
    TextureColorRequest  request;
    request.quantize     = [](const Vec3f &) { return 0; }; // would colour everything, if asked
    request.out_triangle = &triangle_color;
    const indexed_triangle_set out = build_texture_displacement(cube, {layer}, facets, classic_options(), {}, &request);
    REQUIRE_FALSE(out.vertices.empty());
    // Every entry 0 ("this triangle takes no colour from the texture"), so the commit leaves whatever
    // the model was already painted with exactly as it was.
    for (const uint8_t entry : triangle_color)
        CHECK(entry == 0);
 }
 TEST_CASE("transfer_uv_channel resamples a UV channel onto a subdivided copy of the same surface",
          "[TextureDisplacement]")
 {
@@ -1110,28 +1110,6 @@ TEST_CASE("a cache with the wrong cache version is rejected", "[VendorCache]")
    REQUIRE(!out.load_vendor_cache(cache.string(), vid, Semver("1.0.0")));
 }
 TEST_CASE("a cache truncated mid-blob is rejected", "[VendorCache]")
 {
    TempDir           tmp;
    const std::string vid   = "Acme";
    const fs::path     cache = tmp.path / "vendor.opc";
    REQUIRE(save_one_vendor(cache.string(), one_vendor(vid), vid, "1.0.0",
                            {filament_entry(vid + " PLA")}));
    {
        std::ifstream in(cache.string(), std::ios::binary);
        std::vector<char> buf(30); // 20-byte header + 10 bytes of blob
        in.read(buf.data(), 30);
        in.close();
        std::ofstream out(cache.string(), std::ios::binary | std::ios::trunc);
        out.write(buf.data(), 30);
    }
    PresetBundle out;
    REQUIRE(!out.load_vendor_cache(cache.string(), vid, Semver("1.0.0")));
 }
 TEST_CASE("printer model bed texture fields survive a cache round-trip", "[VendorCache]")
 {
    TempDir           tmp;
@@ -34,53 +34,6 @@ using namespace Slic3r;
 using VD = Geometry::VoronoiDiagram;
 using BoostVD = boost::polygon::voronoi_diagram<double>;
 // https://svn.boost.org/trac10/ticket/12067
 // This bug seems to be confirmed.
 // Vojtech supposes that there may be no Voronoi edges produced for
 // the 1st and last sweep line positions.
 TEST_CASE("Voronoi missing edges - points 12067", "[Voronoi]")
 {
    Points pts {
        { -10, -20 },
        {  10, -20 },
        {   5,   0 },
        {  10,  20 },
        { -10,  20 },
        {  -5,   0 }
    };
 #if 0
    for (Point &p : pts) {
        Vec2d q = p.cast<double>();
        p.x() = scale_(p.x());
        p.y() = scale_(p.y());
    }
 #endif
 #if 0
    // Try to rotate, maybe the issue is caused by incorrect handling of vertical or horizontal edges?
    double a = 0.18764587962597876897475f;
    double c = cos(a);
    double s = sin(a);
    for (Point &p : poly.points) {
        Vec2d q = p.cast<double>();
        p.x() = coord_t(q.x() * c + q.y() * s + 0.5);
        p.y() = coord_t(- q.x() * s + q.y() * c + 0.5);
    }
 #endif
    // Construction of the Voronoi Diagram.
    BoostVD vd;
    construct_voronoi(pts.begin(), pts.end(), &vd);
 #ifdef VORONOI_DEBUG_OUT
    dump_voronoi_to_svg(debug_out_path("voronoi-pts.svg").c_str(),
        vd, pts, Lines());
 #endif
 //    REQUIRE(closest_point.z() == Catch::Approx(1.));
 }
 // https://svn.boost.org/trac10/ticket/12707
 // This issue is confirmed, there are no self intersections in the polygon.
 // A minimal test case is created at the end of this test,
@@ -315,36 +268,6 @@ TEST_CASE("Voronoi weirdness", "[Voronoi]")
 #endif
 }
 // https://svn.boost.org/trac10/ticket/12903
 // division by zero reported, but this issue is most likely a non-issue, as it produces an infinity for the interval of validity
 // of the floating point calculation, therefore forcing a recalculation with extended accuracy.
 TEST_CASE("Voronoi division by zero 12903", "[Voronoi]")
 {
    Points pts { { 1, 1 }, { 3, 1 }, { 1, 3 }, { 3, 3 },
                 { -1, 1 }, { 1, -1 }, { 5, 1 }, { 3, -1 },
                 { -1, 3 }, { 1, 5 }, { 5, 3 }, { 3, 5 } };
    {
        auto pts2 { pts };
        std::sort(pts2.begin(), pts2.end(), [](auto &l, auto &r) { return (l.x() == r.x()) ? l.y() < r.y() : l.x() < r.x(); });
        // No point removed -> no duplicate.
        REQUIRE(std::unique(pts2.begin(), pts2.end()) == pts2.end());
    }
    BoostVD vd;
    construct_voronoi(pts.begin(), pts.end(), &vd);
 #ifdef VORONOI_DEBUG_OUT
    // Scale the voronoi vertices and input points, so that the dump_voronoi_to_svg will display them correctly.
    for (auto &pt : vd.vertices()) {
        const_cast<double&>(pt.x()) = scale_(pt.x());
        const_cast<double&>(pt.y()) = scale_(pt.y());
    }
    for (auto &pt : pts)
        pt = Point::new_scale(pt.x(), pt.y());
    dump_voronoi_to_svg(debug_out_path("voronoi-div-by-zero.svg").c_str(), vd, pts, Lines());
 #endif
 }
 // https://svn.boost.org/trac10/ticket/12139
 // Funny sample from a dental industry?
 // Vojtech confirms this test fails and rightly so, because the input data contain self intersections.
@@ -1383,15 +1306,6 @@ TEST_CASE("Voronoi offset", "[VoronoiOffset]")
            OffsetTest { scale_(0.55), 1, 0 }
      }) {
 #if 0
      Polygons offsetted_polygons_out = Slic3r::Voronoi::offset(vd, lines, ot.distance, scale_(0.005));
 #ifdef VORONOI_DEBUG_OUT
      dump_voronoi_to_svg(debug_out_path("voronoi-offset-out-%lf.svg", ot.distance).c_str(),
          vd, Points(), lines, offsetted_polygons_out);
 #endif
      REQUIRE(offsetted_polygons_out.size() == ot.num_outer);
 #endif
      Polygons offsetted_polygons_in = Slic3r::Voronoi::offset(vd, lines, - ot.distance, scale_(0.005));
 #ifdef VORONOI_DEBUG_OUT
      dump_voronoi_to_svg(debug_out_path("voronoi-offset-in-%lf.svg", ot.distance).c_str(),
@@ -1,61 +1 @@
 #include <catch2/catch_all.hpp>
 #include <catch2/catch_test_macros.hpp>
 #include "slic3r/Utils/Http.hpp"
 TEST_CASE("Check SSL certificates paths", "[Http][NotWorking]") {
    Slic3r::Http g = Slic3r::Http::get("https://github.com/");
    unsigned status = 0;
    g.on_error([&status](std::string, std::string, unsigned http_status) {
        status = http_status;
    });
    g.on_complete([&status](std::string /* body */, unsigned http_status){
        status = http_status;
    });
    g.perform_sync();
    REQUIRE(status == 200);
 }
 TEST_CASE("Http digest authentication", "[Http][NotWorking]") {
    Slic3r::Http g = Slic3r::Http::get("https://httpbingo.org/digest-auth/auth/guest/guest");
    g.auth_digest("guest", "guest");
    unsigned status = 0;
    g.on_error([&status](std::string, std::string, unsigned http_status) {
        status = http_status;
    });
    g.on_complete([&status](std::string /* body */, unsigned http_status){
        status = http_status;
    });
    g.perform_sync();
    REQUIRE(status == 200);
 }
 TEST_CASE("Http basic authentication", "[Http][NotWorking]") {
    Slic3r::Http g = Slic3r::Http::get("https://httpbingo.org/basic-auth/guest/guest");
    g.auth_basic("guest", "guest");
    unsigned status = 0;
    g.on_error([&status](std::string, std::string, unsigned http_status) {
        status = http_status;
    });
    g.on_complete([&status](std::string /* body */, unsigned http_status){
        status = http_status;
    });
    g.perform_sync();
    REQUIRE(status == 200);
 }
@@ -73,34 +73,6 @@ TEST_CASE("ActionRegistry takes exclusive ownership of published actions", "[Act
    STATIC_CHECK(std::is_same_v<decltype(&ActionRegistry::upsert), ExpectedUpsert>);
 }
 // A dynamic "Go to Plate N" action is keyed by plate index (not the display title), so renaming
 // a plate never re-keys it - the same contract as a setting action.
 TEST_CASE("Go-to-plate actions are keyed by index, not title", "[ActionSource][SpeedDial]")
 {
    CHECK(AppAction::compose_id("orca_plate_goto", "0", "orca") == "orca_plate_goto:0:orca");
    CHECK(AppAction::compose_id("orca_plate_goto", "2", "orca") == "orca_plate_goto:2:orca");
 }
 // A dynamic "Open recent project" action is keyed by file path (not the display name), so renaming
 // a project or reordering the recents list never re-keys it - the same contract as a setting action.
 TEST_CASE("Recent-project actions are keyed by path, not title", "[ActionSource][SpeedDial]")
 {
    CHECK(AppAction::compose_id("orca_recent_project", "/a/b/project.3mf", "orca") ==
          "orca_recent_project:/a/b/project.3mf:orca");
    CHECK(AppAction::compose_id("orca_recent_project", "C:/Data/cube.3mf", "orca") ==
          "orca_recent_project:C:/Data/cube.3mf:orca");
 }
 // A built-in command is keyed by its stable catalog key (not the localized display title), so a
 // rename or a UI-language switch never re-keys the action and its persisted favourite/stats survive.
 TEST_CASE("Command actions are keyed by catalog key, not display title", "[ActionSource][SpeedDial]")
 {
    CHECK(AppAction::compose_id("orca_command", "save_project", "orca") == "orca_command:save_project:orca");
    // The second field is the stable key, so distinct commands never collide.
    CHECK(AppAction::compose_id("orca_command", "save_project", "orca") !=
          AppAction::compose_id("orca_command", "load_project", "orca"));
 }
 // The real catalog -> action mapping keys by the stable catalog key and copies presentation from the
 // catalog, so a rename or a UI-language switch never re-keys the action. The catalog's English title
 // and group become the action's English search aliases; no translation catalog is loaded in the tests,
@@ -138,22 +138,6 @@ TEST_CASE("PluginConfig serializes the documented on-disk schema", "[PluginConfi
    CHECK(entry.size() == 5);
 }
 TEST_CASE("PluginConfig keeps a capability's config after its plugin goes away", "[PluginConfig]")
 {
    ScopedDataDir data_dir_guard("plugin-config-retention");
    {
        PluginConfig config;
        REQUIRE(config.store_capability_config(capability_id(PluginCapabilityType::Script, "cap_a", "plugin_a"), json{{"token", "keep me"}}));
    }
    // config.json is deliberately not keyed to installed plugins: a record outlives its plugin and is
    // still there on reinstall. Asserts no cleanup path silently drops it.
    PluginConfig after_removal;
    after_removal.load();
    CHECK(after_removal.get_config(capability_id(PluginCapabilityType::Script, "cap_a", "plugin_a"))->config == json{{"token", "keep me"}});
 }
 TEST_CASE("PluginConfig treats a missing config file as an empty store", "[PluginConfig]")
 {
    ScopedDataDir data_dir_guard("plugin-config-missing");
@@ -615,8 +615,22 @@ TEST_CASE("Python module release removes package submodules and owned sys.path",
    // Ensure the next import executes the new submodule rather than reusing a stale package child.
    write_package("'new'");
-    boost::system::error_code ec;
+    {
-    fs::remove_all(package_root / "__pycache__", ec);
+        // The rewrite can keep the old mtime and size, so drop the bytecode wherever the
        // interpreter caches it.
        PythonGILState gil;
        REQUIRE(gil);
        PyObjectPtr util(PyImport_ImportModule("importlib.util"));
        REQUIRE(util);
        for (const char* file : {"__init__.py", "sub.py"}) {
            PyObjectPtr source(PyUnicode_DecodeFSDefault((package_root / file).string().c_str()));
            REQUIRE(source);
            PyObjectPtr cached(PyObject_CallMethod(util.get(), "cache_from_source", "O", source.get()));
            REQUIRE(cached);
            boost::system::error_code ec;
            fs::remove(fs::path(PyUnicode_AsUTF8(cached.get())), ec);
        }
    }
    paths.clear();
    modules.clear();
Author	SHA1	Message	Date
Hanif Koh	b2615a7816	Keep the Segment-Based Voronoi Reproducers The gapfill, weirdness and NaN reproducers go through Orca's repairing VoronoiDiagram once they use it instead of raw boost, so they are converted rather than deleted. Only the two point-input reproducers, which the wrapper neither checks nor repairs, are removed.	2026-10-19 16:58:02 +08:00
Hanif Koh	8a4027ce8c	Include Model.hpp in the fff_print Test Helpers	2026-10-19 15:45:35 +08:00
Hanif Koh	e57945179b	Remove Disabled, Duplicate and Library-Only Unit Tests Delete test cases that never run (#if 0 blocks, [NotWorking] cases, commented-out Perl ports, blocks behind undefined macros), cases that cannot fail, cases that only exercise boost::polygon directly, and cases whose assertions another test already makes on the same path. Remove the helpers, includes and fixture only those cases used.	2026-10-19 15:40:06 +08:00
HanifKoh	d3699dd9f6	Keep Python Bytecode out of the Bundled Python Home (#16550 ) * Keep Python Bytecode out of the Bundled Python Home The bundled stdlib ships .pyc files that are stale against their sources: they are compiled when the Python deps are installed, and the copy into the app does not keep the sources' mtimes. CPython then recompiles every module it imports and writes the result next to the source. A drag-installed macOS app rewrites files inside its own bundle on first launch, so its signature no longer verifies, and read-only installs (AppImage, Flatpak, Program Files) recompile the same modules on every launch. Point pycache_prefix at <data_dir>/python/pycache so the interpreter reads and writes bytecode there. The bundle is never written, the cache survives between launches, and the data directory is already a plugin read/write root. * Drop Cached Bytecode Through the Interpreter in the Module Release Test	2026-10-19 13:48:53 +08:00
HanifKoh	a315a96c5c	Show "No Available Printers" Instead of an Empty Printer Dropdown (#16611 ) * Show "No Available Printers" Instead of an Empty Printer Dropdown A ComboBox with no items opened a one-row-high popup that DropDown never paints, leaving a blank strip with stale pixels under the box. Don't open the list when it is empty, and label the printer box in the print and send-to-printer dialogs when no printer is available. * Add "No Available Printers" to the Translation Template	2026-10-19 13:30:36 +08:00
HanifKoh	24cf1c6bcc	Fix Crash Selecting a User Printer Whose Parent Preset Is Not Loaded (#16579 ) A user preset can inherit a system preset from a vendor that is not installed. get_preset_base() returns null for it, and get_selected_preset_base() dereferenced that unconditionally, so picking the printer crashed while building the shared profiles notification URL. Fall back to the selected preset itself. The create printer and create filament dialogs had the same null dereference on such presets.	2026-10-19 13:29:26 +08:00
		`@@ -1 +0,0 @@`
			solid STL generated by MeshLab␍ facet normal 0.000000e+00 -0.000000e+00 -1.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 0.000000e+00␍ vertex 2.000000e+01 0.000000e+00 0.000000e+00␍ vertex 0.000000e+00 0.000000e+00 0.000000e+00␍ endloop␍ endfacet␍ facet normal -0.000000e+00 0.000000e+00 -1.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 0.000000e+00␍ vertex 0.000000e+00 0.000000e+00 0.000000e+00␍ vertex 0.000000e+00 2.000000e+01 0.000000e+00␍ endloop␍ endfacet␍ facet normal 0.000000e+00 0.000000e+00 1.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 2.000000e+01␍ vertex 0.000000e+00 2.000000e+01 2.000000e+01␍ vertex 0.000000e+00 0.000000e+00 2.000000e+01␍ endloop␍ endfacet␍ facet normal 0.000000e+00 0.000000e+00 1.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 2.000000e+01␍ vertex 0.000000e+00 0.000000e+00 2.000000e+01␍ vertex 2.000000e+01 0.000000e+00 2.000000e+01␍ endloop␍ endfacet␍ facet normal 1.000000e+00 0.000000e+00 -0.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 0.000000e+00␍ vertex 2.000000e+01 2.000000e+01 2.000000e+01␍ vertex 2.000000e+01 0.000000e+00 2.000000e+01␍ endloop␍ endfacet␍ facet normal 1.000000e+00 0.000000e+00 0.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 0.000000e+00␍ vertex 2.000000e+01 0.000000e+00 2.000000e+01␍ vertex 2.000000e+01 0.000000e+00 0.000000e+00␍ endloop␍ endfacet␍ facet normal 0.000000e+00 -1.000000e+00 0.000000e+00␍ outer loop␍ vertex 2.000000e+01 0.000000e+00 0.000000e+00␍ vertex 2.000000e+01 0.000000e+00 2.000000e+01␍ vertex 0.000000e+00 0.000000e+00 2.000000e+01␍ endloop␍ endfacet␍ facet normal 0.000000e+00 -1.000000e+00 0.000000e+00␍ outer loop␍ vertex 2.000000e+01 0.000000e+00 0.000000e+00␍ vertex 0.000000e+00 0.000000e+00 2.000000e+01␍ vertex 0.000000e+00 0.000000e+00 0.000000e+00␍ endloop␍ endfacet␍ facet normal -1.000000e+00 0.000000e+00 0.000000e+00␍ outer loop␍ vertex 0.000000e+00 0.000000e+00 0.000000e+00␍ vertex 0.000000e+00 0.000000e+00 2.000000e+01␍ vertex 0.000000e+00 2.000000e+01 2.000000e+01␍ endloop␍ endfacet␍ facet normal -1.000000e+00 0.000000e+00 0.000000e+00␍ outer loop␍ vertex 0.000000e+00 0.000000e+00 0.000000e+00␍ vertex 0.000000e+00 2.000000e+01 2.000000e+01␍ vertex 0.000000e+00 2.000000e+01 0.000000e+00␍ endloop␍ endfacet␍ facet normal 0.000000e+00 1.000000e+00 0.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 2.000000e+01␍ vertex 2.000000e+01 2.000000e+01 0.000000e+00␍ vertex 0.000000e+00 2.000000e+01 0.000000e+00␍ endloop␍ endfacet␍ facet normal 0.000000e+00 1.000000e+00 0.000000e+00␍ outer loop␍ vertex 2.000000e+01 2.000000e+01 2.000000e+01␍ vertex 0.000000e+00 2.000000e+01 0.000000e+00␍ vertex 0.000000e+00 2.000000e+01 2.000000e+01␍ endloop␍ endfacet␍endsolid vcg␍