Correct angular tolerance in BRepOffset_Analyse and change DefineConnectType function accordingly the newest version

Request from SENR "SENR-21-039 "BRepExtrema_DistShapeShape - bad result "

Extrema_ExtSS.cxx - numbers of sample points are increased

IntPatch_PrmPrmIntersection.cxx

This reverts commit 6a2b9ef69fa79a7fb22140d4a8a9e34a7651b3f0.

* 0031140: Draw command "cirtang" raises exception on the parallel curves

To avoid exception for parallel curves because the number of solutions exceeds the size of the reserved array a check has been added that the number of solutions does not exceed the size of the reserved array.

0030591: Exception is raised by command "bopcheck" on the attached shape

Protection to avoid exception if number of vertices is equal to 1 was added.

Commands.tcl for using checkprops in the tests

0030708: Modeling Data - Exception is raised while initializing TopoDS_Iterator with null shape

Added protection from null shapes to TopoDS_Iterator::Initialize method.

0030591: Exception is raised by command "bopcheck" on the attached shape

Protection to avoid exception if number of vertices is equal to 1 was added.

0030522: Modeling Algorithms - BRepBuilderAPI_MakeWire produces different wires depending on the order of parameters

The following improvement has been implemented in BRepLib_MakeWire class to make it more reliable:
1. Quit adding edges of a wire into result if the current edges have not been added. Return NotDone status.

Test case for the issue.

0026717: Error in IntPatch_PrmPrmIntersection: change of local resolution leads to break of walking line

Test case for issue CR26717

Correction of test cases for issue CR26717

Test case for issue CR26717

0030490: Modeling Algorithms - Incomplete result of Cut operation

Unify the vertices of the small section edges (part of the commit for #0027448).

Test case for the issue.

0029887: Wrong result of CUT operation due to incorrect point-face classification

Using of input tolerance has been eliminated in Geom2dInt_GInter algorithm called from BRepClass_Intersector::Perform(...) method. Now, the input tolerance is used only for check ON-status of classification.

(cherry picked from commit 748c9dc6ae0d6018018fca585b6d8cf511c0dd0b)

0029535: Modeling Algorithms - BRepExtrema_DistShapeShape returns only one solution but the wires have two intersections (for 6.9.1)

Update test system for OCCT 6.9.1

Update testing system for compatibly test case with master during prepare delivery

0029387: Incorrect result of cut a face with several shapes

Eliminate normalization of the section curve while making approximation.
Remove section edges having no valid range.
Test case for the issue.

0029162: Geom2dInt_GInter algorithm does not find intersection of ellipse and line

0029159: Sewing fail when using a maximum tolerance

Force setting truly computed tolerance of the edge if BRepLib::SameParameter has put too large tolerance.

0029102: Missing points using GCPnts_QuasiUniformDeflection

Correct the method PerformCurve of GCPnts_QuasiUniformDeflection to get the proper derivative at the end of the current curve.

Improve the Draw commands "crvpoints" and "crvtpoints" so that to work with wires as composite curves.

0027441: The method IntTools_Context::IsVertexOnLine incorrectly computes parameter of the point on the curve

Choosing the closest bound to the checking point.

Test cases for the issue.

Updating test cases for the current behavior.

Fix for the issue #28849.

0028830: HalfSpace command chooses the wrong side of the given shell

Improve the algorithm BRepPrimAPI_MakeHalfSpace. Earlier it made projection of the point only on faces. If the nearest point does not conform to normal projection criterion the result is wrong. The fix includes search of projection on edges and vertices. This makes the algorithm robust for half spaces with boundaries.

Commit for #28771 (thickshell produces wrong result, SENR-17-060)

Fix for case 180: Error using SnrSplitAlgorithm (691)
Protection to avoid creation of the overlapping edges

0027761: Intersection loops infinitely

The static method CorrectSurfaceBoundaries() in the class IntTools_FaceFace,
which is used for correction of the boundaries of the faces before their intersection,
contains two inconsistent adjustments of the boundaries of the faces based on periodic surfaces.
One of these adjustments has been removed. The remaining adjustment, the one that is based on the
2D bounding boxes of the closed curves, should be sufficient.

Test case for the issue.

(cherry picked from commit 7cb2f40993b9699ca7dc80b5f3c54662d9e345dd)

0028222: Intersection of two cylinders fails

1. The reason of exception has been eliminated.

2. Algorithm in IntPatch_WLineTool::JoinWLines(...) method has been modified in order to forbid join curves in the point where more than two intersection lines meet. More over, joining is forbidden if local curvature in the connection point is too big (see function CheckArgumentsToJoin(...) in the file IntPatch_WLineTool.cxx).

3. Interface of IntPatch_WLineTool::JoinWLines(...) method has been modified in order to reduce number of arguments.

4. Small corrections in IsSeamOrBound(...) static function has been made. Namely, check has been added if two boundaries are in the same period region but are too far each to other (see IntPatch_WLineTool.cxx, IsSeamOrBound(...) function, line # 532).

5. "Reversed" flag has been made local. Now, it is pure local characteristic: the algorithm decides itself, shall we reverse the argument order. This correction makes the algorithm more commutative (see issue #25404). However, IntPatch_WLineTool::JoinWLines(...) method can return non-commutative result.

6. Algorithm of searching small intersection curves has been improved.

7. New methods have been added in Bnd_Range class.

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

Some test cases have been adjusted according to their new behavior.

1. tests\bugs\modalg_6\bug26310_3
   tests\bugs\modalg_6\bug26310_4
   tests\bugs\moddata_2\bug235
   tests\perf\modalg\bug26310_1
   tests\bugs\modalg_5\bug24915

Logic of these cases has been changed. Mover over, additional check has been added in "bug26310_1" test case. Therefore, its performance will be slower than on the current MASTER.

2. tests\bugs\modalg_5\bug25292*

Scripts have been rewritten in order to make it more readable. Logic of these cases has not been changed.

0028009: [Regression vs OCCT7.0.0] Cut produces not valid shape

The algorithm in WorkWithBoundaries::BoundaryEstimation(...) did not take into account opposite directions of cylindrical axes (when the angle between them is obtuse). After the fix it does it.

Small correction of test cases for issue CR28009

(cherry picked from commit c9c7286e56612bbb845e63abcaa9b369faf55e33)

0027856: Regression vs 6.7.1: General Fuse operation fails to fuse the solids

Method WorkWithBoundaries::BoundaryEstimation(...) has been brought in balance with IsParallel(...) method, which checks if cylinder axes are parallel.

(cherry picked from commit 95f8c6082dae20e53c196418ed9790fe78757109)

Patch including (particular) fixes for the issues #23178, #26586, #27766, #26852 and #24023.

0027310: Huge tolerance obtained in the result of intersection of two cylindrical faces

Sometimes start point of the intersection line is in the surface boundary strictly. I.e. the parameter of this point in the surface can be equal to both 0 or 2*PI equivalently. It is important to chose correct parameter value.

The algorithm of prediction is based on monotonicity property (see CylCylMonotonicity(...) function in IntPatch_ImpImpIntersection_4.gxx). Now, this function is used wrongly. The fix improves this situation.

Small optimization in the code.
Creation of test cases .

The logic of returning value by the method BoundariesComputing() has been corrected.

(cherry picked from commit 79997052f158c0512ff4d10ecb896e4899d24077)

0026884: Cylinder/Cylinder intersection algorithm throws an exception

Check, if cylinder axes are parallel is made more carefully.
This check is brought to conformity with check if axes are parallel (see constructor AxeOperator::AxeOperator(...) in IntAna_QuadQuadGeo.cxx file)

Test case for issue CR26884

(cherry picked from commit ce48b00930ae801233d44228e5e1f5a193a5de82)

0028718: Section result has a break

Fix of the problem.

0028683: Wrong result of CUT operation

1. Partial porting of the patches for the issues #0026738 and #0028017:
   Careful treatment of the SD vertices created on the Face/Face intersection stage.

2. Avoid creation of the small section edges.

3. Test case for the issue.

4. Adjusting test cases for current behavior.

0028585: Wrong value of area computation

Another processing of NaturalRestriction flag for face in algorithm of BRepGProp* classes.

Fix for the issue #28491

0028189: Result of Boolean operation is non-manifold wire

1. The result of Boolean operation on the arguments of collection type, containers WIRE/SHELL/COMPSOLID, is now also a collection.
The result of Boolean operation on arguments of non-collection types is now a compound containing the splits of arguments directly, without its enclosure into containers.
Thus, if there were no containers, such as WIRE/SHELL/COMPSOLID, in the input arguments there will be no such containers in the result of BOP.

2. The containers of type WIRE included into result should now also (as the SHELLs) have coherent orientation of its sub-shapes.
For that the new method has been implemented (BOPTools_AlgoTools::OrientEdgesOnWire(TopoDS_Shape&)) which reorients edges of the given shape for correct ordering.

3. The duplicating containers, i.e. containers with the contents completely included in other containers, are now avoided in the result of BOP.

4. Documentation has been updated.

5. New test cases for the issue.

6. Adjusting test cases to current behavior.

Correction for redmine issue #150 - make wire builder more stable.

0027746: Incomplete result of SECTION operation

1. Partial porting of the fix for the issue #0026560:
Function to compute subshape max tolerance has been added.
Fixed bounding box expanding at Face/Face step of boolean operation.

2. New test cases are created.

0027677: Incorrect CUT of a solid by semi-infinite solid

The problem of incorrect CUT is wrong Pcurve after projection onto cylinder of a small edge orthogonal to cylinder's axis. The result of projection is a line along V direction, but it should be along U.

ProjLib_Cylinder has no specific functionality to build projection in most common case, so it should return "not done" flag to start projection based on curve approximation.

Following changes are done:
1. ProjLib_Cylinder returns isDone=false in the following cases:
  * the projected line is not parallel to cylinder's axis
  * the plane of projected circle is not orthogonal to cylinder's axis
2. Test case added

Fix for issue #27664

0027371: Regression: BRepExtrema works too much slower in 691 (from 670)

Lipschitz constant tuning.
Shubert estimation for minimal value is added.

Test case is added.

0027151: Exception is raised during performing command "splitshape" in the Test Harness

Tolerance of vertices's of the degenerated edges are increased
to value of tolerance of the splitting vertex for case when splitting vertex was projected on the degenerated edge.

0026619: Tolerances of operands are modified using bop

The fix forces creation of new sub-shapes (vertex, edge) when the tolerance of some sub-shape of an argument is to be increased.

    This new behavior is turned off by default. It can be turned on using two ways:
    1) Setting 'locking' flag of the arguments.
    2) Calling the method SetNonDestructive(Standard_True) of the API classes.

0027151: Exception is raised during performing command "splitshape" in the Test Harness

Modifications to avoid exception for case overlapped edges was made.
Returned values of the methods LocOpe_SplitShape:AddOpenWire() and LocOpe_SplitShape:AddClosedWire() were modified from void to boolean in order to avoid raising of the exceptions for cases when result wire can not be built.

Old signature of method "reset" is restored to avoid problems with backward compatibility.

0027162: Draw command "(2d)extrema" incorrectly represent underlying algorithm results

Correct handling of infinity solutions added for Curve / Curve case.
Unused code deleted.
Test cases updated to the new behavior.

Possibility to freeze Lipchitz constant is added to improve performance.

0027131: [Regression to 6.7] DistShapeShape performance loss

Lipchitz constant approximation and fixes in global optimization algorithm added to improve performance.
Test case added.

Fix backporting:

0026593: Coding rules - revert compatibility of NCollection_CellFilter constructor with old code

Restored old constructor and old behavior where possible.

Minor correction.

0026395: Merge clasees NCollection_CellFilter_NDim and NCollection_CellFilter

Deleted exceed class CellFilterNDim.
Now dimension count used as input parameter in NCollection_CellFilter.

minor corrections.

0027119: Regression: Draw command "incmesh" hangs on the attacheced face.

BRepMesh_CircleTool: extend radius of circle by quite small value in order to classify points forming inscribed triangle as lying on it. Do not use PConfusion due to false positive result leading algorithm to hanging.

0027114: [Regression to 6.7] DistShapeShape does not find a solution edge-face

Extrema Curve / Surface algorithm changed to perform more accurate search.

test case bug25232_8 - improvement, one additional intersection point is detected now.
test case bug23830 - normal behavior, position of extrema is changed.
Small correction of pro19653 test
test case for original issue added.

Small correction of test case bugs/heal/bug26642 due to shape renaming

0027110: Regression: Draw command "parameters" can not compute parameter on the line.

Useless variable MAXTOLERANCEGEOM is removed in order to allow to place point at any distance from curve/surface.
Algorithm is simplified: particular calculations for analytical geometry are removed, only using of Extrema_... is kept.

Patch for issue #26841

Small correction of qt samples

0026723: Qt IE sample can not import STEP and IGES files with names containing not ascii symbols

Added conversion of the names of the imported files in the  Utf8 coding

Fix samples crash.

Changing OCCT version from 6.9.1 beta to 6.9.1

0026707: Remove references to "opencascade.org" in documentation

Links to OCC web sites in documentation are updated in accordance with the new site design.

0026632: HLR sample doesn't work with Algo mode

The main reason of the problem is Walking-line contains two equal Walking-points (the segment has null-length both in 3D and 2D).

0026701: BOPAlgo_Builder::Perform crash.

Data race eliminated (PCurves are stored in edges after paralleled calculation)

Add comments and make minor cosmetic changes.

0026522: Exception while copying offset on C0 surface

0026671: Infinite loop in ShapeFix_Wire::FixSelfIntersection()

Avoid possible infinite loops in ShapeFix_Wire::FixSelfIntersection() -- when needed, fix is repeated once instead of going back in the loop

Modification of service files for Voxel demo

0026705: Visualization - OpenGl_Workspace aspect setters are not exported

Update samples for VC++ 2013

0026532: Meshing of edge with minSize parameter leads to incorrect result

Fix missed parameter.

Test case for issue CR26532

0026687: SIGSEGV in BRepBuilderAPI_MakeFace

1. Check face location has been added.
2. Message, if mkplane command fails, has been added.
3. Test case for this issue has been created.

Changes in accordance with the remark.

Small correction of test case for issue CR26687

0026688: Visualization, TKOpenGl - apply transparency within GLSL programs

OpenGl_ShaderManager::PushAspectFace(), setup alpha to Diffuse() component as in FFP.
OpenGl_ShaderManager::stdComputeLighting(), take alpha from material Diffuse component.

0026533: Draw command vdisplay requires vclear to update the presentation since OCC 6.9.0.
Update documentation related to vdisplay command and similar.

0026196: Wrong result obtained by projection algorithm.

Improved periodicity handling when trimmed parameters became unclosed or nonperiodic, but underlying geometry closed or periodic.
Added possibility to not perform trim in GeomAdaptor::MakeSurface.

Minor corrections.

Test-case for issue #26196

0026683: VRML 2.0: ImageTexture node not written

0026616: Memory leak in IntTools_Context::Hatcher

I. New features:
    No new features

II.2. class IntTools_Context
- method:
Geom2dHatch_Hatcher& IntTools_Context::Hatcher
  (const TopoDS_Face& aF)
- method:
IntTools_SurfaceRangeLocalizeData& IntTools_Context::SurfaceData
  (const TopoDS_Face& aF)

The memory allocation is provided using inner allocator of the object.

II. Modified entities:
packages:
IntTools

0026691: CMake building procedure should support compiler of version 14.0 of Microsoft Visual C

"vc14" compiler name added to CMake building script

0026642: ShapeUpgrade_UnifySameDomain introduces extremely high vertex tolerances

Algorithm of computation of safe shift value along 2D-line (in order to distance between two points was less than tolerance) has been improved.

Correction of some test cases in accordance with their new behavior.

0026419: [Regression?] Export of a reversed face leads to crash in 6.9.0 although proceeded in 6.8.0

Take basis surface from trimmed surfaces during IGES writing, because pcurves will be transformed, so trim will be shifted, accorded to new face bounds,
fix updating of seam edge,
change condition of using Segment during exporting periodic BSpline surfaces to IGES (if bounds are not in one period).

Test cases for issue CR26419

0026573: IGES file with one entity 128 is not read

Add some workaround to read IGES files with lines, which lengths are less than standard 80.

Test case for issue CR26573

Correction of test cases for issue CR26573

Correction of test case for issue CR26573

0026651: IntTools_FClass2d gives incorrect result of classification

0026588: SIGSEGV in BRepFeat_MakeDPrism::Perform()

Incorrect piece of code is removed (not used)

0024097: BRepAlgoAPI_Section returns wrong result

Test case added: bugs modalg_6 bug24097

0026099: Wrong result done by 2d intersection algorithm

Detection of intersection for almost parallel lines has been improved.

0026567: Exception in Boolean intersection command

Correcting the usage of NCollection_IncAllocator

Adjusting test cases for current state of OCCT

0025553: ShapeFix_Face::FixMissingSeam() fails to correct a face

Modification to support case when edges on face  have incorrect orientation was added.
Modification to support case when wires have intersecting segment was added

Test cases for issue 25553 added; other affected tests corrected (improvements)

Corrected test case

0026575: Normal projection of an edge to a face is wrong

0026628: Button Erase (Delete where erase algorythm is used) works incorrect

Update AIS_InteractiveContext::EraseSelected() method.

Fix HLR sample

Fix samples.

0026153: Draw Harness, ViewerTest - "verase" does not remove selection highlight for shaded object

0026665: Visualization, TKOpenGl - specify precision for integer uniforms explicitly within OpenGL ES

Use lowp for enumerations / Boolean uniforms.
Define default integer precision to highp in Fragment Shader when available.

0026672: Visualization, TKOpenGl - workaround mobile hardware with extremely low precision of mod() in GLSL

0026638: Help of vcaps Draw command mentions wrong argument

0026636: BRepOffsetAPI_ThruSections algorithm crashes on two inconsistent wires

Test case for issue CR26636

0026313: Method BRepBuilderAPI_GTransform::ModifiedShape(...) thows exception

Test cases for issue CR26313

0026173: Wrong result of ChFi3d_ChBuilder algorithm: incorrect processing of G1 junctions at vertex

Test case for issue CR26173

0026650: Coding rules - fix misprint in NCollection_Vec3::operator/()

0026635: UnifySameDomain loses internal edges

Treat internal edges in proper way to save them in the result.

The test "bugs heal bug26489_4" has been modified to reflect the actual behavior.

Test-case for issue #26635

0026498: BRepOffsetAPI_MakeOffset causes segmentation fault

Test case for issue CR26498

0026627: [Regression] Shape Healing hangs as of OCC 6.8.0

Check for orientation of the solid corrected to ensure that cycle always finishes.

Test case added: bugs modalg_6 bug26627
Tests boolean volumemaker A3, B5, B7 corrected (improvements)

0026647: BRepTools::UVBounds() computes zero range by V

Calculation of bounding box on conic 2d curves corrected to work properly if specified parametric range is in negative values.

Test case added

0026582: Wrong result obtained by Common operator

class BOPAlgo_WireSplitter
   - static function:
Standard_Real Angle2D (const TopoDS_Vertex& aV,
                         const TopoDS_Edge& anEdge,
                         const TopoDS_Face& myFace,
                         const GeomAdaptor_Surface& aGAS,
                         const Standard_Boolean bIsIN)
The treatment of circles has been chenged to prevent
the loss of accuracy due to small differences in large values.

Small correction of test case for issue CR26582

Test case for issue CR26582

Conflicts:
src/BOPAlgo/BOPAlgo_WireSplitter_1.cxx

0026644: ShapeUpgrade_UnifySameDomain - strange behavior

Modification to keep connectivity for non--manifold shapes during operation ShapeUpgrade_UnifySameDomain

0026637: Save function works incorrect in OCAF mfc sample

Storage format was corrected.

0026462: Visualization - selection does not adapt to line width change

- added interface for changing sensitivity of a particular selection through both local and interactive context;
- added corresponding methods for changing sensitivity to SelectMgr_SelectionManager, SelectMgr_Selection and SelectBasics_SensitiveEntity;
- option -setSensitivity was implemented in vaspects Draw command;
- test case for issue #26462