Remove self-closing short loops in removeLoops

onthegomap · Nov 7, 2024 · 091bdb8 · 091bdb8
1 parent 801e1de
commit 091bdb8
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/planetiler-core/src/main/java/com/onthegomap/planetiler/util/LoopLineMerger.java b/planetiler-core/src/main/java/com/onthegomap/planetiler/util/LoopLineMerger.java
@@ -98,7 +98,13 @@ private void removeLoops() {
     List<Edge> removedEdges = new ArrayList<>();
     for (var node : output) {
       for (var edge : List.copyOf(node.getEdges())) {
-        if (removedEdges.contains(edge) || edge.from == edge.to) {
+        if (removedEdges.contains(edge)) {
+          continue;
+        }
+        if (edge.from == edge.to && edge.length <= loopMinLength) {
+          // self-closing short loop
+          removedEdges.add(edge);
+          edge.remove();
           continue;
         }
         try {