Skip to content

PMP::isotropic_remeshing - add edge_is_ protected_map#9482

Open
janetournois wants to merge 7 commits into
CGAL:mainfrom
janetournois:PMP_remeshing-edge_is_protected_map-jtournois
Open

PMP::isotropic_remeshing - add edge_is_ protected_map#9482
janetournois wants to merge 7 commits into
CGAL:mainfrom
janetournois:PMP_remeshing-edge_is_protected_map-jtournois

Conversation

@janetournois

@janetournois janetournois commented May 18, 2026

Copy link
Copy Markdown
Member

Summary of Changes

With this addition, the user can combine

  • constrained edges, i.e. that can be only split or collapsed, using the NP edge_is_constrained_map, and
  • protected edges, i.e. that cannot be modified at all, using the NP edge_is_protected_map.

Release Management

for
* for edges of edge_is_protected_map, protect_constraints is ON
* for edges of edge_is_constrained_map, they are modified depending on protect_constraints

+ add edge_is_protected_map to split_long_edges() too
+ when calling PMP::split_long_edges(), do NOT split protected edges
+ use has_protected_edges() to deal with all types of protected edges
inside the split loop
@janetournois

Copy link
Copy Markdown
Member Author

/build:v0

@github-actions

Copy link
Copy Markdown

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

@MaelRL MaelRL added Not yet approved The feature or pull-request has not yet been approved. Small feature CHANGES.md not updated labels May 18, 2026
@MaelRL MaelRL added this to the 6.3-beta milestone May 18, 2026
@sloriot

sloriot commented Jul 15, 2026

Copy link
Copy Markdown
Member

Successfully tested in CGAL-6.3-Ic-31

.edge_is_constrained_map(eif)
.number_of_iterations(3));
std::cout << "Remeshing done." << std::endl;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice to at least count the number of protected edges before and after remeshing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CHANGES.md not updated Not yet approved The feature or pull-request has not yet been approved. Pkg::PMP Small feature Tested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants