search

CGAL / cgal

The public CGAL repository, see the README below
https://github.com/CGAL/cgal#readme
Other
4.96k stars 1.38k forks source link

Unify face normal computation #326

Closed afabri closed 8 years ago

afabri commented 9 years ago

The package Nef_3 has a file CGAL/normal_vector_newell.h This might be provided by the package Polygon mesh processing? Where does it differ from what we have in PMP?

sloriot commented 9 years ago

See also Surface_mesh_skeletonization/include/CGAL/internal/Surface_mesh_skeletonization/get_normal.h

sloriot commented 9 years ago

In Nef this computes the normal of the plane containing a set of points.

afabri commented 9 years ago

It does not produce a unit length vector, and hence also works for Homogeneous

afabri commented 9 years ago

I am wondering, if we should not have different functions for with and without the unit normal. We have that for distances, where we only offer CGAL::squared_distance().

janetournois commented 8 years ago

this issue should be closed @sloriot @lrineau

lrineau commented 8 years ago

The PR #957 is still not merged. Why should this issue be closed?

sloriot commented 8 years ago

This issue is not closed, #155 which is a duplicate is.