fixes after review

This commit is contained in:
Sébastien Loriot 2022-04-11 15:42:59 +02:00
parent a4f4c276f3
commit 2093e60645
4 changed files with 13 additions and 10 deletions

View File

@ -39,7 +39,7 @@ Release date: June 2022
### [Polygon Mesh Processing](https://doc.cgal.org/5.5/Manual/packages.html#PkgPolygonMeshProcessing) ### [Polygon Mesh Processing](https://doc.cgal.org/5.5/Manual/packages.html#PkgPolygonMeshProcessing)
- Added the function `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_soup()`, which enables re-orienting the faces of a triangle soup based on the orientation of the nearest face in a reference triangle soup. - Added the function `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_soup()`, which enables re-orienting the faces of a triangle soup based on the orientation of the nearest face in a reference triangle soup.
- Added the function `CGAL::Polygon_mesh_processing::connected_components_compatible_orientations()`, which enables to retrieve the (in)compatibility of orientations of faces from different connected components. - Added the function `CGAL::Polygon_mesh_processing::compatible_orientations()`, which enables to retrieve the (in)compatibility of orientations of faces from different connected components.
- Added the function `CGAL::Polygon_mesh_processing::tangential_relaxation()`, which applies an area-based tangential mesh smoothing to the vertices of a surface triangle mesh. - Added the function `CGAL::Polygon_mesh_processing::tangential_relaxation()`, which applies an area-based tangential mesh smoothing to the vertices of a surface triangle mesh.
### [2D Polygons](https://doc.cgal.org/5.5/Manual/packages.html#PkgPolygon2) ### [2D Polygons](https://doc.cgal.org/5.5/Manual/packages.html#PkgPolygon2)

View File

@ -132,7 +132,7 @@ The page \ref bgl_namedparameters "Named Parameters" describes their usage.
- `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_mesh()` - `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_mesh()`
- `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_soup()` - `CGAL::Polygon_mesh_processing::orient_triangle_soup_with_reference_triangle_soup()`
- `CGAL::Polygon_mesh_processing::merge_reversible_connected_components()` - `CGAL::Polygon_mesh_processing::merge_reversible_connected_components()`
- `CGAL::Polygon_mesh_processing::connected_components_compatible_orientations()` - `CGAL::Polygon_mesh_processing::compatible_orientations()`
\cgalCRPSection{Hole Filling Functions} \cgalCRPSection{Hole Filling Functions}
- `CGAL::Polygon_mesh_processing::triangulate_hole()` - `CGAL::Polygon_mesh_processing::triangulate_hole()`

View File

@ -55,7 +55,7 @@ int main()
// determine face orientations to be reversed to create compatibility // determine face orientations to be reversed to create compatibility
auto fbm = mesh.add_property_map<Mesh::Face_index, bool>("fbm", false).first; auto fbm = mesh.add_property_map<Mesh::Face_index, bool>("fbm", false).first;
bool is_orientable = PMP::connected_components_compatible_orientations(mesh, fbm); bool is_orientable = PMP::compatible_orientations(mesh, fbm);
assert(is_orientable); assert(is_orientable);
// reverse orientation of faces with bit 1 // reverse orientation of faces with bit 1

View File

@ -1608,10 +1608,9 @@ void merge_reversible_connected_components(PolygonMesh& pm,
/*! /*!
* \ingroup PMP_orientation_grp * \ingroup PMP_orientation_grp
* *
* identifies possible connections between the connected components of `pm` and if re-orientable, * identifies faces whose orientation must be reversed in order to enable stitching of connected components.
* assigns a bit (`false` or `true`) to each face such that * Each face is assigned a bit (`false` or `true`)
* two faces with the same bits have a compatible orientation, while * such that two faces have compatible orientations iff they are assigned the same bits.
* two faces with opposite bits have an incompatible orientation.
* *
* @tparam PolygonMesh a model of `HalfedgeListGraph`, `FaceGraph`. * @tparam PolygonMesh a model of `HalfedgeListGraph`, `FaceGraph`.
* @tparam FaceBitMap a model of `WritablePropertyMap` with `face_descriptor` as key and `bool` as value_type * @tparam FaceBitMap a model of `WritablePropertyMap` with `face_descriptor` as key and `bool` as value_type
@ -1635,9 +1634,13 @@ void merge_reversible_connected_components(PolygonMesh& pm,
* should be available for the vertices of `pm`.} * should be available for the vertices of `pm`.}
* \cgalParamNEnd * \cgalParamNEnd
* \cgalNamedParamsEnd * \cgalNamedParamsEnd
*
* \sa reverse_face_orientations()
* \sa stitch_borders()
*
*/ */
template <class PolygonMesh, class FaceBitMap, class NamedParameters = parameters::Default_named_parameters> template <class PolygonMesh, class FaceBitMap, class NamedParameters = parameters::Default_named_parameters>
bool connected_components_compatible_orientations(const PolygonMesh& pm, bool compatible_orientations(const PolygonMesh& pm,
FaceBitMap fbm, FaceBitMap fbm,
const NamedParameters& np = parameters::default_values()) const NamedParameters& np = parameters::default_values())
{ {