Commit Graph

2999 Commits

Author SHA1 Message Date
Sébastien Loriot f98d968ed0 Merge remote-tracking branch 'cgal/6.1.x-branch' into 'cgal/main' 2025-11-19 14:46:33 +01:00
Sébastien Loriot 9bd1705bf8 Merge remote-tracking branch 'cgal/6.0.x-branch' into 'cgal/6.1.x-branch' 2025-11-19 14:45:21 +01:00
Sebastien Loriot dfc5fb5065
IO: write_PLY() for Epeck (#7874)
## Summary of Changes

As reported in #7868 the function `CGAL::IO::write_PLY()` used in binary
mode does not correctly write the coordinates if the points are from a
kernel with exact constructions.
~~This PR applies `to_double()" to the coordinates.~~

~~After a discussion with @MaelRL we decided that the user is in charge
to pass a `vertex_point_map` as named parameter that does the
conversion. This is straightforward as we offer the
[`Cartesian_converter_property_map`](https://doc.cgal.org/latest/Property_map/structCGAL_1_1Cartesian__converter__property__map.html).~~

Moving back to the previous proposal: hardcode some to_double and
to_float casts such that we meet the requirements of the file format,
whatever the input.

As the problem is the same for the vertex normals we add a named
parameter `vertex_normal_map`.

### Todo
- [x] Fix the generic function `write_polygon_mesh()`. Currently it is
fixed for `Surface_mesh`

## Release Management

* Affected package(s): Stream_support
* Issue(s) solved (if any): fix #7868 and fix
https://github.com/CGAL/cgal/issues/7327
* License and copyright ownership:  unchanged
* upcoming integration, update #9072 and test it
2025-11-13 09:38:55 +01:00
Sébastien Loriot 066159f792 assertion should be before the removal... 2025-11-10 10:17:33 +01:00
Sébastien Loriot 3c7d507530 update doc + apply changes to the constrained version 2025-11-06 19:40:48 +01:00
Andreas Fabri 4b89f17b42 CGAL: An Halfedge 2025-11-03 14:37:50 +00:00
Sébastien Loriot 25005a97d8 clean up 2025-11-03 09:24:31 +01:00
Sébastien Loriot 3fe83c7ce0 handle non-triangular faces 2025-10-31 11:36:44 +01:00
Andreas Fabri 622b652652 Add to BGL dependencies 2025-10-30 13:54:35 +01:00
Andreas Fabri 35613b984f Deal with generic version 2025-10-30 11:45:28 +01:00
Sebastien Loriot d13510bdee
Add missing function doc links (#9074)
## Summary of Changes

Fix missing links in doc.

tentative TODO:
- [ ] Check all \`.+\(.+\).*\` --> stopped at BGL
- [ ] Plenty of wrong links for the overloaded functions like <em>A
shortcut to `CGAL::draw(vd2,
Graphics_scene_options_voronoi_diagram_2{})`.</em>, which points to
`CGAL::draw(T2)`.

## Release Management

* Affected package(s): Various
* Issue(s) solved (if any): https://github.com/CGAL/cgal/issues/7839
* Feature/Small Feature (if any): -
* License and copyright ownership: no change
2025-10-17 15:29:09 +02:00
Andreas Fabri d9ba43a9ba Make it constexpr to avoid error in CI 2025-10-17 12:40:30 +01:00
Andreas Fabri 98a3051235 copy style 2025-10-17 10:48:26 +01:00
Sebastien Loriot d9262b1c46
Respected verbose (#9070)
## Summary of Changes

Added a small piece of code that generates an error message if verbose
is on and adding a face fails.
This fix is the bare minimum....
It would be nice to normalize all error messages and make them look
similar.

## Release Management

* Affected package(s): BGL
* Issue(s) solved (if any): fix #9034 partially. It partially fixes item
4. in the list of the bug report
* Feature/Small Feature (if any):
* Link to compiled documentation (obligatory for small feature) [*wrong
link name to be changed*](httpssss://wrong_URL_to_be_changed/Manual/Pkg)
* License and copyright ownership:
2025-10-13 09:32:12 +02:00
Sébastien Loriot 24024424ef use constexpr 2025-10-02 15:11:02 +02:00
Mael Rouxel-Labbé f697aa5237 Also dump can_add_face output when a face fails to be added 2025-10-02 13:35:48 +02:00
Mael Rouxel-Labbé b72814fb2c Add verbose to BGL::can_add_face 2025-10-02 13:35:34 +02:00
Andreas Fabri 39531baac8 Use has_vnormal 2025-10-02 07:23:22 +01:00
Andreas Fabri 737079e530 Fix warning 2025-10-01 09:15:54 +01:00
Andreas Fabri 36b66946b8 after Mael's review 2025-09-30 08:30:55 +01:00
Andreas Fabri 89ba13f977 fix doxygen warning 2025-09-30 08:13:22 +01:00
Andreas Fabri 33bd58bbef fix doxygen warning 2025-09-30 06:47:14 +01:00
Andreas Fabri 5103fb4eb7 merge 2025-09-26 17:18:25 +01:00
Mael Rouxel-Labbé b653138420 Fix some links 2025-09-19 13:26:36 +02:00
Mael Rouxel-Labbé 4591d9a899 Drive-by doc fixes 2025-09-19 00:28:24 +02:00
Mael Rouxel-Labbé 0721be1a58 Use \link / \ref to add missing links of functions with parameters 2025-09-19 00:26:17 +02:00
Efi Fogel 22c7de1d6a Fixed check 2025-09-17 17:01:38 +03:00
Efi Fogel a5300e16f0 Respected verbose 2025-09-17 16:36:10 +03:00
Sebastien Loriot b2eb8d9331
CGAL: Remove some deprecated functions (#9023)
## Summary of Changes

I start removing some IO functions which are annoying to find in the
search box. More to come.

- [x] `CGAL::write_off()`
- [x] `CGAL::write_wrl()`
- [x] `CGAL::write_vtk()`

### Todo

- [ ] remove the test cases.

## Release Management

* Affected package(s): BGL, Surface_mesh, Point_set_processing

* License and copyright ownership:  unchanged
2025-09-03 16:33:06 +02:00
Andreas Fabri 5a77ef7ab6 remove deprecated test from CMakeLists.txt 2025-08-07 11:17:07 +01:00
Andreas Fabri c87a0d8b29 remove deprecated test from CMakeLists.txt 2025-08-07 11:03:08 +01:00
Andreas Fabri 4ef2b0a350 Remove BGL/test/BGL/test_deprecated_io.cpp 2025-08-07 07:05:22 +01:00
Andreas Fabri edd1868f50 CGAL: Remove some deprecated functions 2025-08-05 16:13:44 +01:00
Andreas Fabri 2ec7520d96 BGL: Add Face_around_face_circulator 2025-08-05 15:36:01 +01:00
Sébastien Loriot 26b6857c4f follow up of commits making default value for dynamic maps a parameter pack
the free function becomes a better match than with a default value for the default parameter
2025-07-31 17:33:04 +02:00
Jane Tournois 0ab9d3eebf Merge remote-tracking branch 'cgal/master' into Triangulation_3-CDT_3-lrineau
# Conflicts:
#	Installation/CHANGES.md
2025-06-10 11:18:38 +02:00
Laurent Rineau bdd7988a4a remove CMake warnings about CMP0167
The issue was when VTK/ITK was searched. The
variable `CMAKE_POLICY_DEFAULT_CMP0167` can be
used to workaround that.
2025-05-27 13:02:04 +02:00
Sébastien Loriot 4d13a4d509 better links 2025-05-22 18:56:23 +02:00
Sébastien Loriot 6d4dccdcd2 update links 2025-05-22 18:29:50 +02:00
Sébastien Loriot 13609a6966 no need to repeat the package name as it is on the line above 2025-05-22 17:53:41 +02:00
Laurent Rineau 97d0ece4c8 improve the page about CGAL BGL concepts 2025-05-22 15:02:42 +02:00
Laurent Rineau 73fd95ab6b fix warnings 2025-05-20 16:49:14 +02:00
Laurent Rineau 4b029e69bd Merge branch 'CGAL:master' into Triangulation_3-CDT_3-lrineau 2025-05-13 17:14:39 +02:00
Sébastien Loriot 59921c1eb8 Merge remote-tracking branch 'cgal/6.0.x-branch' into 'master' 2025-05-07 15:58:42 +02:00
Sébastien Loriot 70d5fce9b7 Merge remote-tracking branch 'cgal/5.6.x-branch' into 'cgal/6.0.x-branch' 2025-05-07 15:58:04 +02:00
Andreas Fabri 68b64b0c56 BGL: Fix Issue 8877 2025-05-07 14:29:04 +01:00
Andreas Fabri eeb4decd2a Use named parameter helper 2025-04-11 10:46:23 +02:00
Sébastien Loriot cc77715588 Merge pull request #6849 from JulyCode/gsoc2022-isosurface
Isosurfacing_3 (new package)
2025-04-03 21:41:22 +02:00
Sebastien Loriot ede0864914
BGL - add `CGAL::dijkstra_shortest_path(vs, vt, mesh)` (#8724)
## Summary of Changes

computes the shortest path between two vertices, using Dijkstra
algorithm

Todo
- [x] write doc and integrate to 
- [x] add np
- [x] add measure to define what "shortest" means
- [x] add to ref manual top page
- [x] use in demo code

## Release Management

* Affected package(s): BGL
* [Small
feature](https://cgalwiki.geometryfactory.com/CGAL/Members/wiki/Features/Small_Features/Add_CGAL::shortest_path(vs,_vt))
* Link to compiled documentation
[CGAL::shortest_path_between_two_vertices()](https://cgal.github.io/8724/v0/BGL/group__PkgBGLTraversal.html#ga5627dee8a6fe4d238cb5c91e123aa763)
* License and copyright ownership: unchanged
2025-04-03 16:07:02 +02:00
Sebastien Loriot 0a47430f59
remove not used namespace 2025-03-27 10:04:38 +01:00