Skip to content

Introduce geometrically NM detection functions#6968

Open
MaelRL wants to merge 7 commits intoCGAL:mainfrom
MaelRL:PMP-Extract_valuables_from_NM_WIP-GF
Open

Introduce geometrically NM detection functions#6968
MaelRL wants to merge 7 commits intoCGAL:mainfrom
MaelRL:PMP-Extract_valuables_from_NM_WIP-GF

Conversation

@MaelRL
Copy link
Member

@MaelRL MaelRL commented Oct 14, 2022

Summary of Changes

Introduce two new PMP functions to collect geometrically non-manifold elements. Also speed up non_manifold_vertices() for the average case.

Some misc fixes along the way.

TODO:

  • Use in NM_feature_map.h
  • small feature

Release Management

  • Affected package(s): PMP
  • Issue(s) solved (if any): -
  • Feature/Small Feature (if any): todo
  • Link to compiled documentation (obligatory for small feature) wrong link name to be changed
  • License and copyright ownership: no change

@MaelRL MaelRL added Not yet approved The feature or pull-request has not yet been approved. Small feature Pkg::PMP labels Oct 14, 2022
@MaelRL MaelRL added this to the 5.6-beta milestone Oct 14, 2022
@MaelRL MaelRL added the TODO label Oct 14, 2022
/// \return the output iterator.
template <typename PolygonMesh, typename OutputIterator,
typename NamedParameters = parameters::Default_named_parameters>
OutputIterator geometrically_non_manifold_edges(const PolygonMesh& pmesh,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should check Polygon_mesh_processing/include/CGAL/Polygon_mesh_processing/Non_manifold_feature_map.h

@MaelRL MaelRL modified the milestones: 5.6-beta, 5.7-beta Mar 23, 2023
@sloriot sloriot modified the milestones: 6.0-beta, 6.1-beta May 21, 2024
@MaelRL MaelRL modified the milestones: 6.1-beta, 6.2-beta Mar 17, 2025
@sloriot sloriot changed the base branch from master to main September 18, 2025 05:54
@MaelRL MaelRL modified the milestones: 6.2-beta, 6.3-beta Mar 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CHANGES.md not updated Not yet approved The feature or pull-request has not yet been approved. Pkg::PMP Small feature TODO

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants