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

fix simplex traverser for CGAL 5.6 #7535

Conversation

lrineau
Copy link
Member

@lrineau lrineau commented Jun 20, 2023

equivalent of #7423, integration branch and PR for 5.6.x-branch, master, and integration.

lrineau and others added 30 commits May 4, 2023 14:58
For the moment, that new test fails.
- call the API with two vertex handles, when possible,
- add a check that the reported simplices intersect the query segment,
- add a chech that two consecutive simplices are incident.
Protected by the macro CGAL_DEBUG_TRIANGULATION_SEGMENT_TRAVERSER_3
with tests on cell_iterator == end()
and then avoid all std std::get<i>(simplex), hard to read
* test_segment_simplex_traverser_3 first failing test is now "032"
* the geometric intersection test fails because of doubles
I have been working on that issue for a few days. Eventually I found
the bug.
The test-case was wrong.
`-.1`, and `.8` are not floating point values.

I have used `.125` (1/8) and `.5` instead.
@lrineau lrineau added this to the 5.6-beta2 milestone Jun 20, 2023
@lrineau lrineau changed the base branch from master to 5.6.x-branch June 20, 2023 16:17
lrineau added 2 commits June 21, 2023 10:19
Jane and I thought that case

  vertex-vertex with `ch_next != Cell_handle()`

was not possible. Let's patch that.
sloriot added a commit to sloriot/cgal that referenced this pull request Jun 22, 2023
…_traverser-GF-CGAL-5.6

fix simplex traverser for CGAL 5.6
@sloriot sloriot added Batch_1 First Batch of PRs under testing Under Testing and removed Under Testing labels Jun 28, 2023
@sloriot
Copy link
Member

sloriot commented Jul 5, 2023

Successfully tested in CGAL-6.0-Ic-15

@sloriot sloriot added Tested and removed Ready to be tested Under Testing Batch_1 First Batch of PRs under testing labels Jul 5, 2023
@lrineau lrineau added rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch labels Jul 5, 2023
@lrineau lrineau merged commit 27b1e99 into CGAL:5.6.x-branch Jul 5, 2023
lrineau added a commit that referenced this pull request Jul 5, 2023
…verser-GF-CGAL-5.6

fix simplex traverser for CGAL 5.6
@lrineau lrineau removed rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch labels Jul 5, 2023
@lrineau lrineau deleted the Triangulation_3-fix_simplex_traverser-GF-CGAL-5.6 branch July 5, 2023 14:46
@lrineau lrineau modified the milestones: 5.6-beta2, 5.6 Jul 20, 2023
@janetournois
Copy link
Member

This PR partially solves issue #7438

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.

3 participants