Sebastien Loriot
Sebastien Loriot
Successfully tested in CGAL-5.6-Ic-79
/build:v1
Successfully tested in CGAL-5.6-Ic-88
If the bug you are trying to fix is the division by 0 when doing the normalization, then we can simply check that the normalization is not done using 0.
The code added is specific to `Surface_mesh`. [Dynamic properties](https://doc.cgal.org/latest/BGL/index.html#title25) should be used.
Did you try calling [`CGAL::Polygon_mesh_processing::stitch_borders()`](https://doc.cgal.org/latest/Polygon_mesh_processing/group__PMP__repairing__grp.html#ga8ae4352e67d2b099994ac8990c13bd41) ?
@danston I'd be interested to discuss region growing on a mesh again to see if I can replace the PCA approach with that.
@danston I moved the segmentation part into the `Surface_mesh_segmenation()` package, into the function `segment_via_plane_fitting()`.
action failed because of this error: `Differences in Polygon_mesh_processing: Principal_component_analysis are new and not committed.`
@ochafik Should be fixed now.