CGAL users discussion list

[Chris Marsh <>]

Ah I just made a note of the type in the comments. I think auto is stillpreferable for the overall example.

I'm sorry to hear this is being so problematic. I've attached the code and cmake file to this email -- I hope the list doesn't strip it out. I've cc'd you on this email just to be sure.

This code correctly compiles and runs for me.

> With this described approach, I  will create several times the same point.

You're right, I was being too general sorry.

I'll describe what I do:

Since the Vertex_handle is just a pointer, you can store them with low overhead.  Then, when you read a face that uses it, you just use that existing i-th vertex. For example:

here I read and make all the vertexes

https://github.com/Chrismarsh/CHM/blob/master/src/mesh/triangulation.cpp#L248

then I grab the i-th vertex when I make the new face, so I never duplicate a vertex

https://github.com/Chrismarsh/CHM/blob/master/src/mesh/triangulation.cpp#L279

I hope that's more clear,

Cheers

Chris

Chris Marsh
PhD Candidate
chrismarsh.ca

On Thu, 8 Nov 2018 at 19:37, Yoann LE BARS <> wrote:

Hello everybody out there!

On 2018/11/09 at 01:47 am, Chris Marsh wrote:
> Oh, no auto is too bad. In this case, it's triangulation::Vertex_handle
> and triangulation::Face_handle. You have to use the template derived class.
> I've updated the SO to make this more clear. Does this let you compile now?

        Well, I cannot see any difference on Stackoverflow.

        Anyway, as I indicated in the previous message, in my own code when I
use “Triangulation::Vertex_handle”, I have this compilation error
indicating it does not provide “set_point”.

> I fear it won't really help you, but on off chance it does, this is a
> section of code where I build the CGAL data structure from a file
> https://github.com/Chrismarsh/CHM/blob/master/src/mesh/triangulation.cpp#L232

        I have started to read this, I will take some times to investigate it.

> The general approach with millions of faces is to, while reading in the
> gmsh/whatever file, you create the 3 vertexes, insert the face, and move
> on.

        It sounds quite reasonable, but I do not know how to do this, as a
vertex usually belongs to several faces. With this described approach, I
will create several times the same point.

        Best regards.

--
Yoann LE BARS
http://le-bars.net/yoann/
Diaspora* :

--
You are currently subscribed to cgal-discuss.
To unsubscribe or access the archives, go to
https://sympa.inria.fr/sympa/info/cgal-discuss

cmake_minimum_required (VERSION 3.12)
project (tri)
find_package(CGAL REQUIRED)
include(${CGAL_USE_FILE})



include_directories(
                ${CGAL_INCLUDE_DIRS}
        ${CGAL_3RD_PARTY_INCLUDE_DIRS})

add_executable(main
                                main.cpp)

target_link_libraries(main ${CGAL_3RD_PARTY_LIBRARIES})

Attachment: main.cpp
Description: Binary data