CGAL users discussion list

["Sebastien Loriot (GeometryFactory)" <>]

I think the problems comes from my fix. Could you try this:

--- a/include/CGAL/Intersections_3/intersection_3_1_impl.h
+++ b/include/CGAL/Intersections_3/intersection_3_1_impl.h
@@ -57,7 +57,7 @@ intersection(const typename K::Plane_3  &plane,
             return Object();
         }
     }
-    return make_object(k.construct_plane_3_object()(
+    return make_object(k.construct_point_3_object()(
         den*line_pt.hx()-num*line_dir.dx(),
         den*line_pt.hy()-num*line_dir.dy(),
         den*line_pt.hz()-num*line_dir.dz(),

Sebastien.

On 02/08/2013 12:38 PM, jeanbaptiste lespiau wrote:
> Thank you for this very fast response.
>
> I have replaced the headers directly in CGAL.
>
> Now "CGAL::make_skin_surface_mesh_3(p, l.begin(), l.end(),
> shrinkfactor);" completly works :)
>
>
> CGAL::subdivide_skin_surface_mesh_3(skin_surface, p);  compiles without
> errors. However, when launched it raises an invalid assertion:
>
> terminate called after throwing an instance of 'CGAL::Assertion_exception'
>    what():  CGAL ERROR: assertion violation!
> File: /usr/local/include/CGAL/Skin_surface_base_3.h
> Line: 484
> Explanation: intersection: no intersection.
>
> Since I have applied subdivide on a theorically valid skin surface
> (created by mesh_skin_surface_3), I think this is still not normal.
>
>
> Thank you for your help.
>
>
> 2013/2/8 Sebastien Loriot (GeometryFactory) 
> <
> <mailto:>>
>
>     Attached to this email, you'll find patched header files than need to be
>     used instead of those from CGAL (you can edit your CMakeLists.txt and
>     add the patch to this directory  include_directories(BEFORE XXX)  )
>
>     Let us know if this solves your problem.
>
>
>     Sebastien.
>
>     On 02/04/2013 11:12 PM, J-B wrote:
>
>         Hello,
>         I am using the CGAL example Skin_surface_3 /
>         skin_surface_pdb_reader.cpp to
>         build a skin surface from an .pdb file.
>
>         It works (of course, the example does), but when I try to use other
>         functions I found at
>         
> http://www.cgal.org/Manual/__latest/doc_html/cgal_manual/__Skin_surface_3/Chapter_main.__html
>         
> <http://www.cgal.org/Manual/latest/doc_html/cgal_manual/Skin_surface_3/Chapter_main.html>
>         awfull errors arrive.
>
>         In particular, it is about the two
>         functions:CGAL::make_skin___surface_mesh_3(p, l.begin(), l.end(),
>         shrinkfactor); (does 2 actions in
>         1)andCGAL::subdivide_skin___surface_mesh_3(skin_surface, p);
>
>         I think it is because ESBTL defines it owns Kernel (typedef
>         ESBTL::CGAL::EPIC_kernel_with___atom    K;) while CGAL uses typedef
>         CGAL::Exact_predicates___inexact_constructions_kernel K, but
>         ESBTL doesn't
>         work if I uses the last typedef.
>
>         I put an example here :
>         
> https://github.com/Gueust/__Pymol-Tunnels-Cavities-Finder/__blob/master/Example.tar.gz
>         
> <https://github.com/Gueust/Pymol-Tunnels-Cavities-Finder/blob/master/Example.tar.gz>
>         using the smaller code possible with one line that creates the
>         errors.
>
>         To make it compile, if you have both ESBTL and CGAL you only
>         need to go to
>         build/ and do cmake .
>
>         It may be as simple as modifying a typedef, but being not
>         skilled with those
>         libraries I am completely lost. If someone has the answer it
>         could be great.
>
>         Thank you.
>
>
>
>         --
>         View this message in context:
>         
> http://cgal-discuss.949826.n4.__nabble.com/Use-of-ESBTL-and-__Skin-Surface-create-errors-__tp4656653.html
>         
> <http://cgal-discuss.949826.n4.nabble.com/Use-of-ESBTL-and-Skin-Surface-create-errors-tp4656653.html>
>         Sent from the cgal-discuss mailing list archive at Nabble.com.
>
>
>
>     --
>     You are currently subscribed to cgal-discuss.
>     To unsubscribe or access the archives, go to
>     https://sympa.inria.fr/sympa/__info/cgal-discuss
>     <https://sympa.inria.fr/sympa/info/cgal-discuss>