aj/OpenNest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

perf: halve rotating calipers edge iterations

Browse Source 
Opposite hull edges (180° apart) produce identical bounding rectangles
(width/height swapped), so only half the edges need to be checked to
find the minimum area. Applied to the unconstrained overload only.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
AJ Isaacs
2026-03-09 22:17:24 -04:00
parent 91908c1732
commit 5fc16e9dae
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
OpenNest.Core/Geometry/RotatingCalipers.cs
View File

@@ -48,9 +48,14 @@ namespace OpenNest.Geometry
return new BoundingRectangleResult(angle, length, 0); return new BoundingRectangleResult(angle, length, 0);
} }
// Only need half the edges — opposite edges produce the same
// bounding rectangle (width/height swapped), so the minimum
// area repeats every 180°.
var edgeCount = (n + 1) / 2;
BoundingRectangleResult best = null; BoundingRectangleResult best = null;
for (int i = 0; i < n; i++) for (int i = 0; i < edgeCount; i++)
{ {
int next = (i + 1) % n; int next = (i + 1) % n;
var edgeX = vertices[next].X - vertices[i].X; var edgeX = vertices[next].X - vertices[i].X;