Archived issue #0026673
BOPTools_AlgoTools::ComputeVV incorrectly checks two vertices for intersection
Description
This method uses the condition:
D(P(V1), P(V2)) <= Tol(V1)+Tol(V2)
But must use the following:
D(P(V1), P(V2)) <= Tol(V1)+Tol(V2)+Precision::Confusion()
So, if we create a box with height equal to 2e-7+1e-14 the self-intersection checker must report an error. But now it does not report such error.
D(P(V1), P(V2)) <= Tol(V1)+Tol(V2)
But must use the following:
D(P(V1), P(V2)) <= Tol(V1)+Tol(V2)+Precision::Confusion()
So, if we create a box with height equal to 2e-7+1e-14 the self-intersection checker must report an error. But now it does not report such error.
Steps to reproduce
box b -1 -1 0 2 2 2e-7+1e-14
bopargcheck b /RFPCS #F
Output:
Shape(s) seem(s) to be valid for BOP.
Must have the line:
Self-Intersections : YES
bopargcheck b /RFPCS #F
Output:
Shape(s) seem(s) to be valid for BOP.
Must have the line:
Self-Intersections : YES
Public activity
5 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Fri Oct 28 18:20:03 2016 +0300
0026673: BOPTools_AlgoTools::ComputeVV incorrectly checks two vertices for intersection
Test case has been created.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Fri Oct 28 18:20:03 2016 +0300
0026673: BOPTools_AlgoTools::ComputeVV incorrectly checks two vertices for intersection
Test case has been created.
This bug is not reproduced in current master.
I have created test case in the branch CR26673.
I have created test case in the branch CR26673.
Please integrate the test case and close the bug.
Test case has been verified
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]