3D-printing/OrcaSlicer
1
0
Fork 0
mirror of https://github.com/SoftFever/OrcaSlicer.git synced 2025-10-31 04:31:15 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

Optimise sandwich wall mode when perimeters are shared with multiple islands(Arachne only) (#1352)

Browse source 
* Update PerimeterGenerator.cpp

* PerimeterGenerator updated with improved Sandwich infill handling

Cleaned up code

* Beautified syntax

* Optimise the logic

---------

Co-authored-by: SoftFever <softfeverever@gmail.com>
This commit is contained in:
igiannakas 2023-06-29 05:02:07 +01:00 • committed by GitHub
parent 8e4a43ea5c
commit 1c780496aa
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 43 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

51
src/libslic3r/PerimeterGenerator.cpp
View file

@ -1374,7 +1374,8 @@ void PerimeterGenerator::process_arachne()
bool is_outer_wall_first = bool is_outer_wall_first =
this->config->wall_infill_order == WallInfillOrder::OuterInnerInfill || this->config->wall_infill_order == WallInfillOrder::OuterInnerInfill ||
this->config->wall_infill_order == WallInfillOrder::InfillOuterInner; this->config->wall_infill_order == WallInfillOrder::InfillOuterInner ||
this->config->wall_infill_order == WallInfillOrder::InnerOuterInnerInfill;
if (is_outer_wall_first) { if (is_outer_wall_first) {
start_perimeter = 0; start_perimeter = 0;
end_perimeter = int(perimeters.size()); end_perimeter = int(perimeters.size());
@ -1502,14 +1503,46 @@ void PerimeterGenerator::process_arachne()
} }
if (this->config->wall_infill_order == WallInfillOrder::InnerOuterInnerInfill) if (this->config->wall_infill_order == WallInfillOrder::InnerOuterInnerInfill) {
if (ordered_extrusions.size() > 1) { if (ordered_extrusions.size() > 2) { // 3 walls minimum needed to do inner outer inner ordering
int last_outer = 0; int position = 0; // index to run the re-ordering for multiple external perimeters in a single island.
int outer = 0; int arr_i = 0; // index to run through the walls
for (; outer < ordered_extrusions.size(); ++outer) int outer, first_internal, second_internal; // allocate index values
if (ordered_extrusions[outer].extrusion->inset_idx == 0 && outer - last_outer > 1) { // run the re-ordering for all wall loops in the same island
std::swap(ordered_extrusions[outer], ordered_extrusions[outer - 1]); while (position < ordered_extrusions.size()) {
last_outer = outer; outer = first_internal = second_internal = -1; // initialise all index values to -1
// run through the walls to get the index values that need re-ordering until the first one for each
// is found. Start at "position" index to enable the for loop to iterate for multiple external
// perimeters in a single island
for (arr_i = position; arr_i < ordered_extrusions.size(); ++arr_i) {
switch (ordered_extrusions[arr_i].extrusion->inset_idx) {
case 0: // external perimeter
if (outer == -1)
outer = arr_i;
break;
case 1: // first internal wall
if (first_internal == -1 && arr_i > outer)
first_internal = arr_i;
break;
case 2: // second internal wall
if (ordered_extrusions[arr_i].extrusion->inset_idx == 2 && second_internal == -1 &&
arr_i > first_internal)
second_internal = arr_i;
break;
}
if (second_internal != -1)
break; // found all three perimeters to re-order
}
if (outer > -1 && first_internal > -1 && second_internal > -1) { // found perimeters to re-order?
const auto temp = ordered_extrusions[second_internal];
ordered_extrusions[second_internal] = ordered_extrusions[first_internal];
ordered_extrusions[first_internal] = ordered_extrusions[outer];
ordered_extrusions[outer] = temp;
} else
break; // did not find any more candidates to re-order, so stop the while loop early
// go to the next perimeter to continue scanning for external walls in the same island
position = arr_i + 1;
}
} }
} }