finalized border computation algorithm

uos · Aug 15, 2024 · 7030533 · 7030533
1 parent 4376909
commit 7030533
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 9 deletions.
diff --git a/include/lvr2/algorithm/GeometryAlgorithms.hpp b/include/lvr2/algorithm/GeometryAlgorithms.hpp
@@ -204,7 +204,7 @@ DenseEdgeMap<float> calcVertexDistances(const BaseMesh<BaseVecT>& mesh);
  * @brief Computes / set costs for each border vertex to predefined value
  *
  * @param mesh          The mesh containing the vertices and edges of interest
- * @param border_cost   Predfined value to set the border vertices to. Defaults to 1.0
+ * @param border_cost   Predefined value to set the border vertices to. Defaults to 1.0
  * @return              The dense edge map with the border cost values
  */
 template<typename BaseVecT>

diff --git a/include/lvr2/algorithm/GeometryAlgorithms.tcc b/include/lvr2/algorithm/GeometryAlgorithms.tcc
@@ -506,25 +506,24 @@ DenseVertexMap<float> calcBorderCosts(
     ++progress;
 
     // Calculate height difference for each vertex
-    // for(auto vH : mesh.vertices())
     #pragma omp parallel for
     for (size_t i = 0; i < mesh.nextVertexIndex(); i++)
     {
         auto vH = VertexHandle(i);
         if (!mesh.containsVertex(vH))
         {
-          continue;
+            continue;
         }
 
-        // new! give border vertex a high high
         bool is_border_vertex = false;
         for(auto edge : mesh.getEdgesOfVertex(vH))
         {
-          if(mesh.isBorderEdge(edge))
-          {
-            is_border_vertex = true;
-            break; // early finish
-          }
+            if(mesh.isBorderEdge(edge))
+            {
+                is_border_vertex = true;
+                // it's a border vertex. stop searching. early finish
+                break;
+            }
         }
 
         // Calculate the final border weight