CGAL users discussion list

["Nico Kruithof" <>]

Hi Benjamin,

The order of the include files shoudn't matter and if it does, I
consider it a bug in my code. In the meantime I switched to industry
and have a deadline on tuesday. After that I'll have a look. At one
point, the algorithm was extremely fast, but also used a lot of
memory. How much memory do you have? In the latest public release
there is a better tradeoff between cpu times and memory consumption.

Bests,
Nico

On Fri, Nov 21, 2008 at 11:16 AM, Benjamin schwarz
<>
 wrote:
> Well... I can't explain why, but I just changed the order of my
> includes... "et voilà" (What is the english for "c'est tombé en
> marche" ?), now it runs a lot faster and in accordance with the
> expected times.
>
> Still, is it possible to choose the subdivision scheme ?
>
> I don't know if the dependance on includes order is a normal behavior,
> if not, here is a hint to solve the problem :
>
> My actual includes
> #include <CGAL/Exact_predicates_inexact_constructions_kernel.h>
> #include <CGAL/Skin_surface_3.h>
> #include <CGAL/Polyhedron_incremental_builder_3.h>
> #include <CGAL/Polyhedron_3.h>
> #include <CGAL/mesh_skin_surface_3.h>
> #include <CGAL/subdivide_skin_surface_mesh_3.h>
>
> #include <CGAL/IO/Geomview_stream.h>
> #include <CGAL/IO/Polyhedron_geomview_ostream.h>
> #include <CGAL/Timer.h>
> #include <list>
> #include <iostream>
> #include <fstream>
>
>
>
> my former includes :
> #include <CGAL/Exact_predicates_inexact_constructions_kernel.h>
> #include <CGAL/Polyhedron_incremental_builder_3.h>
> #include <CGAL/Polyhedron_3.h>
> #include <CGAL/IO/Geomview_stream.h>
> #include <CGAL/IO/Polyhedron_geomview_ostream.h>
> #include "skin_surface_writer.h"
> #include <CGAL/Skin_surface_3.h>
> #include <CGAL/Polyhedron_3.h>
> #include <CGAL/mesh_skin_surface_3.h>
> #include <CGAL/subdivide_skin_surface_mesh_3.h>
> #include <CGAL/Timer.h>
> #include <list>
> #include <iostream>
> #include <fstream>
>
>
>> -DNDEBUG is important for performances.
> I already had a bad experience with that a long time ago ;)
>
>> Try -O3 instead of -O2.
> Ok, I'll try that
>
>> That one is correct too. -O2 is useless here.
> right.
>
> Thanks Laurent
> --
> You are currently subscribed to cgal-discuss.
> To unsubscribe or access the archives, go to
> https://lists-sop.inria.fr/wws/info/cgal-discuss
>