cgal icon indicating copy to clipboard operation
cgal copied to clipboard

Triangulation on hyperbolic surface 2: Delaunay triangulations and epsilon-nets

Open camille-lanuel opened this issue 1 month ago • 10 comments

Summary of Changes

This feature implements Delaunay triangulations on hyperbolic surfaces. Functionalities include point location, point insertion, and the computation of an ε-net of the hyperbolic surface, which is a set of well-distributed points on the surface controlled by the parameter ε.

Release Management

  • Affected package(s): Triangulation on hyperbolic surface 2, Hyperbolic triangulation 2
  • Feature
  • Link to compiled documentation
  • License and copyright ownership: GPL (v3 or later)

camille-lanuel avatar Nov 28 '25 09:11 camille-lanuel

/build:v1

camille-lanuel avatar Nov 28 '25 10:11 camille-lanuel

/build:v1

camille-lanuel avatar Nov 28 '25 10:11 camille-lanuel

The documentation is built. It will be available, after a few minutes, here: https://cgal.github.io/9158/v1/Manual/index.html

github-actions[bot] avatar Nov 28 '25 10:11 github-actions[bot]

I know that it is not a mistake introduced by this PR, but can you add the typedef for FT here

afabri avatar Nov 28 '25 13:11 afabri

And a typo "reprensenting" here

Also there should not be text in red in the User Manual

afabri avatar Nov 28 '25 13:11 afabri

"\f$ \varepsilon \f$-net" as that in the html here

afabri avatar Nov 28 '25 13:11 afabri

And a typo "reprensenting" here

Also there should not be text in red in the User Manual

I think I got it. Red means what you added in this PR

afabri avatar Nov 28 '25 13:11 afabri

You must add Kernel_23 in the dependencies file

afabri avatar Nov 28 '25 15:11 afabri

You must add Kernel_23 in the dependencies file

Kernel_23 is already in /package_info/dependencies. Do you mean to add it in /doc/dependencies ?

camille-lanuel avatar Nov 28 '25 15:11 camille-lanuel

You must add Kernel_23 in the dependencies file

Kernel_23 is already in /package_info/dependencies. Do you mean to add it in /doc/dependencies ?

Yes, because you use Simple_cartesian in an example.

afabri avatar Nov 28 '25 15:11 afabri