occt.git
5 years ago0028123: Boolean Cut produces faulty shape CR0-TEST-WO-29711
emv [Mon, 16 Jul 2018 12:45:02 +0000 (15:45 +0300)]
0028123: Boolean Cut produces faulty shape

Test case for the issue as it is not reproduced anymore.

5 years ago0028113: BOPAlgo_Builder produces invalid shape
emv [Tue, 17 Jul 2018 09:07:59 +0000 (12:07 +0300)]
0028113: BOPAlgo_Builder produces invalid shape

Boolean operations - avoid fast rebuilding of the face in case its edges have been unified during the intersection. Proceed with the BuilderFace algorithm in this case.
Test case for the issue.

5 years ago0029968: Modeling Algorithms - BRepAlgoAPI_Cut produces an empty result with half...
emv [Wed, 18 Jul 2018 12:31:52 +0000 (15:31 +0300)]
0029968: Modeling Algorithms - BRepAlgoAPI_Cut produces an empty result with half-space

IntTools_Context::IsInfiniteFace() - Make the conclusion about infinity of the face basing on its bounding box only. No need to check NaturalRestriction flag.
Test case for the issue.

5 years ago0029830: STEPCAFControl_Reader poor performance - quadratic dependence
msv [Fri, 1 Jun 2018 11:38:39 +0000 (14:38 +0300)]
0029830: STEPCAFControl_Reader poor performance - quadratic dependence

Various performance improvements in the algorithms of STEP read/write:

- Using the call to XCAFDoc_ShapeTool::SearchUsingMap() in STEPCAFControl_Reader::ReadColors() in order to quickly find the label of the shape.
- Place the label of a component to the map in order to avoid iteration on all components to find a label of a component shape. It is done both in XCAFDoc_ShapeTool and in STEPCAFControl_Reader.
- Using the map ShapeLabelMap in the method STEPCAFControl_Reader::FindInstance() to avoid iteration on components.
- Move invariant FindEntities() out of the loop in the method getStyledItem in STEPCAFControl/STEPCAFControl_Writer.cxx.
- Small fixes connected with elimination of excess copying of handles, calls of handle DownCasts and so on.
- Add pointer to the end of binders chain to the Transfer_Binder class in order to speed up adding a binder to the chain.

Remove stack overflow during destruction of STEP model with long chains of Transfer_Binder.

Make it possible to use the Draw commands ReadStep and WriteStep to read/write from the session without accessing the disk file (use '.' for the file name).

Add performance test cases for STEP reading/writing.

5 years ago0029769: Uninitialized data with BSplCLib_Cache, BSplSLib_Cache
abv [Sun, 10 Jun 2018 19:40:12 +0000 (22:40 +0300)]
0029769: Uninitialized data with BSplCLib_Cache, BSplSLib_Cache

Implementation of classes BSplCLib_Cache and BSplSLib_Cache is revised:
- Common functionality dealing with spans along one parametric direction is separated to new struct BSplCLib_CacheParams
- Empty constructors are removed; copying is prohibited
- Code reconsidering degree and other parameters on each call to BuildCache() is eliminated; curve parameters must be the same in constructor and all calls to BuildCache()
- Extra call to BuildCache() from constructor is eliminated

5 years ago0025852: Visualization - Font_BRepFont produces bad faces for circled symbols
skl [Thu, 12 Jul 2018 06:47:02 +0000 (09:47 +0300)]
0025852: Visualization - Font_BRepFont produces bad faces for circled symbols

Font_BRepFont has been modified to use dedicated algorithm
for ordering of wires based on wire classification,
analysis of internal zones and creation few result faces (if it is needed)
instead of relying on ShapeFix.

TKService dependency from TKShHealing has been dropped.

5 years ago0029861: Modeling Algorithms - BiTgte_Blend gives exception
jgv [Fri, 15 Jun 2018 17:22:38 +0000 (20:22 +0300)]
0029861: Modeling Algorithms - BiTgte_Blend gives exception

The algorithm has been corrected to avoid exception.
Now both Draw commands "brollingball" and "rollingball" give correct results.
Test cases "offset faces_type_a A3, A4, A9" are good now.

5 years ago0029959: Coding - new warnings after integration 29846
gka [Mon, 16 Jul 2018 09:08:00 +0000 (12:08 +0300)]
0029959: Coding - new warnings after integration 29846

Erroneous declaration of label was removed

5 years ago0027230: Boolean operations fails on attached shapes
emv [Mon, 16 Jul 2018 10:03:17 +0000 (13:03 +0300)]
0027230: Boolean operations fails on attached shapes

Test case for the issue as it is not reproduced anymore.

5 years ago0029961: Foundation Classes - NCollection_Shared constructor passes arguments by...
kgv [Sun, 15 Jul 2018 08:35:12 +0000 (11:35 +0300)]
0029961: Foundation Classes - NCollection_Shared constructor passes arguments by copy

Added more NCollection_Shared constructors taking arguments by reference.

5 years ago0029960: Foundation Classes - disarm OSD_ThreadPool copy constructor
kgv [Sat, 14 Jul 2018 16:17:26 +0000 (19:17 +0300)]
0029960: Foundation Classes - disarm OSD_ThreadPool copy constructor

Private copy constructors have been added to OSD_ThreadPool
and Standard_Condition to protect misuse.

5 years ago0029846: Data Exchange - STEP validation properties support is obsolete
gka [Thu, 5 Jul 2018 16:10:31 +0000 (19:10 +0300)]
0029846: Data Exchange - STEP validation properties support is obsolete

Name of the PROPERTY_DEFINITION entity for validation properties is changed to "geometric validation property" according to the current version of the recommended practices.
For compatibility with older definitions (with underscores) the check is made using string with underscores replaced by spaces.

5 years ago0029817: Documentation - avoid excessive use of "please"
ysn [Mon, 9 Jul 2018 10:53:01 +0000 (13:53 +0300)]
0029817: Documentation - avoid excessive use of "please"

"Please" is now used in the documentation only in sensitive cases.

5 years ago0029873: Data Exchange - STEPCAFControl_Reader::Transfer() returns TRUE even when...
skl [Tue, 10 Jul 2018 11:45:11 +0000 (14:45 +0300)]
0029873: Data Exchange - STEPCAFControl_Reader::Transfer() returns TRUE even when external files are missing

Output information about failure during reading of external file has been added (it can be checked by using Draw command "tpstat").
If the reader cannot find an external file at the first found location
(using entity APPLIED_EXTERNAL_IDENTIFICATION_ASSIGNMENT) it checks other information about file path
(using entity DOCUMENT_FILE related with the above-mentioned entity) and uses a new location for import of the external file.

5 years ago0029935: Foundation Classes - introduce OSD_ThreadPool class defining a thread pool
kgv [Fri, 6 Jul 2018 23:27:51 +0000 (02:27 +0300)]
0029935: Foundation Classes - introduce OSD_ThreadPool class defining a thread pool

New class OSD_ThreadPool has been introduced to define a Thread Pool for multi-threading algorithm.
Thread Pool assigns a serial number for each thread allowing Multi-Threading algorithm to allocate thread-local storage variables as an array whose size is the same as the number of threads.

OSD_ThreadPool also redirects exceptions to a thread calling parallel execution and consistently initializes FPE exception handling.

New class Standard_Condition provides a platform-independent  tool similar to Event in WinAPI.

A new auxiliary function Standard_Atomic_CompareAndSwap() has been introduced
for performing atomic compare and swap of integer number.
Standard_Atomic_Increment/Standard_Atomic_Decrement fallback implementation
using ASM code for x86 processors for GCC has been dropped;
instead, it is expected that GCC should be properly configured targeting modern x86 architectures.

OSD_Signal now declares fFltExceptions as thread_local variable accessible through OSD::ToCatchFloatingSignals() property.
Standard_THREADLOCAL macro (wrapping thread_local attribute) has been moved to public header Standard_Macro.hxx.

OSD_Parallel::ForEach() has been extended with new optional parameter theNbItems and uses OSD_ThreadPool::DefaultPool().

5 years ago0029425: Visualization - AIS_InteractiveContext::SetAutomaticHilight() has no affect
kgv [Thu, 21 Jun 2018 09:19:08 +0000 (12:19 +0300)]
0029425: Visualization - AIS_InteractiveContext::SetAutomaticHilight() has no affect

AIS_InteractiveContext::MoveTo() and ::Select() now do not highlight entities
when ::AutomaticHilight() option is turned off.

vselprops command has been extended with new arguments -autoHighlight and -highlightSelected
(vhighlightselected command is now deprecated).

5 years ago0029945: Export body of revolution to STEP breaks the shape
ika [Wed, 11 Jul 2018 14:29:34 +0000 (17:29 +0300)]
0029945: Export body of revolution to STEP breaks the shape

Now vertex loops on surfaces of revolution are imported from STEP format in the same way as on spheres and b-spline surfaces: in case of a single vertex loop neutral bounds will be used.

5 years ago0029936: Documentation - improve description of AIS_Animation class
kgv [Sat, 7 Jul 2018 01:08:57 +0000 (04:08 +0300)]
0029936: Documentation - improve description of AIS_Animation class

5 years ago0029947: Visualization - SelectMgr_SelectableObject::GetSelectPresentation() should...
kgv [Wed, 11 Jul 2018 12:52:02 +0000 (15:52 +0300)]
0029947: Visualization - SelectMgr_SelectableObject::GetSelectPresentation() should copy object clipping planes

Methods SelectMgr_SelectableObject::GetHilightPresentation(), ::GetSelectPresentation() and ::UpdateClipping()
have been corrected to copy clipping planes of the object into highlighting and selection presentations.

5 years ago0029948: Uninitialized variable in GeomEvaluator_OffsetSurface::CalculateD0(...)...
nbv [Wed, 11 Jul 2018 14:29:54 +0000 (17:29 +0300)]
0029948: Uninitialized variable in GeomEvaluator_OffsetSurface::CalculateD0(...) method

Variable "NStatus" from GeomEvaluator_OffsetSurface class is now initialized correctly.

5 years ago0029866: Intersector returns two overlapped curves as a result
nbv [Tue, 3 Jul 2018 15:14:30 +0000 (18:14 +0300)]
0029866: Intersector returns two overlapped curves as a result

Since now a new WLine is not created if its start point lies in another earlier computed WLine. It allows avoiding creation of duplicate WLines in the intersection result.

<!break>

1. Methods IsOutSurf1Box(...), IsOutSurf2Box(...), IsOutBox(...) for classes IntSurf_LineOn2S and IntPatch_RLine have been created.

5 years ago0029939: Modeling Algorithms - add NULL check to BRepGProp_Face::Load()
kgv [Wed, 4 Jul 2018 09:09:50 +0000 (12:09 +0300)]
0029939: Modeling Algorithms - add NULL check to BRepGProp_Face::Load()

BRepGProp_Face::Load() has been protected against crash in case of edges without p-curves.

5 years ago0029944: Porting to VC 2017 : Regressions on the gcc (Debian 4.9.2-10) 4.9.2. Excepti...
gka [Tue, 10 Jul 2018 16:20:50 +0000 (19:20 +0300)]
0029944: Porting to VC 2017 : Regressions on the gcc (Debian 4.9.2-10) 4.9.2. Exception is raised during load STEP file.

Boolean parameter "same_sence" from class RWStepShape_RWAdvancedFace.cxx is now initialized correctly.

5 years ago0029942: Porting to Debian80-64 : Regressions in Modeling Algorithms after integratio...
nbv [Tue, 10 Jul 2018 09:50:03 +0000 (12:50 +0300)]
0029942: Porting to Debian80-64 : Regressions in Modeling Algorithms after integration fix for 29807

Correction in the test case perf/modalg/bug27021.

5 years ago0029916: Regressions in Data Exchange on VC 2017
gka [Wed, 4 Jul 2018 16:45:58 +0000 (19:45 +0300)]
0029916: Regressions in Data Exchange on VC 2017

The "end" file from folder "de" has been corrected to exclude comparing the number of edges in the shapes from current result with test data.
"TODO messages generated for differences were removed.
Data exchange tests were regenerated.

5 years ago0029934: Visualization - handle properly orthogonal clipping plane within SelectMgr_R...
kgv [Fri, 6 Jul 2018 16:04:49 +0000 (19:04 +0300)]
0029934: Visualization - handle properly orthogonal clipping plane within SelectMgr_RectangularFrustum::computeClippingRange()

SelectMgr_RectangularFrustum now considers the direction of clipping plane orthogonal to picking ray.

5 years ago0029823: Visualization, TKOpenGl - highlighting by Bounding Box mistreats Local Trans...
mnv [Mon, 2 Jul 2018 08:21:35 +0000 (11:21 +0300)]
0029823: Visualization, TKOpenGl - highlighting by Bounding Box mistreats Local Transformation

The problem when object transformation was applied to the bounding box twice has been fixed in OpenGl_Structure.
OpenGl_ShaderManager has been extended with GLSL program drawing bounding box.
OpenGl_Structure does not create extra Groups for drawing bounding box.

5 years ago0029874: Visualization - capping plane transparency does not work
kgv [Fri, 15 Jun 2018 16:39:31 +0000 (19:39 +0300)]
0029874: Visualization - capping plane transparency does not work

Interface OpenGl_RenderFilter and its implementations have been merged into OpenGl_Workspace.
This limits flexibility of interface, but simplifies logic.
- OpenGl_RenderFilter_FillModeOnly flag replaces OpenGl_CappingAlgoFilter;
- OpenGl_RenderFilter_NonRaytraceableOnly flag replaces OpenGl_RaytraceFilter;
- OpenGl_RenderFilter_OpaqueOnly flag replaces OpenGl_OpaqueFilter;
- OpenGl_RenderFilter_TransparentOnly flag replaces OpenGl_TransparentFilter.

OpenGl_CappingAlgo now:
- avoids redundant Stencil clearing when Layer contains at least one transparent Element;
- renders semitransparent capping plane within transparent elements pass.

vclipplane command has been extended by new argument -transparency.

5 years ago0029929: Visualization, V3d_View - make method ::FitMinMax() public
kgv [Thu, 5 Jul 2018 09:54:05 +0000 (12:54 +0300)]
0029929: Visualization, V3d_View - make method ::FitMinMax() public

5 years ago0029928: Regressions: a lot of warnings produced during generation Overview by Doxygen
apn [Wed, 4 Jul 2018 10:17:51 +0000 (13:17 +0300)]
0029928: Regressions: a lot of warnings produced during generation Overview by Doxygen

Warnings in upgrade.md have been eliminated.

5 years ago0029927: Coding Rules - eliminate GCC compiler warnings -Wmaybe-uninitialized in...
kgv [Wed, 4 Jul 2018 07:49:44 +0000 (10:49 +0300)]
0029927: Coding Rules - eliminate GCC compiler warnings -Wmaybe-uninitialized in BRepApprox

The code has been fixed to avoid taking a reference from a field of a temporary variable
(BRepApprox_ApproxLine::Point() returns non-reference structure).

5 years ago0029834: Coding Rules - eliminate GCC compiler warnings -Wformat-overflow in IGESData...
abv [Tue, 3 Jul 2018 21:54:46 +0000 (00:54 +0300)]
0029834: Coding Rules - eliminate GCC compiler warnings -Wformat-overflow in IGESData_IGESWriter

String buffers have been increased to be sufficient to accommodate result of sprintf() for any possible printed integer values.

5 years ago0029868: Draw Harness - help message for readstl command is unclear
kgv [Thu, 14 Jun 2018 15:45:58 +0000 (18:45 +0300)]
0029868: Draw Harness - help message for readstl command is unclear

readstl syntax has been modified, so that it creates a single-face triangulation by default.
The argument "trinagulation" is no more supported.
The new argument "-brep" has been introduced to generate
a compound of per-triangle faces instead (old default behavior of the command).

5 years ago0029909: Coding - GCC compiler warnings in Qt headers while building Samples in OCCT...
kgv [Tue, 3 Jul 2018 11:31:40 +0000 (14:31 +0300)]
0029909: Coding - GCC compiler warnings in Qt headers while building Samples in OCCT and Products on Debian80 64 bit

Standard_WarningsDisable.hxx has been extended by ignoring -Wunused-local-typedefs within GCC compiler.

5 years ago0029807: [Regression to 7.0.0] Impossible to cut cone from prism
nbv [Fri, 25 May 2018 08:05:58 +0000 (11:05 +0300)]
0029807: [Regression to 7.0.0] Impossible to cut cone from prism

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

<!break>

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

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

3. Test cases have been created.

4. Procedure of trimming IntAna_Curve has been improved.

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

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

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

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

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

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

Correction in test cases.

5 years ago0029925: Foundation Classes - add missing cast to LowerCase() and UpperCase() arguments
kgv [Mon, 2 Jul 2018 21:45:41 +0000 (00:45 +0300)]
0029925: Foundation Classes - add missing cast to LowerCase() and UpperCase() arguments

Argument of LowerCase() and UpperCase() is cast to int via unsigned char to avoid passing negative integer in the case if the argument char is in the extended part of ASCII table (which would result in undefined behavior according to C++ standard).

5 years ago0029924: Configuration - remove obsolete folders from .gitignore
abv [Mon, 2 Jul 2018 15:48:44 +0000 (18:48 +0300)]
0029924: Configuration - remove obsolete folders from .gitignore

Folders drv, sun, sil and ao1 have been removed from .gitignore

5 years ago0029857: Incorrect behavior of Point-Curve Extrema
ifv [Tue, 26 Jun 2018 11:54:35 +0000 (14:54 +0300)]
0029857: Incorrect behavior of Point-Curve Extrema

Check for double solutions has been  added in algorithms Extrema_ELPCOfLocateExtPC and  Extrema_ELPCOfLocateExtPC2d
that find extrema between point and curve.

5 years ago0029894: Visualization - AIS_ColorScale ColorRange returns only min color
mnv [Tue, 26 Jun 2018 13:27:12 +0000 (16:27 +0300)]
0029894: Visualization - AIS_ColorScale ColorRange returns only min color

A typo in AIS_ColorScale::ColorRange function has been fixed.

5 years ago0029890: Data Exchange - Problem in Expand Compounds
ika [Thu, 21 Jun 2018 12:57:39 +0000 (15:57 +0300)]
0029890: Data Exchange - Problem in Expand Compounds

XCAFDoc_ShapeTool::Expand method now avoids creation of subshapes under assemblies.
XCAFDoc_ShapeTool::AddSubShape now avoids creation subshapes under all labels except for simple parts.

5 years ago0029888: [Regression] Data Exchange - XCAFDoc_Editor::Expand() removes names
ika [Thu, 21 Jun 2018 14:21:02 +0000 (17:21 +0300)]
0029888: [Regression] Data Exchange - XCAFDoc_Editor::Expand() removes names

XCAFDoc_Editor::Expand function avoids expanding references to assemblies.

5 years ago0029887: Wrong result of CUT operation due to incorrect point-face classification
nbv [Wed, 20 Jun 2018 14:30:00 +0000 (17:30 +0300)]
0029887: Wrong result of CUT operation due to incorrect point-face classification

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

5 years ago0029885: Visualization, AIS_Manipulator - Translation is done in wrong direction...
kgv [Wed, 20 Jun 2018 11:56:09 +0000 (14:56 +0300)]
0029885: Visualization, AIS_Manipulator - Translation is done in wrong direction depending on Camera orientation

AIS_Manipulator::ObjectTransformation() has ceased to use undefined point for myStartPick initialization within AIS_MM_Translation mode.
AIS_MM_Scaling cas has been merged into AIS_MM_Translation to reduce code duplication.
Extrema_ExtElC/IntAna_IntConicQuad are now used directly instead of more generic GeomAPI_ExtremaCurveCurve/GeomAPI_IntCS.

5 years ago0025467: Visualization - Possibility to remove AIS_LocalContext class
kgv [Wed, 20 Jun 2018 05:48:40 +0000 (08:48 +0300)]
0025467: Visualization - Possibility to remove AIS_LocalContext class

Deprecated functionality related to Local Context has been removed.

The following classes and types have been removed:
AIS_LocalContext, AIS_DataMapOfILC, AIS_LocalStatus, SelectMgr_SequenceOfSelector, SelectMgr_DataMapOfObjectSelectors.
Value AIS_DS_Temporary has been removed from enumeration AIS_DisplayStatus.

The following methods related to Local Context have been removed from AIS_InteractiveContext:
::HasOpenedContext(), ::HighestIndex(), ::LocalContext(),
::LocalSelector(), ::OpenLocalContext(), ::CloseLocalContext(),
::IndexOfCurrentLocal(), ::CloseAllContexts(), ::ResetOriginalState(),
::ClearLocalContext(), ::UseDisplayedObjects(), ::NotUseDisplayedObjects(),
::SetShapeDecomposition(), ::SetTemporaryAttributes(),
::ActivateStandardMode(), ::DeactivateStandardMode(), ::KeepTemporary(),
::SubIntensityOn(), ::SubIntensityOff(),
::ActivatedStandardModes(), ::IsInLocal()
and ::AddOrRemoveSelected() taking TopoDS_Shape.

The methods AIS_InteractiveContext::Display() and ::Load()
taking argument theToAllowDecomposition have been marked as deprecated;
the new methods without this unused argument should be used instead.
AIS_InteractiveContext::DisplayedObjects() - removed argument theOnlyFromNeutral.

SelectMgr_SelectionManager has been simplified so that now it support only single Viewer Selector.

The argument -local has been removed from Draw Harness commands vselmode, vremove, vdisplay, vdisplayall, verase.

5 years ago0029882: Visualization - AcceptDisplayMode() should be redefined in class AIS_Texture...
kgv [Tue, 19 Jun 2018 07:48:50 +0000 (10:48 +0300)]
0029882: Visualization - AcceptDisplayMode() should be redefined in class AIS_TexturedShape

5 years ago0029734: Modeling Algorithms - Compute global properties of tessellated shape
ifv [Thu, 17 May 2018 12:38:17 +0000 (15:38 +0300)]
0029734: Modeling Algorithms - Compute global properties of tessellated shape

New algorithms calculating global properties on mesh data have been added:
- BRepGProp_MeshCinert computes the global properties of polylines represented by a set of points;
- BRepGProp_MeshProps computes the global properties of a surface mesh.

Existing tool BRepGProp now automatically uses new algorithm for triangulation-only faces.
By default, algorithm will use exact geometry objects (surfaces), when it is available (as before the patch);
this behavior can be switched by a new flag UseTriangulation, forcing usage of triangulation instead of exact geometry when both defined.

5 years ago0029859: Coding - GCC compiler warning -Warray-bounds in IntPatch_InterferencePolyhed...
abv [Tue, 12 Jun 2018 19:18:34 +0000 (22:18 +0300)]
0029859: Coding - GCC compiler warning -Warray-bounds in IntPatch_InterferencePolyhedron.cxx

Code is corrected to avoid copying uninitialized element of the array

5 years ago0029847: Visualization, Image_Diff - Tolerance is not effective for 24/32bit image...
dipts [Wed, 6 Jun 2018 19:42:11 +0000 (22:42 +0300)]
0029847: Visualization, Image_Diff - Tolerance is not effective for 24/32bit image formats

Image_Color - removed semibroken summ/difference operators.
Image_Diff now uses signed integer for computing differnce between ubyte3 components;
properly compare squared tolerance.

Image_Diff - dropped declaration of Image_ColorXXX24.
RGB color difference is now computed using Chebyshev distance instead of Euclidean distance
Image_PixMap - added methods RawValue()/ChangeRawValue() returning a pointer
to image where specified pixel data is defined.

5 years ago0029845: UnifySameDomain does not account the accumulated angle when unifying edges
emv [Thu, 7 Jun 2018 05:29:21 +0000 (08:29 +0300)]
0029845: UnifySameDomain does not account the accumulated angle when unifying edges

Check that the accumulated angle does not exceed the given angular tolerance while merging the linear edges.
Test case for the issue.

5 years ago0029841: Test cases from tools/dfbrowser periodically hang on Windows platform
nds [Thu, 7 Jun 2018 12:59:03 +0000 (15:59 +0300)]
0029841: Test cases from tools/dfbrowser periodically hang on Windows platform

Do not start new thread if the thread has been already started. Restart the thread after the previous has finished.

5 years ago0029825: Foundation Classes, NCollection_Vec4 - workaround gcc optimizer issues with...
kgv [Wed, 30 May 2018 15:59:12 +0000 (18:59 +0300)]
0029825: Foundation Classes, NCollection_Vec4 - workaround gcc optimizer issues with xyz() method

Methods of NCollection_Vec3 and NCollection_Vec3 that returned reference to internal buffer as vector of lower dimension (non-const xy(), xyz() etc.) are eliminated.
Use of these methods could led to generation of incorrect binary code by GCC.
Instead added new method SetValues() accepting vector of lower dimension and additional value.

DRAW test command QANColTestVec4 reproducing one situation where the bug occurs is added, along with a test case.

5 years ago0029826: Tests - test case perf/fclasses/strtod is unstable
abv [Tue, 5 Jun 2018 13:38:45 +0000 (16:38 +0300)]
0029826: Tests - test case perf/fclasses/strtod is unstable

Check of performance of accelerated Strtod() vs. standard strtod() in the test is relaxed: now test passes if Strtod() is at least 1.5 times faster

5 years ago0029116: [Regression] OCAF attributes insertion order is violated again
vro [Mon, 4 Jun 2018 07:47:18 +0000 (10:47 +0300)]
0029116: [Regression] OCAF attributes insertion order is violated again

5 years ago0029814: Modeling Data - add method TopoDS_Shape::NbChildren() for simple check of...
kgv [Thu, 24 May 2018 15:06:37 +0000 (18:06 +0300)]
0029814: Modeling Data - add method TopoDS_Shape::NbChildren() for simple check of sub-shapes number

TopoDS_Shape::NbChildren() - new method has been introduced.
TopoDS_Shape.lxx, TopoDS_TShape.lxx - inline methods have been moved to hxx.
TopoDS_TShape_Flags has been redeclared as enumeration instead of preprocessor macros.

Cyclic dependency between headers TopoDS_Shape.hxx and TopoDS_TShape.hxx eliminated.

Places where TopoDS_Iterator is used only for calculation of number of sub-shapes are updated to use NbChildren() instead

5 years ago0029511: Section fails for these two faces
jgv [Fri, 30 Mar 2018 10:30:44 +0000 (13:30 +0300)]
0029511: Section fails for these two faces

Modified method: Approx_ComputeLine::Compute
Check of multicurve is now always unconditional, the procedure of check is modified to avoid infinite loops.

Modified classes: GeomLib_CheckBSplineCurve and GeomLib_Check2dBSplineCurve
Correction of poles at the ends of curve is modified to fit the direction of tangent defined by two first points or two last points of walking line.

Also modified:

BOPAlgo_PaveFiller: modified methods PostTreatFF, RemoveUsedVertices - now unused vertices are included in the list of vertices to be absorbed by other ones.

5 years ago0029813: BRepFill_PipeShell algorithm produces invalid result
jgv [Mon, 28 May 2018 16:13:01 +0000 (19:13 +0300)]
0029813: BRepFill_PipeShell algorithm produces invalid result

Modified method: BRepFill_Sweep::BuildShell
case of global closedness of path containing several C0-connected parts is corrected.

5 years ago0029821: Data Exchange - Wrong processing of subshapes in ShapeTool
ika [Mon, 28 May 2018 10:15:37 +0000 (13:15 +0300)]
0029821: Data Exchange - Wrong processing of subshapes in ShapeTool

Improve FindSubShape method in ShapeTool.
Update Expand compounds, according to changes in ShapeTool.
Add FindSubShape and AddSubShape commands for Draw.

5 years ago0028747: Incorrect result of the section operation after edge refinement
emv [Thu, 26 Apr 2018 11:35:35 +0000 (14:35 +0300)]
0028747: Incorrect result of the section operation after edge refinement

Implementation of the method for simplification of the result of Boolean Operation on the API level.
The method BRepAlgoAPI_BuilderAlgo::SimplifyResult has been added, so the derived classes such as BooleanOpeation and Splitter can also use this method.
The result shape simplification should be called after the operation is done. The simplification is performed by the means of ShapeUpgrade_UnifySameDomain algorithm.

Draw command "bsimplify" has been added to control the simplification options.
Documentation for new functionality and draw commands controlling the options of Boolean operations.
Test cases for the new functionality.

Side-effect change:
The algorithms in Boolean component have been changed to use the BRepTools_History as a History tool.
Now it became possible to disable the collection of shapes modifications during Boolean Operations, which may be useful for performance sake (in draw the option is controlled by *setfillhistory* command).
Draw command "unifysamedom" has been changed to accept the angular tolerance in degrees instead of radians.

5 years ago0029694: Geom2dGcc_Circ2dTanCenGeo crash
nbv [Tue, 22 May 2018 09:15:27 +0000 (12:15 +0300)]
0029694: Geom2dGcc_Circ2dTanCenGeo crash

The reason of this issue is in incorrectly interpreted the generic class "TheExtPC" (eliminated after the fix 0024773) as Extrema_ExtPC2d. Correct interpretation must be "Extrema_EPCOfExtPC2d" class.

Now this problem has been fixed in the class Geom2dGcc_Circ2dTanCenGeo.

New testgrid "lowalgos 2dgcc" has been created.

5 years ago0029712: Extrema algorithm raises exception
nbv [Fri, 20 Apr 2018 14:00:48 +0000 (17:00 +0300)]
0029712: Extrema algorithm raises exception

1. Extrema algorithm calls Curve-surface intersector. This intersector returns flag about infinite solution (in spite of extrema's returning not-parallel status correctly - axes of considered cylinder and circle are not parallel). In this case, attempt of obtaining number of intersection points leads to exception.

So, the fix adds check of infinite solution after the intersection algorithm.

2. The methods IsDone(), IsParallel(), NbExt(), SquareDistance() and Points() (of Extrema_* classes) have been corrected to make them consistent to the documentation.

3. Revision of some Extrema_* classes has been made in order to avoid places with uninitialized variables.

4. Currently Extrema does not store any points in case when the arguments are parallel. It stores the distance only.

5. Some cases on Extrema-algo have been moved from "fclasses"-group to "modalg"-group.

5 years ago0029777: Foundation Classes - The methods of moving of one NCollection_Sequence to...
msv [Wed, 23 May 2018 07:18:49 +0000 (10:18 +0300)]
0029777: Foundation Classes - The methods of moving of one NCollection_Sequence to another are unsafe

Make the methods Append, Prepend, InsertBefore and InsertAfter, which take another sequence as an argument, copying the sequence instead of joining if the allocators are different.

Add test cases for collection classes.

5 years ago0029691: Constructor of BRepFill_MultiLine adjusts V-parameter of sphere in the range...
nbv [Thu, 12 Apr 2018 08:10:22 +0000 (11:10 +0300)]
0029691: Constructor of BRepFill_MultiLine adjusts V-parameter of sphere in the range -PI/2 ... +PI/2 mistakenly.

1. Sphere is not V-periodic surface. See the section "Periodicity concept" written in frame of the fix 0029115. Therefore, applying ElCLib::AdjustPeriodic(...) method is unreasonable.

2. Default U work range of sphere is 0 ... 2*PI. Therefore, adjusting U parameter in range -PI/2 ... +PI/2 is unreasonable.

5 years ago0029586: Revolution creates solid with degenerated edges shared among faces
ifv [Fri, 23 Mar 2018 08:33:27 +0000 (11:33 +0300)]
0029586: Revolution creates solid with degenerated edges shared among faces

Substitution of degenerated edges by their copies is implemented to make edges not shared
in BRepPrimAPI_MakeRevol::Build().
Method BRepPrimAPI_MakeRevol::Generated(...) is changed in order to use history of substitutions.
Test case is added.

5 years ago0029719: Modeling Algorithms - GeomPlate_BuildPlateSurface has no progress informatio...
drazmyslovich [Mon, 23 Apr 2018 14:52:43 +0000 (17:52 +0300)]
0029719: Modeling Algorithms - GeomPlate_BuildPlateSurface has no progress information and is not abortable

The Message_ProgressIndicator handle is added as a parameter to the function LU_Decompose and the the member functions of math_Gauss, Plate_Plate and GeomPlate_BuildPlateSurface classes.

5 years ago0029729: Visualization, Graphic3d_ClipPlane - add support of clipping plane chains
kgv [Fri, 25 May 2018 07:13:38 +0000 (10:13 +0300)]
0029729: Visualization, Graphic3d_ClipPlane - add support of clipping plane chains

Graphic3d_ClipPlane now can define a Chain of Planes (logical AND).
OpenGl_ShaderManager - added new GLSL sub-programs for clipping plane chains.
Bnd_Range::TrimFrom(), ::TrimTo() - added auxiliary methods for trimming the range.
SelectMgr_ViewClipRange now handles non-continuous clipping range.
Graphic3d_SequenceOfHClipPlane now aggregates NCollection_Sequence instead of inheritance.
OpenGl_CappingPlaneResource - triangulation has been adjusted to make front face following CCW order.

5 years ago0029765: BOPTools_AlgoTools::MakeSplitEdge Creates Illegal Edge
Benjamin Bihler [Tue, 15 May 2018 07:43:57 +0000 (10:43 +0300)]
0029765: BOPTools_AlgoTools::MakeSplitEdge Creates Illegal Edge

Method MakeSplitEdge checks arguments order.
This makes the method more generic.
Taking vertex orientation into account.

5 years ago0029520: Visualization - drop deprecated V3d_View::Export() functionality and depende...
kgv [Tue, 20 Feb 2018 05:37:38 +0000 (08:37 +0300)]
0029520: Visualization - drop deprecated V3d_View::Export() functionality and dependency from gl2ps

5 years agoSwitching #define OCC_VERSION_DEVELOPMENT "dev"
bugmaster [Thu, 14 Jun 2018 10:27:03 +0000 (13:27 +0300)]
Switching #define OCC_VERSION_DEVELOPMENT "dev"

5 years agoUpdate version of OCCT up to 7.3.1
bugmaster [Thu, 14 Jun 2018 11:01:17 +0000 (14:01 +0300)]
Update version of OCCT up to 7.3.1

5 years agoCorrection of corrupted png file V7_3_0
bugmaster [Mon, 28 May 2018 10:39:20 +0000 (13:39 +0300)]
Correction of corrupted png file

5 years agoRaise OCCT version up to 7.3.0 (comment out "beta")
abv [Fri, 25 May 2018 15:52:41 +0000 (18:52 +0300)]
Raise OCCT version up to 7.3.0 (comment out "beta")

5 years ago0029669: Crash on opening a document with same Ocaf attributes with different IDs
abv [Fri, 25 May 2018 18:26:45 +0000 (21:26 +0300)]
0029669: Crash on opening a document with same Ocaf attributes with different IDs

Added protection against clash of attributes with the same GUID when more than one attribute of the same type but with different GUIDs are stored in the file.
If attribute with default GUID has already been read, then adding next empty attribute of the same type (done at start of its reading) will fail; in such case another attempt is made with Null GUID.

5 years ago0029816: Add possibility to get/set shape presentations for GD&T label through one...
isn [Tue, 13 Mar 2018 13:22:28 +0000 (16:22 +0300)]
0029816: Add possibility to get/set shape presentations for GD&T label through one function

new methods GetGDTPresentations(.) and GetGDTPresentations(.) for Dimension Tool

5 years ago0029407: Generating static libraries on macOS
apn [Fri, 25 May 2018 15:23:01 +0000 (18:23 +0300)]
0029407: Generating static libraries on macOS

Update "Building with Xcode" documentation

5 years ago0029266: Configuration - CMake install does not copy pdb files in Debug mode due...
apn [Thu, 24 May 2018 13:54:00 +0000 (16:54 +0300)]
0029266: Configuration - CMake install does not copy pdb files in Debug mode due to the changes in 0028287

Define variable "OCCT_INSTALL_BIN_LETTER" only if compiler is some version of Microsoft Visual C.

5 years ago0029791: Visualization - Wrong result of SelectMgr_RectangularFrustum::IsClipped
kgv [Thu, 24 May 2018 17:10:32 +0000 (20:10 +0300)]
0029791: Visualization - Wrong result of SelectMgr_RectangularFrustum::IsClipped

SelectMgr_RectangularFrustum::computeClippingRange() fix skipping depth range starting behind the ray.

5 years ago0029715: Mesh - Estimate the grid size of the acceleration structure by the complexit...
drazmyslovich [Mon, 23 Apr 2018 13:04:29 +0000 (16:04 +0300)]
0029715: Mesh - Estimate the grid size of the acceleration structure by the complexity of the face

The circles acceleration structure sizes are estimated using the required deflection and the complexity of the face.

5 years ago0029640: OCAF - add information on replacement of CDM_MessageDriver in upgrade guide.
abv [Thu, 24 May 2018 15:43:22 +0000 (18:43 +0300)]
0029640: OCAF - add information on replacement of CDM_MessageDriver in upgrade guide.

Information on replacement of CDM_MessageDriver by Message_Messenger is added in the upgrade guide.
This replacement is recorded as rename in upgrade.dat

5 years ago0029621: Application Framework - Impossible to attach existing tessellation to XCAFDo...
snn [Wed, 11 Apr 2018 13:19:07 +0000 (16:19 +0300)]
0029621: Application Framework - Impossible to attach existing tessellation to XCAFDoc_Note

- XCAFNoteObjects_NoteObject transfer object for note's auxiliary data was added. It contains the following fields:
  text and attachment positions, note plane and tesselated presentation
- GetObject/SetObject methods were added to XCAFDoc_Note attribute. The following sub-labels were added to handle transfer object:
  1 - text position
  2 - note plane
  3 - attachment point
  4 - tesselated presentation
- documentation updated

Off-topic: procedure genproj now gives meaningful error message if new package is added but not listed in UDLIST

5 years ago0029811: Tests - unstable tests demo\draw\binpersist_* due to usage of the same tempo...
kgv [Thu, 24 May 2018 09:47:28 +0000 (12:47 +0300)]
0029811: Tests - unstable tests demo\draw\binpersist_* due to usage of the same temporary file name

5 years ago0029800: Upgrade Guide - TInspector correction documentation
nds [Thu, 24 May 2018 04:20:02 +0000 (07:20 +0300)]
0029800: Upgrade Guide - TInspector correction documentation

TInspector documentation is updated

5 years ago0029775: Modeling Data - BRepAdaptor_CompCurve Parameterization Is Incorrect
Benjamin Bihler [Wed, 16 May 2018 09:59:15 +0000 (11:59 +0200)]
0029775: Modeling Data - BRepAdaptor_CompCurve Parameterization Is Incorrect

Added wire connectivity requirement to class documentation of
BRepAdaptor_CompCurve.

5 years ago0029754: Coding Rules - replace Standard_Integer with Graphic3d_ZLayerId for consistency
kgv [Mon, 14 May 2018 05:54:08 +0000 (08:54 +0300)]
0029754: Coding Rules - replace Standard_Integer with Graphic3d_ZLayerId for consistency

5 years ago0029139: Overview contains old screenshots of occt
ysn [Mon, 21 May 2018 11:18:12 +0000 (14:18 +0300)]
0029139: Overview contains old screenshots of occt

Numbers and screenshots have been updated in the overview to correspond to V7.3.0.

5 years ago0029784: Crash at STEP file reading with enabled sub-shapes
gka [Mon, 21 May 2018 13:09:28 +0000 (16:09 +0300)]
0029784: Crash at STEP file reading with enabled sub-shapes

In the method STEPCAFControl_Reader::ExpandShell protection for case when entity ""Connected_Face_Set" contains not only "FACE" entities ( for example "VERTEX_POINT" entities) was added.

5 years ago0029790: Upgrade Guide - problematic grammar and formatting
ysn [Mon, 21 May 2018 13:24:44 +0000 (16:24 +0300)]
0029790: Upgrade Guide - problematic grammar and formatting

Grammar errors and formatting issues have been fixed.

5 years ago0029755: Proofread changes in the user guides from v7.2 to v7.3
ysn [Fri, 11 May 2018 12:17:12 +0000 (15:17 +0300)]
0029755: Proofread changes in the user guides from v7.2 to v7.3

Recent modifications and additions in the user and dev guides have been reviewed (except for upgrade.md and inspector.md).

5 years ago0029764: Foundation Classes, TColStd_MapIteratorOfPackedMapOfInteger - workaround...
kgv [Mon, 14 May 2018 12:33:55 +0000 (15:33 +0300)]
0029764: Foundation Classes, TColStd_MapIteratorOfPackedMapOfInteger - workaround Visual Studio Linker bug with enabled CLI

5 years ago0029768: Visualization, TKOpenGl - structure is entirely clipped by suppressed clipping
kgv [Mon, 14 May 2018 16:25:16 +0000 (19:25 +0300)]
0029768: Visualization, TKOpenGl - structure is entirely clipped by suppressed clipping

OpenGl_Structure::Render(), added missing check for disabled state of the plane.
vclipplane command has been extended with new option -setOverrideGlobal.

5 years ago0029746: Modeling Algoriothms - declare Boolean operation classes in BRepAlgo package...
msv [Thu, 10 May 2018 08:50:03 +0000 (11:50 +0300)]
0029746: Modeling Algoriothms - declare Boolean operation classes in BRepAlgo package as deprecated

The API classes in the package BRepAlgo providing access to old Boolean operations (Fuse, Common, Cut, Section) are marked as deprecated.
Corresponding classes from the package BRepAlgoAPI should be used instead.

5 years ago0026493: BRepProj_Projection failed to project a wire on a shell
emv [Wed, 25 Apr 2018 06:33:50 +0000 (09:33 +0300)]
0026493: BRepProj_Projection failed to project a wire on a shell

Problem is fixed by the fix for the issue #0028599 (porting of the BRepProj_Projection to modern Boolean Operations algorithm)
Thus, only test case is integrated here.

5 years ago0029724: Visualization - add AIS_InteractiveContext::ClearDetected() undoing MoveTo...
kgv [Wed, 25 Apr 2018 08:38:11 +0000 (11:38 +0300)]
0029724: Visualization - add AIS_InteractiveContext::ClearDetected() undoing MoveTo() dynamic highlighting

Introduced new method AIS_InteractiveContext::ClearDetected() allowing
to reset the list of detected objects and clear dynamically highlighted
entity under mouse cursor after previous AIS_InteractiveContext::MoveTo().

vmoveto command has been extended with new argument -reset.

5 years ago0029744: Visualization - SelectMgr_ViewerSelector::PickedPoint() returns wrong 3D...
kgv [Tue, 8 May 2018 12:32:16 +0000 (15:32 +0300)]
0029744: Visualization - SelectMgr_ViewerSelector::PickedPoint() returns wrong 3D point for objects with transformation persistence

SelectMgr_RectangularFrustum::DetectedPoint() now takes into account myScale.

5 years ago0029779: Visualization, SelectMgr_SelectableObject - sensitivity change for object...
nds [Wed, 16 May 2018 14:06:10 +0000 (17:06 +0300)]
0029779: Visualization, SelectMgr_SelectableObject - sensitivity change for object not activated in given mode leads to crash

5 years ago0029743: Samples - Inspector tool - crash by selecting TNaming_UsedShapes tree item
nds [Tue, 8 May 2018 14:55:09 +0000 (17:55 +0300)]
0029743: Samples - Inspector tool - crash by selecting TNaming_UsedShapes tree item

- TNaming_UsedShapes is corrected to do not call methods of empty TopoDS_Shape
- TNaming_NamedShape is corrected to have not NULL TNaming_RefShape even if TopoDS_Shape is NULL(TNaming_Builder::Delete).
- DFBrowserPane presentation tree item for this is corrected in the same way and obsolete methods of processing SortedReferences are removed.

5 years ago0029749: Samples - Inspector tool - remove "modified" column in TNaming_NamedShape...
nds [Thu, 10 May 2018 03:53:15 +0000 (06:53 +0300)]
0029749: Samples - Inspector tool - remove "modified" column in TNaming_NamedShape presentation

5 years ago0029748: Samples - Inspector tool - use recently opened files in TInspectorEXE
nds [Tue, 8 May 2018 17:21:08 +0000 (20:21 +0300)]
0029748: Samples - Inspector tool - use recently opened files in TInspectorEXE

- recently files using in open file dialog
- correction of TInspectorEXE title to show opened file path by Start

5 years ago0029781: Samples - Inspector tool - history tree model is not set into external callb...
nds [Thu, 17 May 2018 06:29:04 +0000 (09:29 +0300)]
0029781: Samples - Inspector tool - history tree model is not set into external callback in VInspector

- AIS_InteractiveContex and VInspector_ViewModelHistory are set into VInspector_CallBack
- minor correction of displaySelectedPresentations () to Hide/Show(by popup menu) any AIS_InteractiveObject, not only AIS_Shape (e.g. AIS_Trihedron)

5 years ago0029747: Samples - Inspector tool - start DFBrowser inside FuncDemo qt sample
nds [Tue, 8 May 2018 16:56:11 +0000 (19:56 +0300)]
0029747: Samples - Inspector tool - start DFBrowser inside FuncDemo qt sample

new Model->DFBrowser action to start Inspector tool with active DFBrowser plugin filled by the sample OCAF application.
To update content of the DFBrowser, use <F5> shortcut.

5 years ago0029735: Draw Harness - command to set 2D mode for viewer in ViewerTest package
nds [Tue, 8 May 2018 16:17:09 +0000 (19:17 +0300)]
0029735: Draw Harness - command to set 2D mode for viewer in ViewerTest package

vinit command has been extended with new option -2d_mode.
Added new command v2dmode to switch On/Off the mode.