From e3e51611d5fbb6a10e59a2270cf3e164fc2f271d Mon Sep 17 00:00:00 2001 From: AJ Isaacs Date: Sat, 28 Mar 2026 14:47:26 -0400 Subject: [PATCH] fix: only remove bend-generated etch entities, preserve user etch lines UpdateEtchEntities was removing all entities on the ETCH layer, which also deleted user-added etch marks like part numbers. Now tags generated bend etch lines with a BendEtch tag and filters on that instead. Co-Authored-By: Claude Opus 4.6 (1M context) --- OpenNest.Core/Bending/Bend.cs | 5 +++-- OpenNest.Core/Geometry/Entity.cs | 5 +++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/OpenNest.Core/Bending/Bend.cs b/OpenNest.Core/Bending/Bend.cs index a1ef2d3..3f8e694 100644 --- a/OpenNest.Core/Bending/Bend.cs +++ b/OpenNest.Core/Bending/Bend.cs @@ -14,6 +14,7 @@ namespace OpenNest.Bending }; private const double DefaultEtchLength = 1.0; + private const string BendEtchTag = "BendEtch"; public Vector StartPoint { get; set; } public Vector EndPoint { get; set; } @@ -73,7 +74,7 @@ namespace OpenNest.Bending /// public static void UpdateEtchEntities(List entities, List bends) { - entities.RemoveAll(e => e.Layer == EtchLayer); + entities.RemoveAll(e => e.Tag == BendEtchTag); if (bends == null) return; foreach (var bend in bends) @@ -82,7 +83,7 @@ namespace OpenNest.Bending private static Line CreateEtchLine(Vector start, Vector end) { - return new Line(start, end) { Layer = EtchLayer, Color = Color.Green }; + return new Line(start, end) { Layer = EtchLayer, Color = Color.Green, Tag = BendEtchTag }; } public override string ToString() diff --git a/OpenNest.Core/Geometry/Entity.cs b/OpenNest.Core/Geometry/Entity.cs index ef80316..44aa743 100644 --- a/OpenNest.Core/Geometry/Entity.cs +++ b/OpenNest.Core/Geometry/Entity.cs @@ -29,6 +29,11 @@ namespace OpenNest.Geometry /// public bool IsVisible { get; set; } = true; + /// + /// Optional tag for identifying generated entities (e.g. bend etch marks). + /// + public string Tag { get; set; } + /// /// Smallest box that contains the entity. ///