Skip to content

Surface Mesh Segmentation Using Region Growing#6024

Closed
danston wants to merge 119 commits intoCGAL:masterfrom
danston:Surface_mesh_segmentation-use_region_growing-danston
Closed

Surface Mesh Segmentation Using Region Growing#6024
danston wants to merge 119 commits intoCGAL:masterfrom
danston:Surface_mesh_segmentation-use_region_growing-danston

Conversation

@danston
Copy link
Copy Markdown
Contributor

@danston danston commented Sep 29, 2021

This PR is a draft version of the planar decimation feature from PMP using Region Growing when segmenting an input mesh. We use Region Growing first to detect planar shapes. We then exploit these shapes to build a graph of polylines on the mesh and apply Region Growing again on top of this graph in order to detect linear regions. When we know all linear regions, we can easily detect unique corners of the mesh.

This PR depends on PR #5563 and PR #5461.

Files to check:

  • PMP decimation_planar_patches.cpp
  • PMP planar_segmentation.h
  • SMS plane_mesh_segmentation.h

Release Management

  • Affected package(s): Polygon_mesh_processing, Surface_mesh_segmentation
  • Issue(s) solved (if any): no fixes
  • Feature/Small Feature (if any): to be discussed
  • Link to compiled documentation (obligatory for small feature): no docs, only internal changes
  • License and copyright ownership: no changes

sloriot and others added 30 commits February 15, 2021 09:32
@danston danston added the depends on another PR This pull-request should only be merged after other ones. label Sep 30, 2021
@MaelRL MaelRL added this to the 5.5-beta1 milestone Oct 25, 2021
@MaelRL MaelRL modified the milestones: 5.5-beta, 5.6-beta Apr 7, 2022
@sloriot sloriot closed this Jun 24, 2022
@sloriot
Copy link
Copy Markdown
Member

sloriot commented Jun 24, 2022

We decided not to add a new API to the segmentation package.

@lrineau lrineau removed this from the 5.6-beta milestone Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants