Commit Graph

110837 Commits

Author SHA1 Message Date
Andreas Fabri 8305f2b37d
Update Point_set_processing_3/include/CGAL/poisson_eliminate.h 2025-03-07 17:32:52 +01:00
Sven Oesau 058d7705fa [skip ci] 2025-02-25 13:20:52 +01:00
Sven Oesau e4f420b53a update changes.md
[skip ci]
2025-02-25 13:19:55 +01:00
Sven Oesau 72b6c16fff updated changes.md 2025-02-25 12:42:57 +01:00
Sven Oesau 24d44e786b Merge remote-tracking branch 'cgal/master' into PMP-PoissonDiskSampling-GF 2025-02-25 12:24:19 +01:00
Sven Oesau bee8a662f6 renaming helper struct to Vector_matching_point
updates following review
2025-02-25 12:20:51 +01:00
Sebastien Loriot 299a89f212
Added a precondition for Arrangement_on_surface_2::merge_edge(e1, e2) (#8742)
Added a precondition for merge_edge(e1, e2) that requires that two
halfedges have the same direction.

## Summary of Changes

The title says it all

## Release Management

* Affected package(s): Arrangement_on_surface_2
* Issue(s) solved (if any): fix #8659
* Feature/Small Feature (if any): 
* Link to compiled documentation:
* License and copyright ownership: TAU
2025-02-24 06:38:09 +01:00
Andreas Fabri 98025ef15b Escape @ to avoid generation of a link 2025-02-22 10:44:12 +00:00
Andreas Fabri 6e3bffebc6 polish 2025-02-22 09:52:03 +00:00
Sebastien Loriot 18f0405065
Spelling correction (#8746)
Spelling correction
2025-02-21 06:56:19 +01:00
Sebastien Loriot 65202060af
Spelling correction (#8747)
Spelling correction

As this is a spelling correction is in code (and outside my normal realm
of PRs) I created a separate PR for it. I cannot test it, b I'm quite
confident that it won't lead to problems. I think the old name contained
a typo.
2025-02-21 06:55:34 +01:00
Sven Oesau 6bcfc6b2c3 fix warnings 2025-02-20 08:32:53 +01:00
albert-github cb1f3fb37a Spelling correction
Spelling correction

As this is a spelling correction is in code (and outside my normal realm of PRs) I created a separate PR for it.
I cannot test it,  b I'm quite confident that it won't lead to problems. I think the old name contained a typo.
2025-02-19 11:38:14 +01:00
albert-github cc9ab04a27 Spelling correction
Spelling correction
2025-02-19 11:29:09 +01:00
Sébastien Loriot 4f9bf703e3 Merge remote-tracking branch 'cgal/6.0.x-branch' in master 2025-02-19 07:19:25 +01:00
Sébastien Loriot f7629fa682 Merge remote-tracking branch 'cgal/5.6.x-branch' into cgal/6.0.x-branch 2025-02-19 07:18:41 +01:00
Sebastien Loriot 133b48b55c
Polyline_simplification: Fix Detection of Unremovable Vertices (#8736)
## Summary of Changes

We missed a variation of overlapping constraints.

## Release Management

* Affected package(s): Polyline_simplification 
* Issue(s) solved (if any): fix #8735
* License and copyright ownership:  unchanged
2025-02-19 07:17:21 +01:00
Sebastien Loriot ad215de1f0
Incorrect rendered formulas (#8732)
The formula in Documentation/doc/biblio/cgal_manual.bib
```
 @article{cgal:cgm-fobbo-11,
   title={Fast Oriented Bounding Box Optimization on the Rotation Group $\SO(3, \mathrm{R})$},
   author={Chang, Chia-Tche and Gorissen, Bastien and Melchior, Samuel},

```
renders incorrectly (see Optimal_bounding_box/citelist.html) as there is
no command `\SO` in LaTeX

The formula in doc/Kernel_d/CGAL/Kernel_d/Aff_transformation_d.h
```
\pre \f$ sin_num^2 + cos_num^2 = den^2\f$ and \f$ 0 \leq e_1 < e_2 < d\f$.
```
renders in an unexpected way (see
Kernel_d/classCGAL_1_1Aff__transformation__d.html) as the `_` is seen as
subscript operator in LaTeX
2025-02-19 07:15:02 +01:00
Sebastien Loriot 8c8618ad04
Remove check doc todos (#8738)
We can consider them as done now
2025-02-19 07:13:33 +01:00
Sebastien Loriot 838f4b8b12
Cmap add test gdamiand (#8737)
## Summary of Changes

Solve a bug for CMap and GMap when we iterate on an empty map with an
iterator that needs to mark cells, and when using index version.

## Release Management

* Affected package(s): CMap GMap
2025-02-19 07:12:54 +01:00
Efi Fogel 40646af54e Added missing const to some traits operators 2025-02-18 17:40:01 +02:00
Sven Oesau 44bce83850 fixed warning 2025-02-18 08:49:32 +01:00
Sven Oesau 093a105716 integrated changes from reviews 2025-02-17 09:37:00 +01:00
Efi Fogel 31f271b88c Added a precondition for merge_edge(e1, e2) that requires that two halfedges have the same direction. 2025-02-17 10:17:11 +02:00
Sven Oesau 2d32b6dcfd
Apply suggestions from code review
Co-authored-by: Sebastien Loriot <sloriot.ml@gmail.com>
2025-02-17 08:49:42 +01:00
Sven Oesau a381b9c6f1 doc bugfix 2025-02-14 15:14:19 +01:00
Sven Oesau 40f70b8521 update on doc and image 2025-02-14 15:10:13 +01:00
Sven Oesau d92cf1f6a8 added missing header 2025-02-14 13:59:31 +01:00
Sven Oesau d673998f52 integrating comments from reviews
poisson_eliminate now works with d-dimensional points
2025-02-14 13:24:16 +01:00
Sébastien Loriot 66965f3a07 remove todo 2025-02-14 11:03:32 +01:00
Guillaume Damiand 4a0e20f938 Bug fix for compact container with index, when iterating on empty container. 2025-02-13 17:47:29 +01:00
Guillaume Damiand 5190030a73 update cmap test empty iterator 2025-02-13 16:11:12 +01:00
Guillaume Damiand 3942e1e2a6 add cmap test 2025-02-13 15:12:15 +01:00
Sven Oesau 2055a79f4e making poisson_eliminate compatible with Point_set_3
integrating reviews
2025-02-13 12:36:43 +01:00
Sébastien Loriot 06b511cc65 Merge pull request #6088 from sloriot/PMP-add_discrete_curvature
Add functions to compute discrete curvatures
2025-02-12 21:22:40 +01:00
Sébastien Loriot de1fb95d15 Merge pull request #8658 from MaelRL/Tr-Document_point-GF
Document `point()` in all triangulations
2025-02-12 21:18:27 +01:00
Sébastien Loriot 9c536ddae6 Merge pull request #8234 from MaelRL/BGL-Add_remove_deg_2_vertex-GF
Add an Euler operation to remove degree 2 vertices
2025-02-12 21:17:43 +01:00
Sébastien Loriot b879d39472 Merge remote-tracking branch 'cgal/6.0.x-branch' 2025-02-12 19:49:46 +01:00
Sébastien Loriot 9b1c378424 Merge remote-tracking branch 'cgal/5.6.x-branch' into 6.0.x-branch 2025-02-12 19:49:09 +01:00
Sebastien Loriot 8836f5e705
Polygon repair using union and intersection (#8510)
## Summary of Changes

Add new repair rules in Polygon_repair package

## Release Management

* Affected package(s): Polygon_repair
* Feature/Small Feature (if any):
[link](https://cgalwiki.geometryfactory.com/CGAL/Members/wiki/Features/Small_Features/Polygon_repair_union_intersection)
-- **pre-approval 2025/01/03**
* Link to compiled documentation:
[link](https://cgal.geometryfactory.com/~afabri/Repair/Manual/packages.html#PkgPolygonRepair)
* License and copyright ownership: GF
2025-02-12 19:47:30 +01:00
Sebastien Loriot f079630f6e
issue #8689 Regression documentation build (#8698)
- compare doxygen version with the available directories on disk and
(try to) use the most appropriate one
- give a warning when there are more directories than expected (already
present), though `FATAL` should probably be `FATAL_ERROR` though the
latest will give problems when adding a new version to the directory
list for testing. Changed the message to a `WARNING`
2025-02-12 19:45:14 +01:00
Sébastien Loriot 043becfe7f Merge pull request #8273 from lrineau/CGAL-followup_PR_8170-lrineau
Improvement backports from CDT_3 branch (Follow-up to PR #8170)
2025-02-12 19:43:35 +01:00
Sebastien Loriot 636368ed8f
Always init time_stamp to -2 (#8714)
It is not clear to me that this is the right thing to do:
- it is done for some classes
- but in some test, a comment mention that it should be uninitialized
(the test works if init to -1 but not to 0).
- Base_time_stamper is doing an init to -2
2025-02-12 19:40:27 +01:00
Sebastien Loriot 925c15fd98
Correct documentation mistakes in double.h and long_double.h (#8729)
## Release Management

* Affected package(s):
* Issue(s) solved (if any): fix #7781 #7782 
* 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-02-12 19:39:16 +01:00
Sebastien Loriot 14b83f0d3e
boost::tie -> std::tie (#8715) 2025-02-12 19:36:19 +01:00
Sebastien Loriot 47263d3aaa
Fix size types in BGL doc (#8722)
## Release Management

* Affected package(s): `BGL`
* Issue(s) solved (if any): fix #8720
* Feature/Small Feature (if any): -
* License and copyright ownership: no change
2025-02-12 19:35:26 +01:00
Sebastien Loriot 3394b6d889
Update application name (#8731) 2025-02-12 19:34:53 +01:00
Andreas Fabri c8e9357080 Use std::prev() and std::next() instead of their boost equivalents 2025-02-12 17:50:35 +00:00
Andreas Fabri 98b6cf398d change iterator category to bidirectional 2025-02-12 17:48:55 +00:00
Andreas Fabri bc6d19e11b The bug was to compare prior and next, whereas we have to compare *prior with *next 2025-02-12 17:35:28 +00:00