3 years ago0028248: [Regression] HLR Algo result is retrieved from the last added shape only
msv [Tue, 19 Dec 2017 13:01:06 +0000 (16:01 +0300)]
0028248: [Regression] HLR Algo result is retrieved from the last added shape only

Mistakes in retrieving of HLRBRep_EdgeData by value instead of reference have been corrected.

3 years ago0029393: Samples - AndroidQt sample build fail
kgv [Tue, 19 Dec 2017 11:00:10 +0000 (14:00 +0300)]
0029393: Samples - AndroidQt sample build fail

AndroidQt - dropped removed arguments within V3d_View::SetWindow() usage [0024776].
AndroidQt_Window - add missing NativeFBConfig() method declared in the interface [0026732].

3 years ago0028722: Conversion of a spherical face to a spline produces an invalid shape
ifv [Mon, 4 Dec 2017 13:20:19 +0000 (16:20 +0300)]
0028722: Conversion of a spherical face to a spline produces an invalid shape

Correction of 2d tolerance calculation for BSpline/Bezier surfaces
Test cases are corrected according to current behavior of algorithm

3 years ago0029362: Data Exchange - Crash during reading step file
ika [Mon, 4 Dec 2017 08:05:02 +0000 (11:05 +0300)]
0029362: Data Exchange - Crash during reading step file

Additional check for wires into searching of attached shapes,
Fix Dimension value processing,

3 years ago0029377: Configuration, CMake - linkage errors while using static OCCT libraries...
kgv [Fri, 8 Dec 2017 12:13:14 +0000 (15:13 +0300)]
0029377: Configuration, CMake - linkage errors while using static OCCT libraries on Windows platform

HAVE_NO_DLL is now defined when building Static libraries.

3 years ago0029376: Coding Rules, TColStd_PackedMapOfInteger - declare Iterator as nested class...
kgv [Thu, 7 Dec 2017 17:58:36 +0000 (20:58 +0300)]
0029376: Coding Rules, TColStd_PackedMapOfInteger - declare Iterator as nested class of map collection

3 years ago0029363: No history for shapes which were produced as a result of intersection
emv [Tue, 12 Dec 2017 12:10:37 +0000 (15:10 +0300)]
0029363: No history for shapes which were produced as a result of intersection

1. Implementation of the *Generated* method for the algorithms in Boolean Component.
In terms of these algorithms the shape from the arguments can have Generated shapes only if these new shapes have been obtained as a result of pure intersection (not overlapping) of this shape with any other shapes from arguments.
Thus, the Generated shapes are always:
* VERTICES created from the intersection points and may be Generated from edges and faces only;
* EDGES created from the intersection edges and may be Generated from faces only.

So, only EDGES and FACES could have information about Generated shapes. For all other types of shapes the list of Generated shapes will be empty.

2. Optimization and simplification of the Modified and IsDeleted methods based on the correct filling of the BOPAlgo_BuilderShape::myImagesResult map.

3. Provide history of unification of the solids in the CellsBuilder algorithm.

4. Update of the documentation of Boolean Operations User guide with new chapter "History Information" describing rules for filling history for operations in Boolean Component.

5. Test cases for the issue. New grid "history" has been added into "boolean" category.

3 years ago0029179: Result of Boolean common depends on an order of arguments
nbv [Thu, 7 Dec 2017 08:28:20 +0000 (11:28 +0300)]
0029179: Result of Boolean common depends on an order of arguments

Usage of Bnd_Box-filtering is eliminated while putting a (definitely) common vertex between two faces on the intersection curve.

Algorithm of putting not-common (ON/IN) vertices has not been changed.

3 years ago0029359: Approximation algorithm computes multidimensional distance in Euclidean...
nbv [Thu, 30 Nov 2017 10:44:48 +0000 (13:44 +0300)]
0029359: Approximation algorithm computes multidimensional distance in Euclidean space incorrectly

Wrong distance computation has been corrected.

Some test-cases have been corrected according to their new behavior. Namely, built (by approximation algorithm) curve(s) has changed its geometrical position.

1. tests/blend/simple/X4

It is not a regression because the result is not correct on both MASTER and FIX (see explanation in the issue #26740). This problem is expected to be solved after porting Fillet-algorithm to new Boolean operation. Old Boolean operations do not cover Edge-Edge tangent-zone by vertex.

2. tests/bugs/modalg_6/bug27341_318

"checknbshapes" has been deleted in order to avoid non-stable behavior (see issue #29360) of this test case. New result is OK on both Linux and Windows platform.

3 years ago0029351: Boolean Operations create invalid pcurves
emv [Tue, 5 Dec 2017 11:22:22 +0000 (14:22 +0300)]
0029351: Boolean Operations create invalid pcurves

When making pcurve for edge on face make sure that the produced 2D curve will have the same range as 3D curve of the edge.

3 years ago0026570: Crash on attempt to rotate a shape.
vro [Wed, 19 Aug 2015 07:25:11 +0000 (10:25 +0300)]
0026570: Crash on attempt to rotate a shape.

An extended draw-command trotate (ttranslate, tmirror, ...) by an additional parameter "-copy".
New check of edge range is added in BRepCheck/BRepCheck_Edge.cxx
The same checking is added in ShapeAnalysis_Edge.cxx
Fixing this problem is added in ShapeFix_Wire.cxx
GeomLib::SameRange(...) and BRepTools_TrsfModification::NewCurve2d(...) are modified to avoid exception in TrimmedCurve

3 years ago0029355: OCCT 6.9.1 persistence restored in OCCT 7.2.0 not working
abv [Sat, 2 Dec 2017 08:24:58 +0000 (11:24 +0300)]
0029355: OCCT 6.9.1 persistence restored in OCCT 7.2.0 not working

Auxiliary classes StdObjMgt_ReadData::Object and StdObjMgt_WriteData::Object are renamed to "ObjectSentry" (to better reflect their nature); constructor is made explicit to ensure that such objects are always created intentionally.
These objects are instantiated explicitly in the body of relevant functions, instead of implicit creation as temporary objects when function requires such object as argument.
Variable used to get char from stream is nullified in several places in FSD_File and other classes, to avoid possible usage of uninitialized memory in case if stream is bad.

3 years ago0029372: Graphic3d_TransformPers - improve description of Local Coordinate system...
kgv [Thu, 7 Dec 2017 07:57:26 +0000 (10:57 +0300)]
0029372: Graphic3d_TransformPers - improve description of Local Coordinate system defined by Transformation Persistence

3 years ago0029365: Visualization, TKOpenGl - do not include hidden structures to Rendered withi...
kgv [Mon, 4 Dec 2017 12:16:34 +0000 (15:16 +0300)]
0029365: Visualization, TKOpenGl - do not include hidden structures to Rendered within frame statistics

OpenGl_Layer::UpdateCulling() now considers structure as culled in case if it has Hidden state.

3 years ago0029312: Using OBB to speed up Boolean Operations
emv [Wed, 1 Nov 2017 08:30:30 +0000 (11:30 +0300)]
0029312: Using OBB to speed up Boolean Operations

1. Implementation of the user-defined option for usage of Oriented Bounding Boxes (OBB) in Boolean Operations for additional filtering (rejection) of selected for intersection pairs of sub-shapes.

By default the usage of OBB is turned off.
To enable/disable its usage the method SetUseOBB(flag) should be used. This method is available for all operations in Boolean Component.
To enable/disable it in draw the command "buseobb 0/1" should be used. Note, that this will affect all subsequent operations.

The OBB for the shapes are built by first necessity and stored into operation context (IntTools_Context).

2. Usage of the OBB in some test cases.

3 years ago0029311: Implementation of the Oriented Bounding Boxes (OBB) functionality
nbv [Wed, 8 Nov 2017 12:47:09 +0000 (15:47 +0300)]
0029311: Implementation of the Oriented Bounding Boxes (OBB) functionality

1. The class Bnd_OBB has been created to describe the Oriented Bounding Box.

2. Several key methods have been implemented: Bnd_OBB::IsOut(...), Bnd_OBB::Add(...) and Bnd_OBB::Enlarge(...).

3. Interface of Bnd_Box class has changed. New methods have been created. See Bnd_Box.hxx for detailed information.

4. BRepBndLib and Draw_Box classes have been amended in order to provide correct work with Bnd_OBB class.

5. Interface of "bounding" DRAW-command has been changed. Please see help for detailed information.

6. New DRAW-command "isbbinterf" has been created. Please see help for detailed information.

7. "boundingstr" and "optbounding" DRAW-commands have been eliminated because their function can be made by "bounding" DRAW-command (e.g. see tests/bugs/vis/buc60857 or samples/tcl/snowflake.tcl test cases).

8. Documentation has been updated.

3 years ago0029366: Visualization, OpenGl_Text - artifacts when using Aspect_TODT_SHADOW/Aspect_...
kgv [Mon, 4 Dec 2017 13:29:43 +0000 (16:29 +0300)]
0029366: Visualization, OpenGl_Text - artifacts when using Aspect_TODT_SHADOW/Aspect_TODT_DEKALE at different zoom level

OpenGl_Text now applies Polygon Offset instead of Z-shift in world coordinates for drawing background.
OpenGl_Context::SetPolygonOffset() - polygon offset state has been moved from OpenGl_Workspace to OpenGl_Context.

3 years ago0029323: Intersection algorithm produces the curve with oscillation
nbv [Mon, 13 Nov 2017 10:55:48 +0000 (13:55 +0300)]
0029323: Intersection algorithm produces the curve with oscillation

Sometimes the algorithm of purging of extra points in the walking line makes enormous difference in distance between two neighbor segments of the line. This badly impacts the quality of approximation result. This patch balances the difference in distances by forbidding deletion of some points.

1. tests/bugs/modalg_6/bug27615

The reason of the correction is explained in the message ~0072580 (see issue #28557).

2. tests/bugs/modalg_7/bug28892*

The reason of the correction is explained in the message ~0072583 (see issue #28984).

3 years ago0029344: Foundation Classes, TCollection_AsciiString - replace confusing strncpy...
kgv [Fri, 24 Nov 2017 15:37:01 +0000 (18:37 +0300)]
0029344: Foundation Classes, TCollection_AsciiString - replace confusing strncpy with memcpy

strncpy() usage within TCollection_AsciiString has been replaced by memcpy()
where string length has been already determined.

TCollection_AsciiString::SetValue() now throws exception
on attempt to set '\0' - TCollection_AsciiString::Trunc() should be used instead.
TCollection_AsciiString(const char* , int ) has been modified to properly set string length
in case of NULL-terminator appeared earlier then specified length.

Interface_LineBuffer has been revised for using NCollection_Array1 instead of TCollection_AsciiString
to avoid misusing TCollection_AsciiString interface.

3 years ago0029346: Visualization, TKOpenGl - collect frame statistics
kgv [Fri, 24 Nov 2017 15:43:01 +0000 (18:43 +0300)]
0029346: Visualization, TKOpenGl - collect frame statistics

Graphic3d_RenderingParams::ToShowStats - new option displaying rendering statistics.

OpenGl_FrameStats - new class for accumulating frame statistics.
OpenGl_Context::FrameStats() provides an access to the frame stats
used for currently rendered context.
OpenGl_View::Redraw() and OpenGl_View::RedrawImmediate()
resets counters within OpenGl_Context::FrameStats().

OpenGl_Layer::UpdateCulling() - simplified resetting of culling state for cullable structures.

3 years ago0029353: Optimization of TPrsStd_AISPresentation::SetSelectionMode()
vro [Wed, 29 Nov 2017 13:36:21 +0000 (16:36 +0300)]
0029353: Optimization of TPrsStd_AISPresentation::SetSelectionMode()

3 years ago0029290: Visualization, TKOpenGl - allow defining Light source per ZLayer
kgv [Thu, 2 Nov 2017 12:36:20 +0000 (15:36 +0300)]
0029290: Visualization, TKOpenGl - allow defining Light source per ZLayer

Graphic3d_CLight is now defined as a class inheriting Standard_Transient,
so that it's fields now should be accessed through methods.
Graphic3d_CLight::IsEnabled() - new property allowing to disable light source everywhere.
Confusing alias OpenGl_Light has been removed.

Graphic3d_CLight::SetAttenuation() - the upper limit 1.0 of attenuation factors has been removed
since it contradicts to OpenGL specs and does not make sense.

Graphic3d_ZLayerSettings::Lights() - light sources list is now property of ZLayer.
When defined, it overrides light sources defined for View/Viewer.
New class Graphic3d_LightSet has been defined to define a set of light sources.

V3d_Light - removed obsolete interface for debug drawing lights sources.
V3d_Light is now an alias to Graphic3d_CLight.
V3d_TypeOfLight is now defined as a typedef to Graphic3d_TypeOfLightSource.

3 years ago0029350: Visualization, OpenGl_Text - add Aspect_TODT_SHADOW text style
kgv [Tue, 28 Nov 2017 06:46:42 +0000 (09:46 +0300)]
0029350: Visualization, OpenGl_Text - add Aspect_TODT_SHADOW text style

Within new style Aspect_TODT_SHADOW, a tiny shadow is drawn at right-bottom corner
with one pixel shift, producing a slightly nicer visual look than Aspect_TODT_DEKALE.

3 years ago0029331: Visualization, TKOpenGl - make OpenGl_PrimitiveArray::IsFillDrawMode() as...
kgv [Thu, 16 Nov 2017 19:57:48 +0000 (22:57 +0300)]
0029331: Visualization, TKOpenGl - make OpenGl_PrimitiveArray::IsFillDrawMode() as virtual method of OpenGl_Element

3 years ago0028416: Visualization - SelectMgr_SelectionManager::Activate() should not implicitly...
kgv [Tue, 31 Jan 2017 08:26:48 +0000 (11:26 +0300)]
0028416: Visualization - SelectMgr_SelectionManager::Activate() should not implicitly deactivate Global Selection Mode

Implicit deactivation of global selection mode has been removed from SelectMgr_SelectionManager::Activate().

Added new method AIS_InteractiveContext::SetSelectionModeActive() as replacement
of AIS_InteractiveContext::Activate() / ::Deactivate().
New method takes an argument AIS_SelectionModesConcurrency which defines
what to do with already activated selection modes in 3 ways:
- AIS_SelectionModesConcurrency_Single, only one selection mode can be activated at
  the same moment - previously activated should be deactivated
- AIS_SelectionModesConcurrency_GlobalOrLocal, either Global (AIS_InteractiveObject::GlobalSelectionMode()
  or Local (multiple) selection modes can be active at the same moment
- AIS_SelectionModesConcurrency_Multiple, any combination of selection modes can be activated.

3 years ago0029333: Boolean Operations - Prevent modification of the input shapes in case their...
emv [Fri, 17 Nov 2017 13:27:36 +0000 (16:27 +0300)]
0029333: Boolean Operations - Prevent modification of the input shapes in case their sub-shapes have not been modified

Prevent modification of the input shapes in destructive mode in case their sub-shapes have not been modified:
1. Prevent edge splitting for the pave blocks with old vertices if it is possible to use the existing edge (*BOPAlgo_PaveFiller::MakeSplitEdges*);
2. Prevent creation of the new containers (WIRES/SHELLS/COMPSOLIDS) if non of its parts have been modified (*BOPAlgo_Builder::FillImagesContainer*);
3. Prevent creation of the new face if non of its wires have been modified (*BOPAlgo_Builder::FillImagesFaces*);
4. If possible, use the original face to be the representative for the group of SD faces (*BOPAlgo_Builder::FillSameDomainFaces*).

Cosmetic changes:
1. Documentation of the *BOPAlgo_Builder* class.
2. Making simple methods of the *BOPAlgo_Builder* class inline.
3. Getting rid of the *BOPAlgo_Builder::mySplits* field as it is excessive. *BOPAlgo_Builder::myImages* can be used instead.
3. Moving the Check Inverted option from *BOPAlgo_Options* to *BOPAlgo_Builder*.

Test cases for the issue.
Adjustment of the test case to their current behavior.
Test case *blend/complex/H2* has been deleted as duplicate of the test case *blend/simple/Z1*.

3 years ago0025609: Clean up the duplicate classes in TKBO project
emv [Thu, 23 Nov 2017 07:57:25 +0000 (10:57 +0300)]
0025609: Clean up the duplicate classes in TKBO project

1. The package BOPCol has been fully removed:
- *BOPCol_BaseAllocator* is replaced with *Handle(NCollection_BaseAllocator)*;
- *BOPCol_BoxBndTree* is replaced with *BOPTools_BoxBndTree*;
- *BOPCol_Box2DBndTree* is removed as unused;
- *BOPCol_DataMapOfIntegerInteger* is replaced with *TColStd_DataMapOfIntegerInteger*;
- *BOPCol_DataMapOfIntegerListOfInteger* is replaced with *TColStd_DataMapOfIntegerListOfInteger*;
- *BOPCol_DataMapOfIntegerListOfShape* is replaced with *TopTools_DataMapOfIntegerListOfShape*;
- *BOPCol_DataMapOfIntegerMapOfInteger.hxx* is removed as unused;
- *BOPCol_DataMapOfIntegerReal* is replaced with *TColStd_DataMapOfIntegerReal*;
- *BOPCol_DataMapOfIntegerShape* is replaced with *TopTools_DataMapOfIntegerShape*;
- *BOPCol_DataMapOfShapeBox* is replaced with *TopTools_DataMapOfShapeBox*;
- *BOPCol_DataMapOfShapeInteger* is replaced with *TopTools_DataMapOfShapeInteger*;
- *BOPCol_DataMapOfShapeListOfShape* is replaced with *TopTools_DataMapOfShapeListOfShape*;
- *BOPCol_DataMapOfShapeReal* is replaced with *TopTools_DataMapOfShapeReal*;
- *BOPCol_DataMapOfShapeShape* is replaced with *TopTools_DataMapOfShapeShape*;
- *BOPCol_DataMapOfTransientAddress* is removed as unused;
- *BOPCol_IndexedDataMapOfIntegerListOfInteger* is removed as unused;
- *BOPCol_IndexedDataMapOfShapeBox* is removed as unused;
- *BOPCol_IndexedDataMapOfShapeInteger* is removed as unused;
- *BOPCol_IndexedDataMapOfShapeListOfShape* is replaced with *TopTools_IndexedDataMapOfShapeListOfShape*;
- *BOPCol_IndexedDataMapOfShapeReal* is removed as unused;
- *BOPCol_IndexedDataMapOfShapeShape* is replaced with *TopTools_IndexedDataMapOfShapeShape*;
- *BOPCol_IndexedMapOfInteger* is replaced with *TColStd_IndexedMapOfInteger*;
- *BOPCol_IndexedMapOfOrientedShape* is replaced with *TopTools_IndexedMapOfOrientedShape*;
- *BOPCol_IndexedMapOfShape* is replaced with *TopTools_IndexedMapOfShape*;
- *BOPCol_ListOfInteger* is replaced with *TColStd_ListOfInteger*;
- *BOPCol_ListOfListOfShape* is replaced with *TopTools_ListOfListOfShape*;
- *BOPCol_ListOfShape* is replaced with *TopTools_ListOfShape*;
- *BOPCol_MapOfInteger* is replaced with *TColStd_MapOfInteger*;
- *BOPCol_MapOfOrientedShape* is replaced with *TopTools_MapOfOrientedShape*;
- *BOPCol_MapOfShape* is replaced with *TopTools_MapOfShape*;
- *BOPCol_PListOfInteger* is removed as unused;
- *BOPCol_PInteger* is removed as unused
- *BOPCol_SequenceOfPnt2d* is replaced with *TColgp_SequenceOfPnt2d*;
- *BOPCol_SequenceOfReal* is replaced with *TColStd_SequenceOfReal*;
- *BOPCol_SequenceOfShape* is replaced with *TopTools_SequenceOfShape*;
- *BOPCol_Parallel* is replaced with *BOPTools_Parallel*;
- *BOPCol_NCVector* is replaced with *NCollection_Vector*;

2. The class *BOPDS_PassKey* and containers for it have been removed as unused;

3. The unused containers from *IntTools* package have been removed:
- *IntTools_DataMapOfShapeAddress* is removed as unused;
- *IntTools_IndexedDataMapOfTransientAddress* is removed as unused;

4. The container *BiTgte_DataMapOfShapeBox* is replaced with *TopTools_DataMapOfShapeBox*;

5. The class *BOPTools* has been removed as duplicate of the class *TopExp*;

3 years ago0029349: Foundation Classes, OSD_Timer - add missing theThisThreadOnly constructor...
kgv [Tue, 28 Nov 2017 05:42:40 +0000 (08:42 +0300)]
0029349: Foundation Classes, OSD_Timer - add missing theThisThreadOnly constructor option available in OSD_Chronometer

OSD_Timer constructor now has theThisThreadOnly option passed to OSD_Chronometer,
which is FALSE by default.

OSD_Chronometer now provides methods ::UserTimeCPU(), ::SystemTimeCPU() allowing
to fetch CPU times without akward syntax through overloaded ::Show().

3 years ago0029289: Wrong derivatives in math_TrigonometricFunctionRoots.cxx file
ifv [Thu, 9 Nov 2017 14:20:10 +0000 (17:20 +0300)]
0029289: Wrong derivatives in math_TrigonometricFunctionRoots.cxx file

Class MyTrigoFunction is removed from file math_TrigonometricFunctionRoots.cxx.
New class math_TrigonometricEquationFunction with the same functionality is created to provide possibilities
for individual testing.
Expressions for derivatives are corrected.
New Draw command "intconcon" for intersection 2d conic curves is created.
Test command OCC29289 (file QABugs_20.cxx) is created for individual testing math_TrigonometricEquationFunction.
It is used in tests/bugs/modalg_7/bug29289

3 years ago0029338: Data Exchange - Add Planes for Tolerance zones in Geometric tolerances
ika [Thu, 23 Nov 2017 09:40:59 +0000 (12:40 +0300)]
0029338: Data Exchange - Add Planes for Tolerance zones in Geometric tolerances

Add new poles to Geometric Tolerance objects
Some code refactoring in GDT attributes
Add Draw functions for affected plane

3 years agoUpdate of tests database
mkv [Thu, 26 Oct 2017 08:59:25 +0000 (11:59 +0300)]
Update of tests database

Test for 0023171: Got wrong result after stitching
Test for 0023612: BRepPrimAPI_MakeTorus
Test for 0023610: checkshape command does not detect mismatch of the tolerance values among the sub-shapes of the shape
Test for 0023660: 'bsection' produces result with gap
Test for 0024397: BRepBuilderAPI_Copy works not correct
Test for 0023927: BRepProj_Projection produces Edge with one Vertex outside from curve
Test for 0023902: Cannot build fillet
Test for 0023669: Error in Boolean operations algorithm: section between cylindrical face and prism is incomplete
Test for 0023660: 'bsection' produces result with gap
Test for 0024100: Bad pcurve is built on a face while converting the shape from STEP format.
Test for 0024319: Sewing makes a solid transparent
Test for 0024302: Section operation modifies input argument
Test for 0024397: BRepBuilderAPI_Copy works not correct
Test for 0024418: Wrong section curves when intersecting analytical surfaces

3 years ago0029142: Exception on Redo.
szy [Mon, 20 Nov 2017 14:31:15 +0000 (17:31 +0300)]
0029142: Exception on Redo.

Order of attributes deltas is fixed.

3 years ago0029175: BRepOffsetAPI_MakeOffset problem
emv [Wed, 22 Nov 2017 13:29:40 +0000 (16:29 +0300)]
0029175: BRepOffsetAPI_MakeOffset problem

The problem is fixed by the fix for the issue #29162.
Creating test case for the issue.

3 years ago0029321: Improve performance of the 3D offset algorithm for the planar cases
emv [Mon, 13 Nov 2017 13:03:31 +0000 (16:03 +0300)]
0029321: Improve performance of the 3D offset algorithm for the planar cases

Disable additional intersection of the splits of offset faces implemented in method BRepOffset_MakeOffset::Intersection3D() for the mode "Complete" and join type "Intersection".
This method is insufficient, as it does not allow to detect the tangential faces, necessary for this mode.
Intersection is performed in the method BuildShellsCompleteInter() developed for this mode.

3 years ago0029322: Unify faces classification procedures in Boolean Operations
emv [Mon, 13 Nov 2017 08:11:27 +0000 (11:11 +0300)]
0029322: Unify faces classification procedures in Boolean Operations

1. Unify the faces classification procedure of the methods BOPAlgo_BuilderSolid::PerformInternalShapes() and BOPAlgo_Builder::FillIn3DParts() using the latter as a base.
The new method BOPAlgo_Tools::ClassifyFaces() has been created for that. Both methods mentioned above have been updated to use the new one.

2. Forced intersection of the edges after enlarge of the tolerance values of their vertices during the operation has been added into BOP's intersection algorithm (BOPAlgo_PaveFiller).
BOPAlgo_Tools::PerformCommonBlocks() method has been updated to avoid loosing faces of the already created Common blocks.

As a result the case "boolean gdml_private ZF6" became more stable, because the intermediate result is no longer invalid.
Additional test cases have been added to verify this improvement (bugs modalg_7 bug29322_*)

3. When building PCurves for edges on faces, check the existing PCurves on its validity for periodic surfaces and adjust PCurves if necessary.
The improvement helps to produce the valid result in the test case "bugs moddata_1 bug152".

4. Avoid creation of empty Edge-Edge interference if the intersection point is lying close to a shared vertex.
The improvement helps obtain the valid result of the "bopcheck" operation in the test case "bugs modalg_7 bug27683".

Adjustment of the test case to current behavior:
- Avoid usage of the self-intersecting torus in the test case "boolean gdml_public A9".
- Fix the input shape in the test case "boolean bopfuse_complex H1" to provide valid result.
- Test cases "bugs moddata_1 bug152_1 bug152_2" have been unified into single test case "bugs moddata_1 bug152". Now, the case rebuilds invalid input data and performs all types of Boolean operations.
- Change the test case "bugs modalg_7 bug22750" to produce valid intermediate results used in subsequent operations as arguments.
- The following test cases are improvements:
- "boolean volumemaker D8"
- "boolean volumemaker G1"
- "bugs modalg_7 bug27683"

3 years ago0025879: result of blend fails the bopcheck
aml [Fri, 10 Nov 2017 15:21:21 +0000 (18:21 +0300)]
0025879: result of blend fails the bopcheck

Protection from the zero-length gp_Dir construction is added.

3 years ago0029113: Coding Rules - fix convention to have one header per API type
abv [Fri, 15 Sep 2017 07:52:14 +0000 (10:52 +0300)]
0029113: Coding Rules - fix convention to have one header per API type

Paragraph on naming of classes is revised to describe correspondence of names of public types and files.

3 years ago0029234: BRepOffsetAPI_NormalProjection produces INTERNAL edges and vertices
msv [Fri, 20 Oct 2017 15:59:01 +0000 (18:59 +0300)]
0029234: BRepOffsetAPI_NormalProjection produces INTERNAL edges and vertices

Make the algorithm BRepOffsetAPI_NormalProjection use section operation instead of common to get the edge-result of projection to be in face restrictions.

Correct Boolean operation algorithm to produce correctly oriented result in the case of common between face and edge.

The algorithm ShapeUpgrade_UnifySameDomain has been patched to correctly gather same domain faces in a compound.

The TCL script snowflake.tcl has been corrected to get rid of dependence on the order of edges in the result of Boolean operation.

3 years ago0029317: Configuration, CMake - CSF_d3d9 should be processed for building TKD3DHost...
kgv [Tue, 14 Nov 2017 08:38:31 +0000 (11:38 +0300)]
0029317: Configuration, CMake - CSF_d3d9 should be processed for building TKD3DHost using MinGW

3 years ago0029287: Move package GProp from TKG2d to TKG3d
msv [Wed, 8 Nov 2017 15:13:47 +0000 (18:13 +0300)]
0029287: Move package GProp from TKG2d to TKG3d

3 years ago0029204: BRepOffsetAPI_MakePipeShell produces invalid result and raises exception...
jgv [Wed, 18 Oct 2017 14:35:58 +0000 (17:35 +0300)]
0029204: BRepOffsetAPI_MakePipeShell produces invalid result and raises exception in Draw

1.The algorithm searching the section in the corner (ChooseSection) is modified to be able to find simple cases with rather big tolerance.

2. The constructor of BRepFill_Section is modified: now it removes locations in the shape of section like it was done in BRepFill_Pipe.

3. Correction of U-edges by Same Parameter has been added to the method BRepFill_Sweep::Build.

3 years ago0025968: boolean returning an invalid shape
aml [Sat, 11 Nov 2017 07:33:45 +0000 (10:33 +0300)]
0025968: boolean returning an invalid shape

Problem is not reproduced on current master.
Testing case was created

3 years ago0029301: Improve performance of Boolean Operations
emv [Wed, 8 Nov 2017 06:16:35 +0000 (09:16 +0300)]
0029301: Improve performance of Boolean Operations

Improve performance of Boolean operations algorithm by:
- Improving the check of Same Domain faces (BOPAlgo_Builder::FillSameDomainFaces());
- Faster rejection of outer faces for solids using Bounding Box classification first (BOPAlgo_Builder::FillIn3DParts());
- Using IncAllocator for local containers.

Quality improvement has been made in BOPAlgo_PaveFiller class:
1. Method IsExistingPaveBlock() has been corrected to provide the correct edge tolerance and to obtain valid intermediate results in the test case "boolean gdml_private ZH3".
   New test case have been added to verify this improvement (bugs modalg_7 bug29301).
2. Method PutClosingPaveOnCurve() has been corrected to use the tolerance of the pave put on the bound for checking curve on closeness.
   Additional check for the curve to have valid range after addition of the pave on the other end has been added to prevent considering the small curves (covered by vertex tolerance) as closed ones.
   As a result of this modification the test case boolean gdml_public B2 has been fixed (TODO removed).

Adjustment of the test cases to current behavior:
- boolean bopcommon_complex J1 - the produced result was incorrect as it was self-interfered. There should be no common in this case.
- offset shape_type_i_c ZZ1 - the incorrect result is now produced instead of null shape.

3 years ago0025966: Booleans with tangent faces
aml [Sat, 11 Nov 2017 07:44:41 +0000 (10:44 +0300)]
0025966: Booleans with tangent faces

Problem is not reproduced on current master.
Testing case was created

3 years ago0025104: Prism from BSpline curve can not be chamfered
aml [Thu, 2 Nov 2017 09:08:18 +0000 (12:08 +0300)]
0025104: Prism from BSpline curve can not be chamfered

Protection from zero-length vectors has been added.

3 years ago0028467: Improve UnifySameDomain performance
msv [Fri, 17 Feb 2017 08:26:25 +0000 (11:26 +0300)]
0028467: Improve UnifySameDomain performance

This patch turns off some not needed modes of fix in the called ShapeFix_Face algorithm.

It stores pcurves on planes in the edges to avoid repeated computation of the same pcurves many times (it is done only when SafeInputMode is false).

It avoids unnecessary replace/apply actions in the modification context.

It removes the code that makes decomposition of surface of the face on several faces.

The new command buildpcurvesonplane has been added, which builds and stores pcurves of edges on planar faces. This is useful for investigations how presence of pcurves on planes influence performance of algorithms.

Make drawing of dimension line in snowflake test independent on the order of vertices in the result.

3 years ago0028760: Visualization, TKOpenGl - avoid excessive frustum culling traverse within...
kgv [Wed, 8 Nov 2017 06:44:27 +0000 (09:44 +0300)]
0028760: Visualization, TKOpenGl - avoid excessive frustum culling traverse within extra OIT rendering pass

Culling traverse is no more called implicitly within OpenGl_Layer::Render().
Instead, all layers are traversed at onces within OpenGl_View::render() beforehand.

OpenGl_BVHTreeSelector methods have been renamed to better reflect their meaning.
Non-persistent culling options has been moved to dedicated structure OpenGl_BVHTreeSelector::CullingContext
so that OpenGl_BVHTreeSelector instance can be used for different Layers without modifying its state.

3 years ago0029283: Visualization - allow defining more than 8 light sources
kgv [Tue, 31 Oct 2017 17:02:13 +0000 (20:02 +0300)]
0029283: Visualization - allow defining more than 8 light sources

OpenGl_ShaderManager now overrides THE_MAX_LIGHTS within built-in shading programs
so that maximum number of lights is now limited only by OpenGL hardware
(e.g. length of GLSL program, number of defined uniforms, result performance, etc.).
THE_MAX_CLIP_PLANES is now also defined by OpenGl_ShaderManager,
so that unused lights and clipping planes do not reserve extra uniforms in GLSL programs.

V3d_View::SetLightOn() does not throw exception anymore, when the number of lights exceeds 8.
Instead, OpenGl_ShaderManager::PushLightSourceState() emits warning
in case of usage of FFP providing consistent behavior with Clipping Planes number limit.

3 years ago0029304: Draw Harness, DBRep_DrawableShape - fix inappropriate use of unordered map
kgv [Thu, 9 Nov 2017 07:18:49 +0000 (10:18 +0300)]
0029304: Draw Harness, DBRep_DrawableShape - fix inappropriate use of unordered map

TColStd_DataMapOfIntegerInteger has been replaced by NCollection_Vector.

3 years ago0029299: Foundation Classes, NCollection - define explicit empty constructor for...
kgv [Tue, 7 Nov 2017 14:23:29 +0000 (17:23 +0300)]
0029299: Foundation Classes, NCollection - define explicit empty constructor for map classes

Ambiguous constructors have been marked with explicit keyword for classes
NCollection_DataMap, NCollection_DoubleMap, NCollection_IndexedDataMap, NCollection_IndexedMap,
NCollection_List, NCollection_LocalArray, NCollection_Map, NCollection_Sequence,
NCollection_SparseArray, NCollection_UBTree,

3 years ago0029302: Foundation Classes, NCollection - optimize iteration of indexed maps
kgv [Wed, 8 Nov 2017 12:25:51 +0000 (15:25 +0300)]
0029302: Foundation Classes, NCollection - optimize iteration of indexed maps

NCollection_IndexedMap and NCollection_IndexedDataMap
now access Key by Index number without computing Hash code.
IndexedMapNode::myNext2 and IndexedDataMapNode::myNext2 fields
have been removed, so that indexed map now may utilize less memory.

TCollection::NextPrimeForMap() has been extended upto 2038431745
(almost full signed 32-bit integer range),
and NCollection_BaseMap::mySaturated property has been removed.

NCollection_IndexedDataMap::RemoveFromIndex(), FindKey(), FindFromIndex(),
ChangeFromIndex() - removed duplicating checks for out of range input.

3 years ago0029300: Visualization, TKOpenGl - provide depth pre-pass option
kgv [Tue, 7 Nov 2017 21:11:57 +0000 (00:11 +0300)]
0029300: Visualization, TKOpenGl - provide depth pre-pass option

OpenGl_LayerList::Render() now handles new option Graphic3d_RenderingParams::ToEnableDepthPrepass
which prepends additional pass to rendering pipeline filling Depth Buffer in advance.

3 years ago0029292: Coding Rules - remove Graphic3d_Vector duplicating gp_XYZ
kgv [Thu, 2 Nov 2017 13:29:17 +0000 (16:29 +0300)]
0029292: Coding Rules - remove Graphic3d_Vector duplicating gp_XYZ

Graphic3d_Vector has been replaced by gp_Pnt/gp_XYZ/gp_Dir depending on context.
StdSelect_ViewerSelector3d::ToPixMap() - fixed unsafe float math causing out-of-range color results.

3 years ago0029162: Geom2dInt_GInter algorithm does not find intersection of ellipse and line
ifv [Thu, 12 Oct 2017 13:54:35 +0000 (16:54 +0300)]
0029162: Geom2dInt_GInter algorithm does not find intersection of ellipse and line

Analytical intersection algorithm is implemented for ellipse-line intersection

3 years ago0029297: [Regression] Configuration - Mingw-w64 build fails for TKOpenGl due to missi...
abv [Mon, 6 Nov 2017 13:43:34 +0000 (16:43 +0300)]
0029297: [Regression] Configuration - Mingw-w64 build fails for TKOpenGl due to missing link to OpenGl

The order of external libraries is corrected in TKOpenGl/EXTERNLIB so that more higher-level libs are mentioned earlier than low-level libs on which the former depend (e.g. Gl2Ps before OpenGl) to ensure that GCC linker is able to resolve dependencies.

3 years ago0029258: Foundation Classes - provide move constructors for string classes
kgv [Sun, 15 Oct 2017 13:08:01 +0000 (16:08 +0300)]
0029258: Foundation Classes - provide move constructors for string classes

New macro OCCT_NO_RVALUE_REFERENCE is introduced to disable methods using move semantics on obsolete compilers that do not support rvalue references.

TCollection_AsciiString, TCollection_ExtendedString, NCollection_UtfString - added method Swap(), move constructor, and move assignment operator.

Draw command QATestArrayMove is added to test for memory corruption if NCollection_Array1<> bound to local C buffer is returned from function by value.

3 years ago0029295: Visualization, TKOpenGl - provide distance culling option
kgv [Sun, 5 Nov 2017 15:07:25 +0000 (18:07 +0300)]
0029295: Visualization, TKOpenGl - provide distance culling option

Graphic3d_ZLayerSettings::CullingDistance() and ::CullingSize() - added
new properties configuring culling of small and distant objects, disabled by default.
OpenGl_BVHTreeSelector now handles size culling and distance culling in addition to frustom culling.

3 years ago0029285: Visualization, V3d_View::UpdateLights() - eliminate implicit redraw
kgv [Thu, 2 Nov 2017 07:04:39 +0000 (10:04 +0300)]
0029285: Visualization, V3d_View::UpdateLights() - eliminate implicit redraw

3 years ago0029286: Visualization, TKOpenGl - do not update FFP state when OpenGl_Caps::ffpEnabl...
kgv [Thu, 2 Nov 2017 07:28:01 +0000 (10:28 +0300)]
0029286: Visualization, TKOpenGl - do not update FFP state when OpenGl_Caps::ffpEnable is disabled

3 years ago0029293: Boolean Operations algorithm does not preserve the orientations of the faces
emv [Fri, 3 Nov 2017 09:12:34 +0000 (12:12 +0300)]
0029293: Boolean Operations algorithm does not preserve the orientations of the faces

While building splits of faces (BOPAlgo_Builder::FillImagesFaces()) make sure that the orientation of the input face will be passed to its splits.

Extend draw command "normals" with new key "[-print]" which allows printing values of the normal vector.

3 years ago0029133: Unstable test cases
mnt [Wed, 18 Oct 2017 11:13:35 +0000 (14:13 +0300)]
0029133: Unstable test cases

Adding new and early removed for some reasons test cases

Removing TODO ?

Adding necessary TODO ? on corresponding platforms for OK and BAD cases

3 years ago0029282: Data Exchange - UpdateAssemblies is not working for located root assemblies
ika [Mon, 30 Oct 2017 15:04:12 +0000 (18:04 +0300)]
0029282: Data Exchange - UpdateAssemblies is not working for located root assemblies

Add check for located root assemblies.

3 years ago0029237: Improve performance of Boolean Operations
emv [Wed, 18 Oct 2017 08:05:24 +0000 (11:05 +0300)]
0029237: Improve performance of Boolean Operations

In order to improve performance of Boolean Operations on the relatively fast cases the following improvements have been made:
1. Initialization of the FaceInfo information for the faces participating in Face/Face interference, even when the gluing is ON, to take into account intersection of their sub-shapes.
2. Code simplification & duplication removal - the methods BOPAlgo_ShellSplitter::MakeConnexityBlocks and BOPAlgo_WireSplitter::MakeConnexityBlocks have been unified into BOPTools_AlgoTools::MakeConnexityBlocks.
3. Avoid unnecessary bounding box computation for solids during DS initialization. The bounding boxes for solids will be computed during the building stage to find faces located inside solids.
   For the shape self-interference check (performed by the BOPAlgo_CheckerSI), the bounding box is still computed, as it is necessary to resolve Shape/Solid intersections.
4. Use only three sample points to check coincidence of line and plane.
5. Perform necessity of planes intersection only when the gluing is off.
6. Avoid repeated initialization of 2D classifier while building splits of the faces.
7. Post treat stage:
7.1. Method CorrectWires: Save edge's data (PCurve, parameter of the vertex, range) to avoid its recalculation.
7.2. Method CheckEdge: While checking vertices on edges avoid unnecessary calculation of their location.
8. Provide possibility to disable the classification of the input solids on the inverted status (to be the holes in the space).
9. Avoid building of bounding boxes for faces/solids during splitting of the input arguments for their classification relatively hole faces/shells if there are no holes created.
10. Avoid rebuilding of the faces/solids from arguments which does not acquire any inside parts from other arguments during the operation by using their draft versions as their splits.

Test cases for the issue.
Correction of the test cases boolean gdml_public A9 and bugs modalg_7 bug28485 as they are improvements.
Additional test case for the issue #28485 as it is fixed by the current changes.

3 years ago0027784: Thickness fails on cylinder with draft
aml [Tue, 31 Oct 2017 03:43:57 +0000 (06:43 +0300)]
0027784: Thickness fails on cylinder with draft

Incorrect calculation of intersection in 2D space is fixed.

3 years ago0029277: Configuration, Cmake - remove 3rd-party libraries from INTERFACE_LINK_LIBRAR...
ski [Tue, 31 Oct 2017 09:07:18 +0000 (12:07 +0300)]
0029277: Configuration, Cmake - remove 3rd-party libraries from INTERFACE_LINK_LIBRARIES property of installed OCCT targets

3rd-party libraries were removed from INTERFACE_LINK_LIBRARIES properties of OCCT targets.

3 years ago0029249: Configuration - Standard_Failure compilation fails on VS2013 + Intel Compile...
kgv [Tue, 24 Oct 2017 08:21:03 +0000 (11:21 +0300)]
0029249: Configuration - Standard_Failure compilation fails on VS2013 + Intel Compiler due to unavailability of thread_local

Check __INTEL_COMPILER version in combination with _MSC_VER on Windows.

3 years ago0029262: Visualization - AIS_InteractiveContext::Load() does not register Object...
kgv [Wed, 25 Oct 2017 14:27:23 +0000 (17:27 +0300)]
0029262: Visualization - AIS_InteractiveContext::Load() does not register Object in the Viewer

AIS_InteractiveContext::Load() now loads the object regardless specified selection mode and decomposition flag.
AIS_InteractiveContext::Load() and ::KeepTemporary() now register object in the Viewer
in the same way as ::Display() does.

Draw Harness command vdisplay has been extended with new flag -erased to load object into context in erased state.

3 years ago0029250: Configuration, TKIVtk - build failure with VTK 6.3+ due to vtkRenderingFreeT...
apn [Tue, 24 Oct 2017 08:00:54 +0000 (11:00 +0300)]
0029250: Configuration, TKIVtk - build failure with VTK 6.3+ due to vtkRenderingFreeTypeOpenGL being removed

Remove vtkRenderingFreeTypeOpenGL from EXTERNLIB if used vtk version is 6.3.0 and above

3 years ago0029251: Configuration - problem compiling OCCT 7.2 with glibc 2.26
kgv [Sat, 21 Oct 2017 17:46:59 +0000 (20:46 +0300)]
0029251: Configuration - problem compiling OCCT 7.2 with glibc 2.26

Standard_CLocaleSentry does not include xlocale.h when using glibc anymore.
HAVE_XLOCALE_H has been renamed to OCCT_CLOCALE_POSIX2008 to avoid confusion.


3 years ago0029255: Configuration, .gitignore - do not track generated files with extensions...
kgv [Tue, 24 Oct 2017 10:55:48 +0000 (13:55 +0300)]
0029255: Configuration, .gitignore - do not track generated files with extensions VC.db, VC.opendb

3 years ago0029252: Coding rules - eliminate GCC compiler warnings -Wformat-overflow
kgv [Sat, 21 Oct 2017 18:17:14 +0000 (21:17 +0300)]
0029252: Coding rules - eliminate GCC compiler warnings -Wformat-overflow

OSD_DirectoryIterator and OSD_FileIterator now uses TCollection_AsciiString instead of unsafe sprintf.

3 years ago0024574: ICC compiler warnings on Windows
abv [Thu, 19 Oct 2017 14:12:05 +0000 (17:12 +0300)]
0024574: ICC compiler warnings on Windows

NCollection_UtfString and NCollection_UtfIterator classes are refactored to use methods overloading instead of switches to dispatch implementation depending on character (Unicode code unit) size.

ICC-specific preprocessor directives are added to avoid warnings.
Unused local functions and variables, class methods, unreachable statements, and extra throw() declarations reported by ICC are removed.
Usage of "expl" for name of local variable is avoided as it conflicts with standard C function "expl" defined in math.h as preprocessor macro.

Non-standard (MS-specific) argument envp is removed in definition of main() function on Windows.
Functions _main_ and _WinMain_ are renamed to Draw_Main and Draw_WinMain, respectively, to avoid using names reserved in C++.

Doxygen warning is fixed in XDE User's Guide.

3 years ago0029214: Application Framework - TPrsStd_AISPresentation::AISUpdate() should not...
kgv [Thu, 12 Oct 2017 08:31:41 +0000 (11:31 +0300)]
0029214: Application Framework - TPrsStd_AISPresentation::AISUpdate() should not implicitly redraw 3D Viewer

3 years ago0029247: Tests, CheckCommands.tcl - Tcl exception "expected integer but got 3dviewer...
apn [Thu, 19 Oct 2017 09:08:05 +0000 (12:08 +0300)]
0029247: Tests, CheckCommands.tcl - Tcl exception "expected integer but got 3dviewer" in test cases on macOS

Test cases on macOS (using native tcl from /usr/lib) fail when $lst starts with numeral, because upvar guesses that its first parameter is level rather than otherVar.
The fix is to explicitly supply level to 1 (default value if uplevel is omitted) to avoid tcl exception "expected integer but got 3dviewer".

3 years ago0029183: Result of general fuse of shell and edge produces a shape with too large...
ifv [Fri, 6 Oct 2017 11:52:20 +0000 (14:52 +0300)]
0029183: Result of general fuse of shell and edge produces a shape with too large tolerance

Adjusting parameters on approximation (boundary condition) for case when projected curve is far from surface
Test case is created

3 years ago0029157: Modeling - suspicious pass-through of case labels in switch statements
msv [Wed, 18 Oct 2017 16:20:30 +0000 (19:20 +0300)]
0029157: Modeling - suspicious pass-through of case labels in switch statements

Suspicious passes through case labels have been resolved either by using Standard_FALLTHROUGH macro or by redesigning the code.

3 years ago0029182: BOPAlgo_PaveFiller sometimes raises exception in parallel mode
msv [Fri, 6 Oct 2017 14:07:50 +0000 (17:07 +0300)]
0029182: BOPAlgo_PaveFiller sometimes raises exception in parallel mode

Data races have been prevented in the code of BOPAlgo_PaveFiller that makes pcurves of edges on faces. For that:
- Put into treatment only unique edge-face pairs.
- If the same edge is treated with different faces in different threads simultaneously this also causes data races. To avoid this make the edge's copy in each thread and update the copy. The original edge is updated only after finishing parallel processing.

The new method BOPTools_AlgoTools::CopyEdge has been added to make a copy of an edge with vertices.

Big screenshot in the test script tests/bugs/modalg_7/bug28200 has been replaced with a small one.

3 years ago0029186: Move AddTool(), SetTools(), Tools() and other common methods of BOP tools...
oan [Thu, 12 Oct 2017 10:47:00 +0000 (13:47 +0300)]
0029186: Move AddTool(), SetTools(), Tools() and other common methods of BOP tools to separate interface class

Methods AddTool(), SetTools(), Tools() have been moved to BOPAlgo_ToolsProvider class;
BOPAlgo_BOP and BOPAlgo_Splitter are now successors of BOPAlgo_ToolsProvider

3 years ago0029228: Coding Rules - define rule for avoiding header inclusion list pollution
kgv [Fri, 13 Oct 2017 15:49:18 +0000 (18:49 +0300)]
0029228: Coding Rules - define rule for avoiding header inclusion list pollution

3 years ago0029229: Crash at Poly_Triangulation::Normal
dbv [Sat, 14 Oct 2017 15:14:06 +0000 (18:14 +0300)]
0029229: Crash at Poly_Triangulation::Normal

Fixed creation of returning gp_Dir

3 years ago0028763: Projection of a short line segment on a polar surface causes exception
emv [Fri, 22 Sep 2017 07:34:29 +0000 (10:34 +0300)]
0028763: Projection of a short line segment on a polar surface causes exception

Test case for the issue.
The problem has been fixed by the fix for the issue 0028150

3 years ago0029225: Visualization - Font_FTFont::AdvanceX() retrieves kerning value for incorrec...
kgv [Fri, 13 Oct 2017 12:52:51 +0000 (15:52 +0300)]
0029225: Visualization - Font_FTFont::AdvanceX() retrieves kerning value for incorrect characters pair

Fixed FT_Get_Kerning misuse within Font_FTFont::AdvanceX()/Font_FTFont::AdvanceY().
Font_FTFont::loadGlyph() has been corrected to not return TRUE
in case if method called with 0 argument second+ time.

3 years ago0029188: Null shape is produced by 3D offset algorithm (mode="Complete", Join Type...
emv [Fri, 6 Oct 2017 06:35:54 +0000 (09:35 +0300)]
0029188: Null shape is produced by 3D offset algorithm (mode="Complete", Join Type="Intersection")

The following improvements have been made in the 3D offset algorithm for mode "Complete" and Join type "Intersection":
- RemoveInsideFaces() - Removal of the invalid parts outside of the solids built from the splits of offset faces is now performed. It helps to avoid their rebuilding and speed-up the computation.
- FindVerticesToAvoid() - Strengthening the criteria for the vertices to be avoided in the new splits.

Test cases for the issue.
Adjustment of the test cases to current behavior.

3 years agoUpdate of tests database
apv [Thu, 12 Oct 2017 07:57:25 +0000 (10:57 +0300)]
Update of tests database

Test for 0021134: checkshape command does not return faulty shapes.
Test for 0021494: Intersection between cone and sphere fails
Test for 0022642: Refactoring of ShapeFix_Wire::FixIntersectingEdges
Test for 0022732: Reading a STEP file produces invalid shape

3 years ago0029196: Tests - some test cases fail with Intel HD graphics
abv [Mon, 9 Oct 2017 10:19:20 +0000 (13:19 +0300)]
0029196: Tests - some test cases fail with Intel HD graphics

Test bugs vis bug26357 is revised to check colors in more robust way.
In tests bugs vis bug29091_*, size of produced file is checked to be within range of known valid sizes, rather than hard-coded (per-platform) exact sizes.
FPE signals are disabled for test bugs vis bug29127 to prevent exceptions cause by software OpenGl driver.

File data/occ/Box.brep is removed to avoid confusion with file box.brep in public data files used in tests.

Two additional non-alphabetic-order combinations of complex types are handled in RWStepAP214_ReadWriteModule.cxx

3 years ago0029171: Foundation Classes - C signal handler does not work on MinGW
abv [Sat, 7 Oct 2017 16:26:38 +0000 (19:26 +0300)]
0029171: Foundation Classes - C signal handler does not work on MinGW

Setting signal handler is enabled in OSD::SetSignal() for MinGW (works only for SEH builds of MinGW, not for SJLJ builds).

Due to absence of function _set_se_translator(), handler is set using C signal() function and thus is called asynchronously.
Macro OCC_CONVERT_SIGNALS is enabled for MinGW build to support converting signals to C++ exceptions using long jumps (the same as on Linux).
Code raising exceptions in OSD::SetSignal() is corrected to use method Jump() instead of C++ throw.

3 years ago0029158: Visualization - suspicious pass-through of case labels in switch statements
kgv [Wed, 13 Sep 2017 15:12:57 +0000 (17:12 +0200)]
0029158: Visualization - suspicious pass-through of case labels in switch statements

MeshVS_Mesh::ComputeSelection() - fixed incorrect fall-through
from MeshVS_SMF_Mesh to MeshVS_SMF_Group.

3 years ago0029081: With Mingw-w64 Unicode Paths Do Not Work
kgv [Wed, 6 Sep 2017 08:14:53 +0000 (11:14 +0300)]
0029081: With Mingw-w64 Unicode Paths Do Not Work

OSD_OpenStream() now uses __gnu_cxx::stdio_filebuf extension for opening UNICODE files on MinGW when using C++ file streams.
Variant accepting filebuf returns bool (true if succeeded and false otherwise).

Checks of ofstream to be opened made via calls to low-level ofstream::rdbuf() are replaced by calls to ofstream::is_open(); state of the stream is also checked (to be good).
Unicode name used for test file in test bugs fclasses bug22125 is described (for possibility to check it).

3 years ago0029152: Coding Rules - eliminate GCC compiler warnings -Wmisleading-indentation...
kgv [Thu, 28 Sep 2017 18:29:22 +0000 (21:29 +0300)]
0029152: Coding Rules - eliminate GCC compiler warnings -Wmisleading-indentation when using MinGW

OSD_File::myFileHandle/myIO pair is now switched within class definition
instead of myFileHandle_is_only_for_Windows/myFileChannel_is_only_for_Linux macros.

OSD_Directory/OSD_Disk/OSD_File methods implementation of WinAPI/non-WinAPI
has been merged within .cxx file and reformatted.
Resolved several inconsistences on handling invalid input
between WinAPI/non-WinAPI implementations.
NCollection_Array1 is now used in several places instead of
raw memory allocation (HeapAlloc()/HeapFree()/new/delete).
TCollection_ExtendedString is used instead of StringCchCopyW.

Unused field OSD_Disk::myQuotaSize and not implemented methods OSD_Disk::*Quota*() have been removed.

Obsolete code fragment "for Visual Age compiler" is removed.
Macro __try is undefined before redefinition on MinGW to avoid warning.

3 years agoUpdate of tests database
apv [Mon, 2 Oct 2017 12:51:49 +0000 (15:51 +0300)]
Update of tests database

Test for 0028167: Fillet has strange form at its end
Test for 0028168: Exception is raised during Boolean operation
Test for 0028194: BRepMesh - Standard_OutOfRange exception on valid shape
Test for 0028200: Invalid topology of the general fuse result
Test for 0028216: Intersection of a face and a solid is broken up leading to empty result of Boolean common operation
Test for 0028245: Result of Cells Builder algorithm becomes invalid after removal of internal boundaries on faces
Test for 0028247: Effect of minSize parameter of BRepMesh_IncrementalMesh seems to be too strong
Test for 0028274: BOPAlgo_MakerVolume fails to create solid
Test for 0028279: Greater oscillations and higher degree of resulting B-Spline in cylinder-cylinder intersections in 7.1.0
Test for 0028354: Filleting destroys shape if the filleted edge touches a seam edge
Test for 0028366: BOP Common fails with solid and planar face
Test for 0028379: BRepMesh produces mangled mesh for a cone
Test for 0028388: Boolean common of simple face and solid results in empty compound
Test for 0028393: Boolean cut produce wrong result.
Test for 0028402: Modeling - calculation of properties of complex compounds is slow
Test for 0028406: High tolerances after boolean operation on OCCT 6.9.1
Test for 0028694: IGES reader produces too small edge covered by its vertices
Test for 0028719: Mesh - Display issue for special model
Test for 0028108: Sweep fails in "Right Corner" mode
Test for 0028696: Offset 2D algorithm produces micro edges
Test for 0028700: BRepBuilderAPI_Sewing produces too small edge covered by its vertices
Test for 0028722: Conversion of a spherical face to a spline produces an invalid shape
Test for 0028739: Draft on face fails
Test for 0028662: Data Exchange - Exception during reading large STEP file.
Test for 0028768: ShapeFix slighly corrupts the source face
Test for 0028485: Fuse of two shapes using gluing and non-destructive options gives invalid result
Test for 0028500: Artifact in shaded view of the shape
Test for 0028523: Fixshape reverses the orientation of the face
Test for 0028544: Big tolerance value is returned by Implicit-parametric algorithm
Test for 0028571: BRepOffsetAPI_MakeOffset build invalid wire
Test for 0028572: Wrong result of the mkface command
Test for 0028595: Shifting 2d curves for non-periodic surfaces leads to incorrect result.
Test for 0028596: Erroneous fillet
Test for 0028640: BRepBuilderAPI_Transform build invalid shape
Test for 0028656: 3d offset modifies the input shape
Test for 0000077: BUC60866. The BRepFill_Pipe fails.
Test for 0000078: BUC60875. Pipe is created incorrectly on two wires.
Test for 0000082: BUC60908. Incomplete section of shell by plane.
Test for 0000083: BUC60912. Section of simple BSpline surfaces is performed too slow.
Test for 0000862: GeomAPI_ExtremaCurveCurve fails on given curves
Test for 0002442: Error in BRepClass3d_SolidClassifier
Test for 0003936: Face on a surface of extrusion is extended after import to IGES
Test for 0005081: BRepOffset_MakeOffsetShape algorithm does not return valid result.
Test for 0005145: Wrong result of projection of a circle on a sphere.
Test for 0014846: ShapeFix_Wireframe::FixWireGaps() works wrong on a shape with seam edge
Test for 0016424: Exception in reading of attached IGES file
Test for 0010377: Incorrect fuse result when objects tangent by planar face
Test for 0013565: Incorrect result of Common operation
Test for 0013566: Incorrect result of Extrema between a large circle and a line
Test for 0024632: Artifacts during Boolean cut operation on neighboring parts
Test for 0005582: Impossible to make offset on the shape with degenerated edge

3 years ago0029135: 3D Offset algorithm produces a NULL shape
emv [Thu, 28 Sep 2017 08:02:05 +0000 (11:02 +0300)]
0029135: 3D Offset algorithm produces a NULL shape

UpdateValidEdges in BRepOffset_MakeOffset_1 - Perform the filtering of splits of the edges in two stages:
1. Separate filtering of the connected blocks using localized bounding edges taken only from the splits of offset faces from the block;
2. Combined treatment of the remaining splits using bounding edges from the splits of all offset faces.

3 years ago0029124: Visualization, Tests - failures when tests are executed via Remote Desktop...
abv [Tue, 19 Sep 2017 10:04:23 +0000 (13:04 +0300)]
0029124: Visualization, Tests - failures when tests are executed via Remote Desktop connection

Protection against accessing null pointer is added in OpenGl_VertexBuffer::Create().
In command vreadpixel OpenGl-related warnings are redirected to cout so that they do not contaminate the command output.
Test grid caf presentation is configured to ignore OpenGl-related warning messages.
Tests in grid tools are configured to report status SKIPPED if tools are not available (OCCT is built without Qt).

3 years ago0029117: Adding translation of Multileader entity
anv [Thu, 21 Sep 2017 12:15:12 +0000 (15:15 +0300)]
0029117: Adding translation of Multileader entity

Scaling of width of glyphs is added om Font_FTFont class.
Font "SimSan" is specified as fallback alias for font "NSimSan" (contains Chinese hieroglyphs).

3 years ago0029029: Exception Raised with no Result during reading file 2033zsh1_1.stp
gka [Tue, 3 Oct 2017 14:28:32 +0000 (17:28 +0300)]
0029029: Exception Raised with no Result during reading file 2033zsh1_1.stp

Static function stepstrcmp() in StepData_StepReaderData.cxx is replaced by plain strcmp() to have the full strings compared (stepstrcmp() returned true if one of strings was longer than the other but common part was equal).
Protection is added to avoid exception for cases when representation entities are NULL

3 years ago0028245: Result of Cells Builder algorithm becomes invalid after removal of internal...
imn [Wed, 12 Jul 2017 07:52:41 +0000 (10:52 +0300)]
0028245: Result of Cells Builder algorithm becomes invalid after removal of internal boundaries on faces

When removing internal boundaries between faces of the same material do it once for the whole shape, and at this keep boundaries between areas with different material.

3 years ago0029184: Visualization - DrawWindow::Save() fails when using WinCodec with PNG codec
kgv [Wed, 13 Sep 2017 15:12:57 +0000 (17:12 +0200)]
0029184: Visualization - DrawWindow::Save() fails when using WinCodec with PNG codec

DrawWindow::Save() now uses Image_Format_BGR instead of Image_Format_BGR32
for better compatibility with image encoders when dumping WinAPI bitmap.

3 years ago0029109: Visualization, AIS_Trihedron - add option hiding arrows tips
Zia ul Azam [Wed, 13 Sep 2017 15:12:57 +0000 (17:12 +0200)]
0029109: Visualization, AIS_Trihedron - add option hiding arrows tips

Prs3d_DatumAspect::DrawDatumPart() now handles new flag ToDrawArrows().

vtrihedron - added new option -arrowTip.
Fixed misprint in command description -arrowscolor -> -arrowcolor.

3 years ago0028909: Algorithm of BO is stuck while fusing shell and edges
ifv [Tue, 11 Jul 2017 07:15:28 +0000 (10:15 +0300)]
0028909: Algorithm of BO is stuck while fusing shell and edges

Approximation parameters: degmin, degmax, max number of segments, boundary condition, maximal distance of projecting are added in interface of classes ProjLib_ProjectedCurve, ProjLib_ComputeApprox,
Algorithm of Approx/Approx_ComputeCLine is modified in order to treat maximal number of segments allowed for cutting.
Algorithm of method BOPTools_AlgoTools2D::MakePCurveOnFace(...) is modified in order to manage cases with big edge tolerances.
Test case added
Some test cases were modified according to new behavior of algorithms

3 years ago0029176: Exception while projection 2D-point on 2D-line
nbv [Tue, 3 Oct 2017 07:30:47 +0000 (10:30 +0300)]
0029176: Exception while projection 2D-point on 2D-line

DRAW-command "2dproj" has been changed. Now, it can return not only 2D-line as extrema but 2D-point.

Test case has been updated.

3 years ago0029160: Coding - AppleCLang 9 compiler warning "binding dereferenced null pointer...
abv [Fri, 29 Sep 2017 08:18:11 +0000 (11:18 +0300)]
0029160: Coding - AppleCLang 9 compiler warning "binding dereferenced null pointer to reference has undefined behavior"

Avoid returning reference to null in AppDef_MyLineTool