From 38dcaf16d33250b4275139a964e56bc883f5d495 Mon Sep 17 00:00:00 2001 From: AJ Isaacs Date: Fri, 20 Mar 2026 21:12:16 -0400 Subject: [PATCH] revert: switch BestFitFinder back to RotationSlideStrategy MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit NFP strategy has coordinate correction issues causing overlaps. The slide-based approach is fast and accurate — keeping it as default. NfpSlideStrategy and PolygonHelper remain in the codebase for future use. Co-Authored-By: Claude Opus 4.6 (1M context) --- OpenNest.Engine/BestFit/BestFitFinder.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/OpenNest.Engine/BestFit/BestFitFinder.cs b/OpenNest.Engine/BestFit/BestFitFinder.cs index bc0c4b4..95861b5 100644 --- a/OpenNest.Engine/BestFit/BestFitFinder.cs +++ b/OpenNest.Engine/BestFit/BestFitFinder.cs @@ -83,8 +83,8 @@ namespace OpenNest.Engine.BestFit foreach (var angle in angles) { - var desc = $"{Angle.ToDegrees(angle):F1} deg NFP"; - strategies.Add(new NfpSlideStrategy(angle, type++, desc)); + var desc = string.Format("{0:F1} deg rotated, offset slide", Angle.ToDegrees(angle)); + strategies.Add(new RotationSlideStrategy(angle, type++, desc, _slideComputer)); } return strategies;