123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405 |
- #include <catch2/catch.hpp>
- #include "libslic3r/GCodeReader.hpp"
- #include "test_data.hpp" // get access to init_print, etc
- using namespace Slic3r::Test;
- using namespace Slic3r;
- SCENARIO("Shells", "[Shells]") {
- GIVEN("20mm box") {
- auto test = [](const DynamicPrintConfig &config){
- std::vector<coord_t> zs;
- std::set<coord_t> layers_with_solid_infill;
- std::set<coord_t> layers_with_bridge_infill;
- const double solid_infill_speed = config.opt_float("solid_infill_speed") * 60;
- const double bridge_speed = config.opt_float("bridge_speed") * 60;
- GCodeReader parser;
- parser.parse_buffer(Slic3r::Test::slice({ Slic3r::Test::TestMesh::cube_20x20x20 }, config),
- [&zs, &layers_with_solid_infill, &layers_with_bridge_infill, solid_infill_speed, bridge_speed]
- (Slic3r::GCodeReader &self, const Slic3r::GCodeReader::GCodeLine &line)
- {
- double z = line.new_Z(self);
- REQUIRE(z >= 0);
- if (z > 0) {
- coord_t scaled_z = scaled<float>(z);
- zs.emplace_back(scaled_z);
- if (line.extruding(self) && line.dist_XY(self) > 0) {
- double f = line.new_F(self);
- if (std::abs(f - solid_infill_speed) < EPSILON)
- layers_with_solid_infill.insert(scaled_z);
- if (std::abs(f - bridge_speed) < EPSILON)
- layers_with_bridge_infill.insert(scaled_z);
- }
- }
- });
- sort_remove_duplicates(zs);
- auto has_solid_infill = [&layers_with_solid_infill](coord_t z) { return layers_with_solid_infill.find(z) != layers_with_solid_infill.end(); };
- auto has_bridge_infill = [&layers_with_bridge_infill](coord_t z) { return layers_with_bridge_infill.find(z) != layers_with_bridge_infill.end(); };
- auto has_shells = [&has_solid_infill, &has_bridge_infill, &zs](int layer_idx) { coord_t z = zs[layer_idx]; return has_solid_infill(z) || has_bridge_infill(z); };
- const int bottom_solid_layers = config.opt_int("bottom_solid_layers");
- const int top_solid_layers = config.opt_int("top_solid_layers");
- THEN("correct number of bottom solid layers") {
- for (int i = 0; i < bottom_solid_layers; ++ i)
- REQUIRE(has_shells(i));
- for (int i = bottom_solid_layers; i < int(zs.size() / 2); ++ i)
- REQUIRE(! has_shells(i));
- }
- THEN("correct number of top solid layers") {
- // NOTE: there is one additional layer with enusring line under the bridge layer, bridges would be otherwise anchored weakly to the perimeter.
- size_t additional_ensuring_anchors = top_solid_layers > 0 ? 1 : 0;
- for (int i = 0; i < top_solid_layers + additional_ensuring_anchors; ++ i)
- REQUIRE(has_shells(int(zs.size()) - i - 1));
- for (int i = top_solid_layers + additional_ensuring_anchors; i < int(zs.size() / 2); ++ i)
- REQUIRE(! has_shells(int(zs.size()) - i - 1));
- }
- if (top_solid_layers > 0) {
- THEN("solid infill speed is used on solid infill") {
- for (int i = 0; i < top_solid_layers - 1; ++ i) {
- auto z = zs[int(zs.size()) - i - 1];
- REQUIRE(has_solid_infill(z));
- REQUIRE(! has_bridge_infill(z));
- }
- }
- THEN("bridge used in first solid layer over sparse infill") {
- auto z = zs[int(zs.size()) - top_solid_layers];
- REQUIRE(! has_solid_infill(z));
- REQUIRE(has_bridge_infill(z));
- }
- }
- };
- auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- { "skirts", 0 },
- { "perimeters", 0 },
- { "solid_infill_speed", 99 },
- { "top_solid_infill_speed", 99 },
- { "bridge_speed", 72 },
- { "first_layer_speed", "100%" },
- { "cooling", "0" }
- });
- WHEN("three top and bottom layers") {
- // proper number of shells is applied
- config.set_deserialize_strict({
- { "top_solid_layers", 3 },
- { "bottom_solid_layers", 3 }
- });
- test(config);
- }
- WHEN("zero top and bottom layers") {
- // no shells are applied when both top and bottom are set to zero
- config.set_deserialize_strict({
- { "top_solid_layers", 0 },
- { "bottom_solid_layers", 0 }
- });
- test(config);
- }
- WHEN("three top and bottom layers, zero infill") {
- // proper number of shells is applied even when fill density is none
- config.set_deserialize_strict({
- { "perimeters", 1 },
- { "top_solid_layers", 3 },
- { "bottom_solid_layers", 3 }
- });
- test(config);
- }
- }
- }
- static std::set<double> layers_with_speed(const std::string &gcode, int speed)
- {
- std::set<double> out;
- GCodeReader parser;
- parser.parse_buffer(gcode, [&out, speed](Slic3r::GCodeReader &self, const Slic3r::GCodeReader::GCodeLine &line) {
- if (line.extruding(self) && is_approx<double>(line.new_F(self), speed * 60.))
- out.insert(self.z());
- });
- return out;
- }
- SCENARIO("Shells (from Perl)", "[Shells]") {
- GIVEN("V shape, Slic3r GH #1161") {
- int solid_speed = 99;
- auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- { "layer_height", 0.3 },
- { "first_layer_height", 0.3 },
- { "bottom_solid_layers", 0 },
- { "top_solid_layers", 3 },
- // to prevent speeds from being altered
- { "cooling", "0" },
- { "bridge_speed", solid_speed },
- { "solid_infill_speed", solid_speed },
- { "top_solid_infill_speed", solid_speed },
- // to prevent speeds from being altered
- { "first_layer_speed", "100%" },
- // prevent speed alteration
- { "enable_dynamic_overhang_speeds", 0 }
- });
-
- THEN("correct number of top solid shells is generated in V-shaped object") {
- size_t n = 0;
- for (auto z : layers_with_speed(Slic3r::Test::slice({TestMesh::V}, config), solid_speed))
- if (z <= 7.2)
- ++ n;
- REQUIRE(n == 3 + 1/*one additional layer with ensuring for bridge anchors*/);
- }
- }
- //TODO CHECK AFTER REMOVAL OF "ensure_vertical_wall_thickness"
- // GIVEN("V shape") {
- // // we need to check against one perimeter because this test is calibrated
- // // (shape, extrusion_width) so that perimeters cover the bottom surfaces of
- // // their lower layer - the test checks that shells are not generated on the
- // // above layers (thus 'across' the shadow perimeter)
- // // the test is actually calibrated to leave a narrow bottom region for each
- // // layer - we test that in case of fill_density = 0 such narrow shells are
- // // discarded instead of grown
- // int bottom_solid_layers = 3;
- // auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- // { "perimeters", 1 },
- // { "fill_density", 0 },
- // // to prevent speeds from being altered
- // { "cooling", "0" },
- // // to prevent speeds from being altered
- // { "first_layer_speed", "100%" },
- // // prevent speed alteration
- // { "enable_dynamic_overhang_speeds", 0 },
- // { "layer_height", 0.4 },
- // { "first_layer_height", 0.4 },
- // { "extrusion_width", 0.55 },
- // { "bottom_solid_layers", bottom_solid_layers },
- // { "top_solid_layers", 0 },
- // { "solid_infill_speed", 99 }
- // });
- // THEN("shells are not propagated across perimeters of the neighbor layer") {
- // std::string gcode = Slic3r::Test::slice({TestMesh::V}, config);
- // REQUIRE(layers_with_speed(gcode, 99).size() == bottom_solid_layers);
- // }
- // }
- // GIVEN("sloping_hole") {
- // int bottom_solid_layers = 3;
- // int top_solid_layers = 3;
- // int solid_speed = 99;
- // auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- // { "perimeters", 3 },
- // // to prevent speeds from being altered
- // { "cooling", "0" },
- // // to prevent speeds from being altered
- // { "first_layer_speed", "100%" },
- // // prevent speed alteration
- // { "enable_dynamic_overhang_speeds", 0 },
- // { "layer_height", 0.4 },
- // { "first_layer_height", 0.4 },
- // { "bottom_solid_layers", bottom_solid_layers },
- // { "top_solid_layers", top_solid_layers },
- // { "solid_infill_speed", solid_speed },
- // { "top_solid_infill_speed", solid_speed },
- // { "bridge_speed", solid_speed },
- // { "filament_diameter", 3. },
- // { "nozzle_diameter", 0.5 }
- // });
- // THEN("no superfluous shells are generated") {
- // std::string gcode = Slic3r::Test::slice({TestMesh::sloping_hole}, config);
- // REQUIRE(layers_with_speed(gcode, solid_speed).size() == bottom_solid_layers + top_solid_layers);
- // }
- // }
- GIVEN("20mm_cube, spiral vase") {
- double layer_height = 0.3;
- auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- { "perimeters", 1 },
- { "fill_density", 0 },
- { "layer_height", layer_height },
- { "first_layer_height", layer_height },
- { "top_solid_layers", 0 },
- { "spiral_vase", 1 },
- { "bottom_solid_layers", 0 },
- { "skirts", 0 },
- { "start_gcode", "" },
- { "temperature", 200 },
- { "first_layer_temperature", 205}
- });
-
- // TODO: this needs to be tested with a model with sloping edges, where starting
- // points of each layer are not aligned - in that case we would test that no
- // travel moves are left to move to the new starting point - in a cube, end
- // points coincide with next layer starting points (provided there's no clipping)
- auto test = [layer_height](const DynamicPrintConfig &config) {
- size_t travel_moves_after_first_extrusion = 0;
- bool started_extruding = false;
- bool first_layer_temperature_set = false;
- bool temperature_set = false;
- std::vector<double> z_steps;
- GCodeReader parser;
- parser.parse_buffer(Slic3r::Test::slice({TestMesh::cube_20x20x20}, config),
- [&](Slic3r::GCodeReader &self, const Slic3r::GCodeReader::GCodeLine &line) {
- if (line.cmd_is("G1")) {
- if (line.extruding(self))
- started_extruding = true;
- if (started_extruding) {
- if (double dz = line.dist_Z(self); dz > 0)
- z_steps.emplace_back(dz);
- if (line.travel() && line.dist_XY(self) > 0 && ! line.has(Z))
- ++ travel_moves_after_first_extrusion;
- }
- } else if (line.cmd_is("M104")) {
- int s;
- if (line.has_value('S', s)) {
- if (s == 205)
- first_layer_temperature_set = true;
- else if (s == 200)
- temperature_set = true;
- }
- }
- });
- THEN("first layer temperature is set") {
- REQUIRE(first_layer_temperature_set);
- }
- THEN("temperature is set") {
- REQUIRE(temperature_set);
- }
- // we allow one travel move after first extrusion: i.e. when moving to the first
- // spiral point after moving to second layer (bottom layer had loop clipping, so
- // we're slightly distant from the starting point of the loop)
- THEN("no gaps in spiral vase") {
- REQUIRE(travel_moves_after_first_extrusion <= 1);
- }
- THEN("no gaps in Z") {
- REQUIRE(std::count_if(z_steps.begin(), z_steps.end(),
- [&layer_height](auto z_step) { return z_step > layer_height + EPSILON; }) == 0);
- }
- };
- WHEN("solid model") {
- test(config);
- }
- WHEN("solid model with negative z-offset") {
- config.set_deserialize_strict("z_offset", "-10");
- test(config);
- }
- // Disabled because the current unreliable medial axis code doesn't always produce valid loops.
- // $test->('40x10', 'hollow model with negative z-offset');
- }
- GIVEN("20mm_cube, spiral vase") {
- double layer_height = 0.4;
- auto config = Slic3r::DynamicPrintConfig::full_print_config_with({
- { "spiral_vase", 1 },
- { "perimeters", 1 },
- { "fill_density", 0 },
- { "top_solid_layers", 0 },
- { "bottom_solid_layers", 0 },
- { "retract_layer_change", 0 },
- { "skirts", 0 },
- { "layer_height", layer_height },
- { "first_layer_height", layer_height },
- { "start_gcode", "" },
- // { "use_relative_e_distances", 1}
- });
- config.validate();
-
- std::vector<std::pair<double, double>> this_layer; // [ dist_Z, dist_XY ], ...
- int z_moves = 0;
- bool bottom_layer_not_flat = false;
- bool null_z_moves_not_layer_changes = false;
- bool null_z_moves_not_multiples_of_layer_height = false;
- bool sum_of_partial_z_equals_to_layer_height = false;
- bool all_layer_segments_have_same_slope = false;
- bool horizontal_extrusions = false;
- GCodeReader parser;
- parser.parse_buffer(Slic3r::Test::slice({TestMesh::cube_20x20x20}, config),
- [&](Slic3r::GCodeReader &self, const Slic3r::GCodeReader::GCodeLine &line) {
- if (line.cmd_is("G1")) {
- if (z_moves < 2) {
- // skip everything up to the second Z move
- // (i.e. start of second layer)
- if (line.has(Z)) {
- ++ z_moves;
- if (double dz = line.dist_Z(self); dz > 0 && ! is_approx<double>(dz, layer_height))
- bottom_layer_not_flat = true;
- }
- } else if (line.dist_Z(self) == 0 && line.has(Z)) {
- if (line.dist_XY(self) != 0)
- null_z_moves_not_layer_changes = true;
- double z = line.new_Z(self);
- if (fmod(z + EPSILON, layer_height) > 2 * EPSILON)
- null_z_moves_not_multiples_of_layer_height = true;
- double total_dist_XY = 0;
- double total_dist_Z = 0;
- for (auto &seg : this_layer) {
- total_dist_Z += seg.first;
- total_dist_XY += seg.second;
- }
- if (std::abs(total_dist_Z - layer_height) >
- // The first segment on the 2nd layer has extrusion interpolated from zero
- // and the 1st segment has such a low extrusion assigned, that it is effectively zero, thus the move
- // is considered non-extruding and a higher epsilon is required.
- (z_moves == 2 ? 0.0021 : EPSILON))
- sum_of_partial_z_equals_to_layer_height = true;
- //printf("Total height: %f, layer height: %f, good: %d\n", sum(map $_->[0], @this_layer), $config->layer_height, $sum_of_partial_z_equals_to_layer_height);
- for (auto &seg : this_layer)
- // check that segment's dist_Z is proportioned to its dist_XY
- if (std::abs(seg.first * total_dist_XY / layer_height - seg.second) > 0.2)
- all_layer_segments_have_same_slope = true;
- this_layer.clear();
- } else if (line.extruding(self) && line.dist_XY(self) > 0) {
- if (line.dist_Z(self) == 0)
- horizontal_extrusions = true;
- //printf("Pushing dist_z: %f, dist_xy: %f\n", $info->{dist_Z}, $info->{dist_XY});
- this_layer.emplace_back(line.dist_Z(self), line.dist_XY(self));
- }
- }
- });
- THEN("bottom layer is flat when using spiral vase") {
- REQUIRE(! bottom_layer_not_flat);
- }
- THEN("null Z moves are layer changes") {
- REQUIRE(! null_z_moves_not_layer_changes);
- }
- THEN("null Z moves are multiples of layer height") {
- REQUIRE(! null_z_moves_not_multiples_of_layer_height);
- }
- THEN("sum of partial Z increments equals to a full layer height") {
- REQUIRE(! sum_of_partial_z_equals_to_layer_height);
- }
- THEN("all layer segments have the same slope") {
- REQUIRE(! all_layer_segments_have_same_slope);
- }
- THEN("no horizontal extrusions") {
- REQUIRE(! horizontal_extrusions);
- }
- }
- }
- #if 0
- // The current Spiral Vase slicing code removes the holes and all but the largest contours from each slice,
- // therefore the following test is no more valid.
- {
- my $config = Slic3r::Config::new_from_defaults;
- $config->set('perimeters', 1);
- $config->set('fill_density', 0);
- $config->set('top_solid_layers', 0);
- $config->set('spiral_vase', 1);
- $config->set('bottom_solid_layers', 0);
- $config->set('skirts', 0);
- $config->set('first_layer_height', $config->layer_height);
- $config->set('start_gcode', '');
-
- my $print = Slic3r::Test::init_print('two_hollow_squares', config => $config);
- my $diagonal_moves = 0;
- Slic3r::GCode::Reader->new->parse(Slic3r::Test::gcode($print), sub {
- my ($self, $cmd, $args, $info) = @_;
-
- if ($cmd eq 'G1') {
- if ($info->{extruding} && $info->{dist_XY} > 0) {
- if ($info->{dist_Z} > 0) {
- $diagonal_moves++;
- }
- }
- }
- });
- is $diagonal_moves, 0, 'no spiral moves on two-island object';
- }
- #endif
|