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

Introduce remove_all_elements(Graph) #7439

Merged
merged 11 commits into from
Apr 22, 2024

Conversation

MaelRL
Copy link
Member

@MaelRL MaelRL commented May 12, 2023

Summary of Changes

Using clear() removes mesh properties, which we might not want to do. The code will still remove everything, but iteratively.

TODO:

  • Small feature

Release Management

  • Affected package(s): BGL, Polygon_mesh_processing
  • Issue(s) solved (if any): -
  • Documentation: link
  • Feature/Small Feature (if any): link pre-approved on 2024/04/04
  • License and copyright ownership: no changes

@MaelRL MaelRL added this to the 5.4.5 milestone May 12, 2023
@MaelRL MaelRL force-pushed the PMP-Remove_degen_faces-no-clear-GF branch from 55ec70f to 7a2e541 Compare May 15, 2023 10:43
@MaelRL MaelRL changed the base branch from 5.4.x-branch to master May 15, 2023 10:43
@MaelRL MaelRL modified the milestones: 5.4.5, 6.0-beta May 15, 2023
@MaelRL MaelRL added Not yet approved The feature or pull-request has not yet been approved. Small feature and removed Work in progress labels May 15, 2023
@sloriot

This comment was marked as outdated.

@MaelRL MaelRL added the TODO label Jun 7, 2023
@MaelRL MaelRL removed the Tested label Jun 21, 2023
@sloriot sloriot added Under Testing Batch_2 Second Batch of PRs under testing and removed Under Testing labels Jun 28, 2023
@sloriot
Copy link
Member

sloriot commented Sep 18, 2023

Successfully tested in CGAL-6.0-Ic-64

@MaelRL MaelRL changed the title Do not use clear() to empty the mesh if all the faces are degenerate Introduce remove_all_elements(Graph) Sep 18, 2023
@github-actions github-actions bot removed the Tested label Apr 3, 2024
Copy link

github-actions bot commented Apr 3, 2024

This pull-request was previously marked with the label Tested, but has been modified with new commits. That label has been removed.

@afabri
Copy link
Member

afabri commented Apr 3, 2024

/build:v0

Copy link

github-actions bot commented Apr 3, 2024

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

@sloriot sloriot added the pre-approved For pre-approved small features. After 15 days the feature will be accepted. label Apr 4, 2024
@sloriot
Copy link
Member

sloriot commented Apr 10, 2024

Successfully tested in CGAL-6.0-Ic-21[56]

@lrineau lrineau self-assigned this Apr 10, 2024
@sloriot sloriot added Accepted small feature and removed Not yet approved The feature or pull-request has not yet been approved. pre-approved For pre-approved small features. After 15 days the feature will be accepted. labels Apr 19, 2024
@sloriot
Copy link
Member

sloriot commented Apr 19, 2024

Ready to be merged

@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 Apr 19, 2024
@lrineau lrineau merged commit c4165fe into CGAL:master Apr 22, 2024
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 Apr 22, 2024
@lrineau lrineau deleted the PMP-Remove_degen_faces-no-clear-GF branch April 22, 2024 13:02
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