aab8a12801
ConcentricGapFill pattern was used for internal narrow solid infill. Use arachne engine instead to remove gap fill inside the pattern and improve the extrusion path Signed-off-by: salt.wei <salt.wei@bambulab.com> Change-Id: I758d7c72eb71cc37026b7cebf746cc345014c3f5 (cherry picked from commit 0b6bacd21a091afc13d7b36a69e5b10f155bc6f8)
234 lines
12 KiB
C++
234 lines
12 KiB
C++
//Copyright (c) 2022 Ultimaker B.V.
|
|
//CuraEngine is released under the terms of the AGPLv3 or higher.
|
|
|
|
#ifndef UTILS_POLYLINE_STITCHER_H
|
|
#define UTILS_POLYLINE_STITCHER_H
|
|
|
|
#include "SparsePointGrid.hpp"
|
|
#include "PolygonsPointIndex.hpp"
|
|
#include "../../Polygon.hpp"
|
|
#include <unordered_set>
|
|
#include <cassert>
|
|
|
|
namespace Slic3r::Arachne
|
|
{
|
|
|
|
/*!
|
|
* Class for stitching polylines into longer polylines or into polygons
|
|
*/
|
|
template<typename Paths, typename Path, typename Junction>
|
|
class PolylineStitcher
|
|
{
|
|
public:
|
|
/*!
|
|
* Stitch together the separate \p lines into \p result_lines and if they
|
|
* can be closed into \p result_polygons.
|
|
*
|
|
* Only introduce new segments shorter than \p max_stitch_distance, and
|
|
* larger than \p snap_distance but always try to take the shortest
|
|
* connection possible.
|
|
*
|
|
* Only stitch polylines into closed polygons if they are larger than 3 *
|
|
* \p max_stitch_distance, in order to prevent small segments to
|
|
* accidentally get closed into a polygon.
|
|
*
|
|
* \warning Tiny polylines (smaller than 3 * max_stitch_distance) will not
|
|
* be closed into polygons.
|
|
*
|
|
* \note Resulting polylines and polygons are added onto the existing
|
|
* containers, so you can directly output onto a polygons container with
|
|
* existing polygons in it. However, you shouldn't call this function with
|
|
* the same parameter in \p lines as \p result_lines, because that would
|
|
* duplicate (some of) the polylines.
|
|
* \param lines The lines to stitch together.
|
|
* \param result_lines[out] The stitched parts that are not closed polygons
|
|
* will be stored in here.
|
|
* \param result_polygons[out] The stitched parts that were closed as
|
|
* polygons will be stored in here.
|
|
* \param max_stitch_distance The maximum distance that will be bridged to
|
|
* connect two lines.
|
|
* \param snap_distance Points closer than this distance are considered to
|
|
* be the same point.
|
|
*/
|
|
static void stitch(const Paths& lines, Paths& result_lines, Paths& result_polygons, coord_t max_stitch_distance = scaled<coord_t>(0.1), coord_t snap_distance = scaled<coord_t>(0.01))
|
|
{
|
|
if (lines.empty())
|
|
return;
|
|
|
|
SparsePointGrid<PathsPointIndex<Paths>, PathsPointIndexLocator<Paths>> grid(max_stitch_distance, lines.size() * 2);
|
|
|
|
// populate grid
|
|
for (size_t line_idx = 0; line_idx < lines.size(); line_idx++)
|
|
{
|
|
const auto line = lines[line_idx];
|
|
grid.insert(PathsPointIndex<Paths>(&lines, line_idx, 0));
|
|
grid.insert(PathsPointIndex<Paths>(&lines, line_idx, line.size() - 1));
|
|
}
|
|
|
|
std::vector<bool> processed(lines.size(), false);
|
|
|
|
for (size_t line_idx = 0; line_idx < lines.size(); line_idx++)
|
|
{
|
|
if (processed[line_idx])
|
|
{
|
|
continue;
|
|
}
|
|
processed[line_idx] = true;
|
|
const auto line = lines[line_idx];
|
|
bool should_close = isOdd(line);
|
|
|
|
Path chain = line;
|
|
bool closest_is_closing_polygon = false;
|
|
for (bool go_in_reverse_direction : { false, true }) // first go in the unreversed direction, to try to prevent the chain.reverse() operation.
|
|
{ // NOTE: Implementation only works for this order; we currently only re-reverse the chain when it's closed.
|
|
if (go_in_reverse_direction)
|
|
{ // try extending chain in the other direction
|
|
chain.reverse();
|
|
}
|
|
int64_t chain_length = chain.polylineLength();
|
|
|
|
while (true)
|
|
{
|
|
Point from = make_point(chain.back());
|
|
|
|
PathsPointIndex<Paths> closest;
|
|
coord_t closest_distance = std::numeric_limits<coord_t>::max();
|
|
grid.processNearby(from, max_stitch_distance,
|
|
std::function<bool (const PathsPointIndex<Paths>&)> (
|
|
[from, &chain, &closest, &closest_is_closing_polygon, &closest_distance, &processed, &chain_length, go_in_reverse_direction, max_stitch_distance, snap_distance, should_close]
|
|
(const PathsPointIndex<Paths>& nearby)->bool
|
|
{
|
|
bool is_closing_segment = false;
|
|
coord_t dist = (nearby.p().template cast<int64_t>() - from.template cast<int64_t>()).norm();
|
|
if (dist > max_stitch_distance)
|
|
{
|
|
return true; // keep looking
|
|
}
|
|
if ((nearby.p().template cast<int64_t>() - make_point(chain.front()).template cast<int64_t>()).squaredNorm() < snap_distance * snap_distance)
|
|
{
|
|
if (chain_length + dist < 3 * max_stitch_distance // prevent closing of small poly, cause it might be able to continue making a larger polyline
|
|
|| chain.size() <= 2) // don't make 2 vert polygons
|
|
{
|
|
return true; // look for a better next line
|
|
}
|
|
is_closing_segment = true;
|
|
if (!should_close)
|
|
{
|
|
dist += scaled<coord_t>(0.01); // prefer continuing polyline over closing a polygon; avoids closed zigzags from being printed separately
|
|
// continue to see if closing segment is also the closest
|
|
// there might be a segment smaller than [max_stitch_distance] which closes the polygon better
|
|
}
|
|
else
|
|
{
|
|
dist -= scaled<coord_t>(0.01); //Prefer closing the polygon if it's 100% even lines. Used to create closed contours.
|
|
//Continue to see if closing segment is also the closest.
|
|
}
|
|
}
|
|
else if (processed[nearby.poly_idx])
|
|
{ // it was already moved to output
|
|
return true; // keep looking for a connection
|
|
}
|
|
bool nearby_would_be_reversed = nearby.point_idx != 0;
|
|
nearby_would_be_reversed = nearby_would_be_reversed != go_in_reverse_direction; // flip nearby_would_be_reversed when searching in the reverse direction
|
|
if (!canReverse(nearby) && nearby_would_be_reversed)
|
|
{ // connecting the segment would reverse the polygon direction
|
|
return true; // keep looking for a connection
|
|
}
|
|
if (!canConnect(chain, (*nearby.polygons)[nearby.poly_idx]))
|
|
{
|
|
return true; // keep looking for a connection
|
|
}
|
|
if (dist < closest_distance)
|
|
{
|
|
closest_distance = dist;
|
|
closest = nearby;
|
|
closest_is_closing_polygon = is_closing_segment;
|
|
}
|
|
if (dist < snap_distance)
|
|
{ // we have found a good enough next line
|
|
return false; // stop looking for alternatives
|
|
}
|
|
return true; // keep processing elements
|
|
})
|
|
);
|
|
|
|
if (!closest.initialized() // we couldn't find any next line
|
|
|| closest_is_closing_polygon // we closed the polygon
|
|
)
|
|
{
|
|
break;
|
|
}
|
|
|
|
coord_t segment_dist = (make_point(chain.back()).template cast<int64_t>() - closest.p().template cast<int64_t>()).norm();
|
|
assert(segment_dist <= max_stitch_distance + scaled<coord_t>(0.01));
|
|
const size_t old_size = chain.size();
|
|
if (closest.point_idx == 0)
|
|
{
|
|
auto start_pos = (*closest.polygons)[closest.poly_idx].begin();
|
|
if (segment_dist < snap_distance)
|
|
{
|
|
++start_pos;
|
|
}
|
|
chain.insert(chain.end(), start_pos, (*closest.polygons)[closest.poly_idx].end());
|
|
}
|
|
else
|
|
{
|
|
auto start_pos = (*closest.polygons)[closest.poly_idx].rbegin();
|
|
if (segment_dist < snap_distance)
|
|
{
|
|
++start_pos;
|
|
}
|
|
chain.insert(chain.end(), start_pos, (*closest.polygons)[closest.poly_idx].rend());
|
|
}
|
|
for(size_t i = old_size; i < chain.size(); ++i) //Update chain length.
|
|
{
|
|
chain_length += (make_point(chain[i]).template cast<int64_t>() - make_point(chain[i - 1]).template cast<int64_t>()).norm();
|
|
}
|
|
should_close = should_close & !isOdd((*closest.polygons)[closest.poly_idx]); //If we connect an even to an odd line, we should no longer try to close it.
|
|
assert( ! processed[closest.poly_idx]);
|
|
processed[closest.poly_idx] = true;
|
|
}
|
|
if (closest_is_closing_polygon)
|
|
{
|
|
if (go_in_reverse_direction)
|
|
{ // re-reverse chain to retain original direction
|
|
// NOTE: not sure if this code could ever be reached, since if a polygon can be closed that should be already possible in the forward direction
|
|
chain.reverse();
|
|
}
|
|
|
|
break; // don't consider reverse direction
|
|
}
|
|
}
|
|
if (closest_is_closing_polygon)
|
|
{
|
|
result_polygons.emplace_back(chain);
|
|
}
|
|
else
|
|
{
|
|
PathsPointIndex<Paths> ppi_here(&lines, line_idx, 0);
|
|
if ( ! canReverse(ppi_here))
|
|
{ // Since closest_is_closing_polygon is false we went through the second iterations of the for-loop, where go_in_reverse_direction is true
|
|
// the polyline isn't allowed to be reversed, so we re-reverse it.
|
|
chain.reverse();
|
|
}
|
|
result_lines.emplace_back(chain);
|
|
}
|
|
}
|
|
}
|
|
|
|
/*!
|
|
* Whether a polyline is allowed to be reversed. (Not true for wall polylines which are not odd)
|
|
*/
|
|
static bool canReverse(const PathsPointIndex<Paths> &polyline);
|
|
|
|
/*!
|
|
* Whether two paths are allowed to be connected.
|
|
* (Not true for an odd and an even wall.)
|
|
*/
|
|
static bool canConnect(const Path &a, const Path &b);
|
|
|
|
static bool isOdd(const Path &line);
|
|
};
|
|
|
|
} // namespace Slic3r::Arachne
|
|
#endif // UTILS_POLYLINE_STITCHER_H
|