Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Triangulation_3 - add vertices() helper functions #7808

Merged

Conversation

janetournois
Copy link
Member

@janetournois janetournois commented Oct 19, 2023

Summary of Changes

This PR introduces new helper functions in Triangulation_3, that returns a std::array<Vertex_handle, dim_of_the_simplex> for each type of simplex (vertex, edge, facet, cell).

Iterating over the vertices of a simplex is implemented quite a few times in Tetrahedral_remeshing for example, and these helper functions would help the readability of the code (and also avoid mistakes).

Release Management

  • Affected package(s): Triangulation_3
  • Small Feature : here
  • Link to compiled documentation here
  • License and copyright ownership: unchanged

@janetournois
Copy link
Member Author

/build:v0

@github-actions
Copy link

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

@janetournois
Copy link
Member Author

/force-build:v0

@github-actions
Copy link

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

@janetournois janetournois added Small feature Not yet approved The feature or pull-request has not yet been approved. labels Oct 19, 2023
@afabri
Copy link
Member

afabri commented Oct 19, 2023

Note that in the 2D case for faces we have vertices_around_face(Face_handle, const Triangulation&)

@janetournois
Copy link
Member Author

/build:v1

@github-actions
Copy link

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

@lrineau lrineau added pre-approved For pre-approved small features. After 15 days the feature will be accepted. and removed Not yet approved The feature or pull-request has not yet been approved. labels Oct 24, 2023
@janetournois
Copy link
Member Author

/build:v2

@github-actions
Copy link

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

@sloriot sloriot added Batch_1 First Batch of PRs under testing Under Testing and removed Batch_1 First Batch of PRs under testing labels Oct 31, 2023
@lrineau lrineau added Accepted small feature and removed pre-approved For pre-approved small features. After 15 days the feature will be accepted. labels Nov 9, 2023
@sloriot
Copy link
Member

sloriot commented Nov 15, 2023

Successfully tested in CGAL-6.0-Ic-106

@lrineau lrineau self-assigned this Nov 15, 2023
@lrineau lrineau added the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Nov 15, 2023
@lrineau lrineau merged commit 01f021d into CGAL:master Nov 15, 2023
9 checks passed
@lrineau lrineau removed the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Nov 15, 2023
@lrineau lrineau deleted the Triangulation_3-vertices_array-jtournois branch November 15, 2023 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants