CGAL users discussion list

["Sebastien Loriot (GeometryFactory)" <>]

Hello,

if you add a call to
  PMP::merge_duplicated_vertices_in_boundary_cycles(mesh); [1]

before detecting border cycles, then you will get what you want.

I tested it locally and your example now adds only 3 new triangles.

Best,

Sebastien.

[1] 
https://doc.cgal.org/latest/Polygon_mesh_processing/group__PMP__combinatorial__repair__grp.html#ga2af4c26afc4872bafd0caabe6af23f9a

On 12/12/24 10:11, Matthijs Sypkens Smit 
( via cgal-discuss Mailing List) wrote:
> Hello all,
>
> I'm wondering whether it is possible to use CGAL to perform hole-filling 
> for a mesh as provided by /sawtooth.obj/.
> I have not been able to obtain a satisfactory result. In particular the 
> attached example compiles, and runs, but produces more triangles than 
> are expected.
> Probably this is because 3 non-manifold vertices are duplicated. The 
> documentation for 'triangulate_hole()' stipulates that there cannot be a 
> non-manifold vertex in the boundary, which is achieved by duplication, 
> but apparently such a case is not expected to work?
> Is there a way to fill the holes of the /sawtooth.obj/ input with CGAL?
>
> Attached you find code to reproduce the hole-filling (/main.cpp/), the 
> input file /sawtooth.obj/, the generated output 
> /sawtooth_filled.obj/ and a screenshot that shows that there are more 
> triangles than we would have expected. The input has 44 triangles, 
> whereas the output has 54 triangles. For this example I would like the 
> output to have 44+4 = 48 triangles.
>
> With regards,
>
> --
> Matthijs Sypkens Smit
>
> --
> You are currently subscribed to cgal-discuss.
> To unsubscribe or access the archives, go to
> https://sympa.inria.fr/sympa/info/cgal-discuss