fix(ModelColocation): Upgraded the verbose mode for the ModelComponentsMeshColocation

Author: MelchiorSchuh

src/bin/geode-inspector-brep.cpp

@@ -245,7 +245,7 @@ void inspect_brep( const geode::BRep& brep )
     {
         tasks.emplace_back( async::spawn( [&brep_inspector] {
             const auto nb =
-                brep_inspector.components_with_colocated_points().size();
+                brep_inspector.components_nb_colocated_points().size();
             geode::Logger::info(
                 nb, " components with colocated points in their mesh." );
         } ) );

src/geode/inspector/criterion/colocation/component_meshes_colocation.cpp

@@ -25,6 +25,7 @@
 
 #include <geode/basic/logger.h>
 #include <geode/basic/pimpl_impl.h>
+#include <geode/basic/uuid.h>
 
 #include <geode/mesh/core/edged_curve.h>
 #include <geode/mesh/core/solid_mesh.h>
@@ -50,21 +51,31 @@ namespace
         for( const auto& line : model.lines() )
         {
             const geode::EdgedCurveColocation< dimension > inspector{
-                line.mesh(), verbose
+                line.mesh(), false
             };
             if( inspector.mesh_has_colocated_points() )
             {
                 components_with_colocation.push_back( line.id() );
+                if( verbose )
+                {
+                    geode::Logger::info( "Line with uuid ", line.id().string(),
+                        " has colocated points." );
+                }
             }
         }
         for( const auto& surface : model.surfaces() )
         {
             const geode::SurfaceMeshColocation< dimension > inspector{
-                surface.mesh(), verbose
+                surface.mesh(), false
             };
             if( inspector.mesh_has_colocated_points() )
             {
                 components_with_colocation.push_back( surface.id() );
+                if( verbose )
+                {
+                    geode::Logger::info( "Surface with uuid ",
+                        surface.id().string(), " has colocated points." );
+                }
             }
         }
         return components_with_colocation;
@@ -85,11 +96,15 @@ namespace
                 model, verbose );
         for( const auto& block : model.blocks() )
         {
-            const geode::SolidMeshColocation3D inspector{ block.mesh(),
-                verbose };
+            const geode::SolidMeshColocation3D inspector{ block.mesh(), false };
             if( inspector.mesh_has_colocated_points() )
             {
                 components_with_colocation.push_back( block.id() );
+                if( verbose )
+                {
+                    geode::Logger::info( "Block with uuid ",
+                        block.id().string(), " has colocated points." );
+                }
             }
         }
         return components_with_colocation;
@@ -105,25 +120,36 @@ namespace
         for( const auto& line : model.lines() )
         {
             const geode::EdgedCurveColocation< dimension > inspector{
-                line.mesh(), verbose
+                line.mesh(), false
             };
             const auto nb_colocated = inspector.nb_colocated_points();
             if( nb_colocated > 0 )
             {
                 components_nb_colocated_points.emplace(
                     line.id(), nb_colocated );
+                if( verbose )
+                {
+                    geode::Logger::info( "Line with uuid ", line.id().string(),
+                        " has ", nb_colocated, " colocated points." );
+                }
             }
         }
         for( const auto& surface : model.surfaces() )
         {
             const geode::SurfaceMeshColocation< dimension > inspector{
-                surface.mesh(), verbose
+                surface.mesh(), false
             };
             const auto nb_colocated = inspector.nb_colocated_points();
             if( nb_colocated > 0 )
             {
                 components_nb_colocated_points.emplace(
                     surface.id(), nb_colocated );
+                if( verbose )
+                {
+                    geode::Logger::info( "Surface with uuid ",
+                        surface.id().string(), " has ", nb_colocated,
+                        " colocated points." );
+                }
             }
         }
         return components_nb_colocated_points;
@@ -146,13 +172,18 @@ namespace
                 model, verbose );
         for( const auto& block : model.blocks() )
         {
-            const geode::SolidMeshColocation3D inspector{ block.mesh(),
-                verbose };
+            const geode::SolidMeshColocation3D inspector{ block.mesh(), false };
             const auto nb_colocated = inspector.nb_colocated_points();
             if( nb_colocated > 0 )
             {
                 components_nb_colocated_points.emplace(
                     block.id(), nb_colocated );
+                if( verbose )
+                {
+                    geode::Logger::info( "Block with uuid ",
+                        block.id().string(), " has ", nb_colocated,
+                        " colocated points." );
+                }
             }
         }
         return components_nb_colocated_points;

src/geode/inspector/criterion/private/colocation_impl.cpp

@@ -74,9 +74,14 @@ namespace geode
         template < index_t dimension, typename Mesh >
         index_t ColocationImpl< dimension, Mesh >::nb_colocated_points() const
         {
-            return mesh_points_colocated_info< dimension, Mesh >(
+            auto nb_colocated = mesh_points_colocated_info< dimension, Mesh >(
                 mesh_, global_epsilon )
-                .nb_colocated_points();
+                                    .nb_colocated_points();
+            if( nb_colocated > 0 && verbose_ )
+            {
+                Logger::info( "Mesh has ", nb_colocated, " colocated points" );
+            }
+            return nb_colocated;
         }
 
         template < index_t dimension, typename Mesh >