0029807: [Regression to 7.0.0] Impossible to cut cone from prism
authornbv <nbv@opencascade.com>
Fri, 25 May 2018 08:05:58 +0000 (11:05 +0300)
committerbugmaster <bugmaster@opencascade.com>
Fri, 6 Jul 2018 12:52:48 +0000 (15:52 +0300)
commit3306fdd954ad583d2652f0454612b6183176eb2e
tree2c2504b342baefbe20dbcc584e16fe32bdbedf09
parentd9d3107d8df19cacc3d7a0cd46ba51deee0bf6f2
0029807: [Regression to 7.0.0] Impossible to cut cone from prism

The algorithm has been improved for the cases when the intersection line goes through the cone apex.

<!break>

1. All special points are put to the ALine forcefully (if they are true intersection point). Currently this step has not been implemented yet.

2. Now the tolerance of IntPatch_Point (put into ALine) is computed in order to cover the distance between it and the correspond ALine.

3. Test cases have been created.

4. Procedure of trimming IntAna_Curve has been improved.

5. Criterion when the discriminant of IntAna_Curve can be considered to be equal to 0 has been improved.

6. Methods IntAna_Curve::FindParameter(...) (and IntPatch_ALine::FindParameter(...)) currently returns list of all parameters corresponding the given point (IntAna_Curve can be self-interfered curve). Before the fix, this method always returned only one (randomly chosen) parameter.

7. Interfaces of the following methods have been changed: IntAna_Curve::FindParameter(...), IntPatch_ALine::FindParameter(...), IntPatch_ALine::ChangeVertex(...), IntPatch_SpecialPoints::AddPointOnUorVIso(...), IntPatch_SpecialPoints::AddSingularPole(...), IntPatch_WLineTool::ExtendTwoWLines().

8. Following methods have been added: IntAna_Quadric::SpecialPoints(...), IntPatch_ALineToWLine::GetSectionRadius(...), IntPatch_SpecialPoints::ProcessSphere(...), IntPatch_SpecialPoints::ProcessCone(...), IntPatch_SpecialPoints::GetTangentToIntLineForCone(...).

------------------
1) tests/boolean/volumemaker/C5
   tests/boolean/volumemaker/C6
   tests/boolean/volumemaker/E7

They are real IMPROVEMENTS. In the FIX (in compare with MASTER), section result between pairs of faces f2&f6 (C5), f3&f7 (C6) and f1&f5 (E7) is closed. Separated test cases have been created in order to focus on the problem with section. Bug #28503 has been fixed.

Correction in test cases.
81 files changed:
src/BRepTest/BRepTest_CheckCommands.cxx
src/Geom/Geom_ConicalSurface.hxx
src/IntAna/IntAna_Curve.cxx
src/IntAna/IntAna_Curve.hxx
src/IntAna/IntAna_IntQuadQuad.cxx
src/IntAna/IntAna_Quadric.cxx
src/IntAna/IntAna_Quadric.hxx
src/IntPatch/IntPatch_ALine.hxx
src/IntPatch/IntPatch_ALine.lxx
src/IntPatch/IntPatch_ALineToWLine.cxx
src/IntPatch/IntPatch_ALineToWLine.hxx
src/IntPatch/IntPatch_ImpImpIntersection_0.gxx
src/IntPatch/IntPatch_ImpImpIntersection_2.gxx
src/IntPatch/IntPatch_ImpImpIntersection_4.gxx
src/IntPatch/IntPatch_ImpPrmIntersection.cxx
src/IntPatch/IntPatch_Intersection.cxx
src/IntPatch/IntPatch_Point.cxx
src/IntPatch/IntPatch_Point.hxx
src/IntPatch/IntPatch_PointLine.cxx
src/IntPatch/IntPatch_SpecialPoints.cxx
src/IntPatch/IntPatch_SpecialPoints.hxx
src/IntPatch/IntPatch_WLineTool.cxx
src/IntPatch/IntPatch_WLineTool.hxx
src/IntStart/IntStart_SearchOnBoundaries.gxx
src/QABugs/QABugs_19.cxx
src/QABugs/QABugs_20.cxx
src/gp/gp_Cone.hxx
tests/boolean/volumemaker/C5
tests/boolean/volumemaker/C6
tests/boolean/volumemaker/E7
tests/bugs/fclasses/bug23972 [deleted file]
tests/bugs/modalg_2/bug20964_1
tests/bugs/modalg_2/bug20964_2
tests/bugs/modalg_2/bug20964_3
tests/bugs/modalg_2/bug20964_4
tests/bugs/modalg_2/bug20964_5
tests/bugs/modalg_6/bug27269
tests/bugs/modalg_6/bug27282_2
tests/bugs/modalg_6/bug28626_1
tests/bugs/modalg_6/bug28626_2
tests/bugs/modalg_6/bug28626_3
tests/bugs/modalg_7/bug23176
tests/bugs/modalg_7/bug23972 [new file with mode: 0644]
tests/bugs/modalg_7/bug25542 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b1 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b2 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b3a [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b3b [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b4a [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b4b [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b5a [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_b5b [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1001 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1002 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1003 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1004 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1005 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i1006 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2001 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2002 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2003 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2004 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2005 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i2006 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i3001 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i3002 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i3003 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i3004 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i3005 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i4001 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i4002 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i4003 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i4004 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i4005 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i5001 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_i5002 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_sc01 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_svm01 [new file with mode: 0644]
tests/bugs/modalg_7/bug29807_svm02 [new file with mode: 0644]
tests/bugs/modalg_7/bug29824 [new file with mode: 0644]
tests/bugs/moddata_2/bug565