Skip to content

Commit 6b85237

Browse files
committed
added temporary fix to facet manifold test
1 parent 8864b85 commit 6b85237

File tree

1 file changed

+6
-11
lines changed

1 file changed

+6
-11
lines changed

src/geode/inspector/criterion/manifold/brep_meshes_manifold.cpp

Lines changed: 6 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -258,8 +258,8 @@ namespace geode
258258
absl::StrAppend( &message, polygon_vertex, " " );
259259
}
260260
absl::StrAppend( &message,
261-
" is not manifold: it belongs to an internal "
262-
"surface with only one facet" );
261+
" is not manifold: it belongs to internal surface ",
262+
surface.id().string(), " with only one facet" );
263263
issues.add_issue(
264264
BRepNonManifoldFacet{ facet_vertices, { surface.id() } },
265265
message );
@@ -270,18 +270,13 @@ namespace geode
270270
bool several_cmvs_on_one_vertex(
271271
absl::Span< const index_t > unique_vertices ) const
272272
{
273+
std::vector< uuid > surfaces;
273274
for( const auto unique_vertex : unique_vertices )
274275
{
275-
std::vector< uuid > components;
276-
for( const auto& cmv :
277-
model().component_mesh_vertices( unique_vertex ) )
276+
if( model().component_mesh_vertices( unique_vertex ).size()
277+
> 1 )
278278
{
279-
if( absl::c_find( components, cmv.component_id.id() )
280-
!= components.end() )
281-
{
282-
return true;
283-
}
284-
components.push_back( cmv.component_id.id() );
279+
return true;
285280
}
286281
}
287282
return false;

0 commit comments

Comments
 (0)