From d228b6b81296a649de8527c65673da462b723909 Mon Sep 17 00:00:00 2001 From: AJ Isaacs Date: Mon, 20 Apr 2026 09:20:33 -0400 Subject: [PATCH] refactor(engine): share MBR between PartClassifier and CanonicalAngle --- OpenNest.Engine/PartClassifier.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/OpenNest.Engine/PartClassifier.cs b/OpenNest.Engine/PartClassifier.cs index f53f4ab..e0ceef7 100644 --- a/OpenNest.Engine/PartClassifier.cs +++ b/OpenNest.Engine/PartClassifier.cs @@ -64,8 +64,8 @@ namespace OpenNest.Engine var mbrArea = mbr.Area; var mbrPerimeter = 2 * (mbr.Width + mbr.Height); - // Store primary angle (negated to align MBR with axes, same as RotationAnalysis). - result.PrimaryAngle = -mbr.Angle; + // Share the single angle formula with CanonicalAngle (no duplicate MBR compute). + result.PrimaryAngle = CanonicalAngle.FromMbr(mbr); // Drawing perimeter for circularity and perimeter ratio. var drawingPerimeter = polygon.Perimeter();