fix(v1.0.1): restore contour loop closing point for better LS fitting
CI / Build & Test (push) Failing after 32s
CI / Release Build (push) Failing after 31s
Build & Test / build-and-test (push) Has been cancelled
Build & Test / python-bindings (push) Has been cancelled

The contour extraction previously relied on LS fitting to handle
loop closure internally, but removing the closing point caused
fitting quality issues. Restore comp.push_back(comp[0]) to ensure
closed curves are properly fitted.

Related: VDE-021 mesh contour extraction
This commit is contained in:
茂之钳
2026-07-28 18:21:01 +08:00
parent 64be0f1cda
commit d949da2719
+2 -1
View File
@@ -453,7 +453,8 @@ std::vector<curves::NurbsCurve> mesh_contour_to_curves(
cur = next;
}
if (comp.size() >= 3) {
// Don't add closing duplicate — LS fitting handles closure internally
// Close the loop for fitting
comp.push_back(comp[0]);
components.push_back(std::move(comp));
}
}