0026431: Can't cut a sphere from a cylinder
authornbv <nbv@opencascade.com>
Tue, 10 Nov 2015 07:31:19 +0000 (10:31 +0300)
committerbugmaster <bugmaster@opencascade.com>
Fri, 4 Dec 2015 10:15:04 +0000 (13:15 +0300)
commit4e14c88f77d5a79a6be8acf62e17afb62771831b
tree68a9ff05d407fd0ef603ed9021f9fe510270a395
parentf44aa19760788a447e843e2fff98231e94666d84
0026431: Can't cut a sphere from a cylinder

This branch contains fixes for 26675 and 26431 bugs.

1. Normalization has been eliminated.
2. Interfaces of AppDef_Compute::Parametrization(...) and BRepAlgo_BooleanOperations::SetApproxParameters() methods have been changed.
3. Overloaded methods for ApproxInt_Approx::SetParameters(...), TopOpeBRepTool_GeomTool::GetTolerances(...) and TopOpeBRepTool_GeomTool::SetTolerances(...) have been removed (because some fields of these classes are not used more).
4. Comments for some methods have been changed in BRepApprox_TheMultiLineOfApprox.hxx and GeomInt_TheMultiLineOfWLApprox.hxx files.
5. Some fields have been deleted from ApproxInt_MultiLine class. Kept members have become constant.
6. Interface of ksection DRAW-command has been changed.
7. Now, 2dintersect DRAW-command prints information about found segments.
8. Some code fragments have been rewritten to make them easier.
9. Algorithm of splitting WLine, which goes through pole of sphere has been improved.
10. Improve approximation algorithm in order to it will compute correct 2D- and 3D-tangent at the end of bezier constraints (including case when curve goes through or finishes on singular points).
11. Interface of IntPatch_WLine::Dump(...) method has been corrected.
12. Some methods for working with Walking-line are made more universal (available for both GeomInt and IntTools packages).
13. Problem in BRepLib::SameParameter(...) method has been fixed (see corresponding comment).
14. Small correction in Draft package.
15. Any outputs in IntPatch_Intersection::Dump(...) method have become disabled because they are useless. If anybody need in this outputs he/she will correct this method himself/herself.

Adjusting some test cases according to their new behavior.
Creation of new test cases.

----------------------------------------------------------------------------------------------------------------------------

Some explanation of new behavior of some test cases:

 1. Regressions:

a) blend simple X4
The problem is described in the issue #0026740. According to this description,  the result on the current MASTER seems to be wrong indeed.

b) boolean bcommon_complex C7 and boolean bcut_complex Q1
These test case use same shapes with different Boolean operation (COMMON and CUT). They are already BAD (on the MASTER). Now, some sub-shapes have become not-shared, simply. In my opinion, we shall apply new behavior of these tests.

c) boolean bsection M3
The problem described in the issue #0026777 exists even on the current MASTER.

d) boolean bsection M9
The problem is described in the message http://tracker.dev.opencascade.org/view.php?id=26815#c47546. Here, we have really regression in the picture.

e) boolean bsection N2

The problem is described in issue #0026814.

f) boolean volumemaker G1

The problem is described in issue #26020.

g) bugs modalg_1 bug1255 (and bug1255_1)

The problem is described in issue #26815.

h) bugs modalg_2 bug5805_18, bugs modalg_2 bug5805_42, bugs modalg_2 bug5805_46

The problem is described in issue #25925.

i) bugs modalg_3 bug602

The problem is describes in issue #602.

j) bugs modalg_5 bug24915

The problem is described in the message http://tracker.dev.opencascade.org/view.php?id=25929#c48565. It is not fixed by this issue.

k) bugs modalg_5 bug25838

The main reason is described in issue #0026816.

----------------------------------------------------------------------------
2. Improvements:

a) boolean volumemaker F9
b) bugs modalg_1 bug10160_3
c) bugs modalg_2 bug22557
d) bugs modalg_5 bug25319_1 (_2)
e) draft angle G2
f) offset shape A1
g) offset with_intersect_80 N7
81 files changed:
src/AppDef/AppDef_Compute.hxx
src/AppParCurves/AppParCurves_LeastSquare.gxx
src/Approx/Approx_ComputeLine.gxx
src/ApproxInt/ApproxInt_Approx.gxx
src/ApproxInt/ApproxInt_ImpPrmSvSurfaces.gxx
src/ApproxInt/ApproxInt_KnotTools.cxx
src/ApproxInt/ApproxInt_MultiLine.gxx
src/BRepAlgo/BRepAlgo_BooleanOperations.cxx
src/BRepAlgo/BRepAlgo_BooleanOperations.hxx
src/BRepApprox/BRepApprox_Approx.hxx
src/BRepApprox/BRepApprox_TheComputeLineBezierOfApprox.hxx
src/BRepApprox/BRepApprox_TheMultiLineOfApprox.hxx
src/BRepLib/BRepLib.cxx
src/BSplCLib/BSplCLib_2.cxx
src/ChFi3d/ChFi3d_Builder_0.cxx
src/Draft/Draft_Modification_1.cxx
src/GeomInt/GeomInt_IntSS.hxx
src/GeomInt/GeomInt_IntSS.lxx
src/GeomInt/GeomInt_IntSS_1.cxx
src/GeomInt/GeomInt_LineTool.cxx
src/GeomInt/GeomInt_LineTool.hxx
src/GeomInt/GeomInt_TheComputeLineBezierOfWLApprox.hxx
src/GeomInt/GeomInt_TheMultiLineOfWLApprox.hxx
src/GeomInt/GeomInt_WLApprox.hxx
src/GeomliteTest/GeomliteTest_API2dCommands.cxx
src/HLRTopoBRep/HLRTopoBRep_DSFiller.cxx
src/IntPatch/FILES
src/IntPatch/IntPatch_ImpPrmIntersection.cxx
src/IntPatch/IntPatch_Intersection.cxx
src/IntPatch/IntPatch_Intersection.hxx
src/IntPatch/IntPatch_WLineTool.cxx [new file with mode: 0644]
src/IntPatch/IntPatch_WLineTool.hxx [new file with mode: 0644]
src/IntTools/IntTools_Context.cxx
src/IntTools/IntTools_FaceFace.cxx
src/IntTools/IntTools_WLineTool.cxx
src/IntTools/IntTools_WLineTool.hxx
src/ProjLib/ProjLib_ComputeApprox.cxx
src/QABugs/FILES
src/QABugs/QABugs.cxx
src/QABugs/QABugs.hxx
src/QABugs/QABugs_20.cxx [new file with mode: 0644]
src/QABugs/QABugs_3.cxx
src/ShapeAlgo/ShapeAlgo_AlgoContainer.cxx
src/TopOpeBRep/TopOpeBRep_FacesIntersector.cxx
src/TopOpeBRepTool/TopOpeBRepTool_CurveTool.cxx
src/TopOpeBRepTool/TopOpeBRepTool_GeomTool.cxx
src/TopOpeBRepTool/TopOpeBRepTool_GeomTool.hxx
tests/blend/buildevol/B3
tests/blend/simple/G6
tests/blend/simple/X4
tests/boolean/bcommon_complex/C7
tests/boolean/bsection/M3
tests/boolean/bsection/N2
tests/boolean/volumemaker/C5
tests/boolean/volumemaker/C6
tests/boolean/volumemaker/F9
tests/boolean/volumemaker/G1
tests/bugs/modalg_1/buc60533
tests/bugs/modalg_1/buc60555_1
tests/bugs/modalg_1/buc60555_2
tests/bugs/modalg_1/buc60555_3
tests/bugs/modalg_1/bug10160_3
tests/bugs/modalg_1/bug1255
tests/bugs/modalg_1/bug1255_1
tests/bugs/modalg_2/bug22557
tests/bugs/modalg_2/bug5805_18
tests/bugs/modalg_2/bug5805_42
tests/bugs/modalg_2/bug5805_46
tests/bugs/modalg_3/bug602
tests/bugs/modalg_5/bug24585_1
tests/bugs/modalg_5/bug24915
tests/bugs/modalg_5/bug25319_1
tests/bugs/modalg_5/bug25319_2
tests/bugs/modalg_5/bug25838
tests/bugs/modalg_6/bug26431_1 [new file with mode: 0644]
tests/bugs/modalg_6/bug26431_2 [new file with mode: 0644]
tests/bugs/modalg_6/bug26431_3 [new file with mode: 0644]
tests/bugs/modalg_6/bug26675 [new file with mode: 0644]
tests/draft/angle/G2
tests/offset/shape/A1
tests/offset/with_intersect_80/L6