CGAL users discussion list

["Nico Kruithof" <>]

Hi Sylvain,

I didn't know about the assertion categorization. I'll check the
assertions and move some of them to the expensive set.

Nico

On Fri, Nov 21, 2008 at 11:25 AM, Sylvain Pion
<>
 wrote:
> 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>
>
> This one is probably for Nico.
>
>>> -DNDEBUG is important for performances.
>>
>> I already had a bad experience with that a long time ago ;)
>
> Note that NDEBUG is not supposed to dramatically improve performance.
> The difference is supposed to be around 20% maximum, and at the maximum
> one order of magnitude.  If you notice more than that, please report it.
> CGAL has several categories of assertions, and the costly ones should
> not be activated by default (CGAL_CHECK_EXPENSIVE enables those).
> So there might be some errors in the categorization.
>
> --
> Sylvain Pion
> INRIA Sophia-Antipolis
> Geometrica Project-Team
> CGAL, http://cgal.org/
>
>