Merge pull request #1319 from Geode-solutions/fix/geometrical_operations_on_mesh

fix(GeometricalOperationsOnMesh): update template parameter for geome…

Author: BotellaA

bindings/python/src/mesh/helpers/geometrical_operations_on_mesh.cpp

@@ -43,13 +43,13 @@
     const auto translate_name##mesh##dimension = absl::StrCat(                 \
         "translate_", mesh_name, std::to_string( dimension ), "D" );           \
     module.def( translate_name##mesh##dimension.c_str(),                       \
-        &translate_mesh< mesh, mesh##Builder, dimension > )
+        &translate_mesh< mesh##dimension##D > )
 
 #define PYTHON_RESCALE( mesh, mesh_name, dimension )                           \
     const auto rescale_name##mesh##dimension = absl::StrCat(                   \
         "rescale_", mesh_name, std::to_string( dimension ), "D" );             \
     module.def( rescale_name##mesh##dimension.c_str(),                         \
-        &rescale_mesh< mesh, mesh##Builder, dimension > )
+        &rescale_mesh< mesh##dimension##D > )
 
 namespace geode
 {

include/geode/mesh/helpers/geometrical_operations_on_mesh.hpp

@@ -29,30 +29,26 @@
 
 namespace geode
 {
-    template < template < index_t > class Mesh,
-        template < index_t > class MeshBuilder,
-        index_t dimension >
-    void translate_mesh( Mesh< dimension >& mesh,
-        MeshBuilder< dimension >& builder,
-        const Vector< dimension >& translation )
+    template < class Mesh >
+    void translate_mesh( Mesh& mesh,
+        typename Mesh::Builder& builder,
+        const Vector< Mesh::dim >& translation )
     {
         for( const auto v : Range{ mesh.nb_vertices() } )
         {
             builder.set_point( v, mesh.point( v ) + translation );
         }
     }
 
-    template < template < index_t > class Mesh,
-        template < index_t > class MeshBuilder,
-        index_t dimension >
-    void rescale_mesh( Mesh< dimension >& mesh,
-        MeshBuilder< dimension >& builder,
-        const std::array< double, dimension >& scale )
+    template < class Mesh >
+    void rescale_mesh( Mesh& mesh,
+        typename Mesh::Builder& builder,
+        const std::array< double, Mesh::dim >& scale )
     {
         for( const auto v : Range{ mesh.nb_vertices() } )
         {
             auto point = mesh.point( v );
-            for( const auto d : LRange{ dimension } )
+            for( const auto d : LRange{ Mesh::dim } )
             {
                 point.set_value( d, point.value( d ) * scale[d] );
             }