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

0026195: Visualization - optimize selection algorithms

- initial transformation of triangulation is now applied to selecting frustum;
- switched from NCollection_Vec3 to gp collections to avoid conversions and usage of macros;
- calculation of frustum was refactored to reduce its build time;
- double pixel tolerances for selection were replaced by integer ones;
- switched to splitting along the main axis only in SelectMgr BVH selection primitive sets.

0026451: Crash importing STeP file

Add check for NULL.

Test case for issue CR26451

0026075: Make Extrema_GenExtCC return IsParallel flag in case of parallel curves

1) Added check for parallel curves.
2) Changed unefficient o(n^2) duplicates deleting algorithm to o(n) algorithm.
3) Deleted useless upper level duplicates deleting algorithm.

Test-case for issue #26075

0026251: Section curve cannot be found

Current algorithm ignores found triangles intersection (IntPolyh_PMaillageAffinage) and, consequently, cannot find start point for intersection line. Branch fixes this problem.

Test case for issue CR26251

Change version from "6.9.1.dev" to "6.9.1.beta"

0026245: Unable to write SAT file by default

XSAlgo_AlgoContainer is corrected to return original shape if Shape Processing fails.
Handling of situation of absence of defined operator sequence is corrected to apply DirectFaces operator by default on writing, regardless of target format.
ShapeProcess is corrected to output warning if operator sequence is not defined in resource file, and to continue if one operator in sequence has failed.
Message names are made more meaningful.

0026019: Unstable behavior of test cases with operation mkvolume

The maps have been replaced with indexed maps to provide stability
of the algorithms based on the General Fuse algorithm.

0026611: Visualization, TKService - fix NULL-dereference in Font_FontMgr on broken font

Check FT_Face::family_name for NULL during detection.
Skip fonts without mandatory UNICODE charset in Font_FontMgr.

0026613: Coding - avoid use of macros in Resource_Manager.cxx

Replace macros by enum Resource_KindOfLine.

0026310: Very slow boolean cut operations on cylinders

1. JoinWLines algorithm has been improved.
2. Reference to the V-boundaries is deleted when computing step.
3. Decreasing the tolerance when computing parameters of WLine.
4. Adding boundary point is forbidden if it lies in prolongation of found ones.
5. Possible reason of exception has been eliminated.
6. Processing of critical point has been improved.

Test cases for this issue have been created.
Correction of some test case(s).

Test case bugs/modalg_6/bug26310_4 is added

Update of test-cases for issue #26310

0026241: Sewing algorithm computes tolerance of joint vertex too rough

Upgrade method of computing vertex tolerance.
Update function, which used the old method.
Test case for issue CR26241
Delete obsolete variables.

Revert previously made adjustment of test cases for Linux

- boolean volumemaker C1: the problems disappeared on Windows are still present on Linux
- boolean volumemaker F8: improvement on Linux found on master due to #24682 does not happen in OCCT 6.9.1, since #24682 is not included

0026605: Possible array out of bounds read in Extrema_GExtPC.gxx

Degenerated case fixed.

0026376: Imported STEP shape is partially wrong

Modification of the translation of the face based on the toroidal surface with negative major radius was made.
Seems that such face is considered by "Solid Works" and "ProE wildfire 5.0" systems as face having reversed orientation.
This step file was written by "Solid Works 2014".

0026461: STEP Import interface crashes

Add a check for NULL;
some code refactoring.

0026574: Remove redundant dependence of TKIGES on TKOffset

Adjustment of test cases

- bugs modalg_6 bug26238: check real values with tolerance to avoid false failures due to numeric deviations
- draft angle G8: removed TODOs that do not exhibit in 6.9.1 branch
- boolean gdml_private O1: unstable

0026440: Invalid shape as a result of solid construction in BRepOffset_MakeOffset

Fixed degeneration in wall building in thicksolid mode.

0026556: Infinite calculations of BRepOffset_MakeOffset

Method for pre-analysis of input data added (CheckInputData).
Now it is possible to work with Error() function, which return error code.

Tests corrected according to changed error appearance.

0026458: BRepBuilderAPI_Copy does not copy mesh structure

The possibility to preserve triangulation in the copied shape is implemented.
It may be enabled by copyMesh flag, by default it is disabled.
Depending on copyGeom flag, the triangulation is shared with original shape (if False) or copied (if True).

Poly_Triangulation::Copy() method is added.

0026447: Performance degradation intersecting cylindrical surfaces

Added test case bugs/modalg_6/bug26447

0025981: Wrong result obtained by projection algorithm.

Adjust periodic added.

processing of V parameter periodicity added.

Test case for issue CR25981

0025820: No Intersection Curves between surface of revolution and planes

Thin shapes processing has been improved.

Test-cases for issue #25820

Comment has been changed

Add test case for #26446

0026289: STEP import/export produces an empty shape

Exception is raised during reading step file containing p-curve having not-unique last knot.
Modification correcting misprint during computation new numbers of poles for such case was made in the file StepToGeom_MakeBSplineCurve.pxx
Protection against exception was made in the file StepToTopoDS/StepToTopoDS_TranslateEdge.cxx.
Warning message was added in the file StepToTopoDS_TranslateEdgeLoop.cxx.

Test case for issue CR26289

0026553: Out of range exception in BRepFill_Pipe::FindEdge

Test case for issue CR26553
Warning fix

0026554: Error in IntPatch_PrmPrmIntersection: initial step for walking line is not related with actual tolerance

Test case for issue CR26554
Small corrections for issue CR26554

0025775: "Default Gradient Backgound" in V3d_Viewer has no effect in new created V3d_View.

Complete the fix: add gradient background in second variant of V3d_View::SetWindow()