4 years ago0027868: Visualization - AIS_InteractiveContext::Remove() might keep removed object...
vpa [Mon, 31 Oct 2016 13:34:15 +0000]
0027868: Visualization - AIS_InteractiveContext::Remove() might keep removed object in selection list

- AIS_InteractiveContext::unhighlightOwners is corrected to handle all objects in selection regardless of owner's selection state

4 years agoFixed compilation warnings on vc14 caused by CR27431_5
apn [Thu, 3 Nov 2016 11:20:04 +0000]
Fixed compilation warnings on vc14 caused by CR27431_5

4 years ago0027386: BRepOffsetAPI_MakePipeShell does not provide history of generations
jgv [Mon, 17 Oct 2016 10:59:58 +0000]
0027386: BRepOffsetAPI_MakePipeShell does not provide history of generations

The method BRepOffsetAPI_MakePipeShell::Generated now returns generated shapes for sub-edges and sub-vertices of sections.

The method BRepFill_PipeShell::BuildHistory is redesigned: now it builds generated shapes for sub-edges and sub-vertices of sections.

New Draw command "gensweep" is added to sweep commands for testing of history building.

4 years ago0027870: Modeling - refactoring of HLR algorithms
abk [Wed, 5 Oct 2016 10:49:44 +0000]
0027870: Modeling - refactoring of HLR algorithms

Toolkit 'TKHLR' was fully refactored for 'Standard_Address' and macros except about half of package 'HLRBREP' there 'Standard_Address' is used through the 'generic' mechanism.

4 years ago0028014: PMIVis - some tolerances are attached to points out of the models
ika [Fri, 28 Oct 2016 12:51:58 +0000]
0028014: PMIVis - some tolerances are attached to points out of the models

Fix a misprint.

4 years ago0028031: Visualization, TKV3d - do not use height/width ratio of window during V3d_Vi...
osa [Mon, 31 Oct 2016 14:08:37 +0000]
0028031: Visualization, TKV3d - do not use height/width ratio of window during V3d_View::FitAll()

4 years ago0028010: Visualization, Prs3d_Arrow - add Shading presentation builder
isk [Fri, 14 Oct 2016 06:17:36 +0000]
0028010: Visualization, Prs3d_Arrow - add Shading presentation builder

StdPrs_ToolCylinder, StdPrs_ToolDisk, StdPrs_ToolQuadric and StdPrs_ToolSphere
have been moved from StdPrs package to Prs3d.

4 years ago0027008: Visualization - polygonal selection algorithm does not work with MeshVS_Mesh
vpa [Fri, 28 Oct 2016 12:03:38 +0000]
0027008: Visualization - polygonal selection algorithm does not work with MeshVS_Mesh

- correct calculation of normals for SelectMgr_TriangularFrustum;
- test case for issue #27008

4 years ago0028012: Exception while intersecting two surfaces
anv [Wed, 2 Nov 2016 12:46:52 +0000]
0028012: Exception while intersecting two surfaces

Test case for issue CR28012

4 years ago0026673: BOPTools_AlgoTools::ComputeVV incorrectly checks two vertices for intersection
msv [Fri, 28 Oct 2016 15:20:03 +0000]
0026673: BOPTools_AlgoTools::ComputeVV incorrectly checks two vertices for intersection

Test case has been created.

4 years ago0024932: addsweep does not check if the supplied variable contains a shape
msv [Wed, 2 Nov 2016 12:43:50 +0000]
0024932: addsweep does not check if the supplied variable contains a shape

Check for null shape has been added in the command addsweep implementation.

// eliminate warning

Test case for issue CR24932
Test case has been corrected.

4 years ago0027530: Visualization - AIS_InteractiveContext::HilightNextDetected() doesn't work...
vpa [Thu, 27 Oct 2016 11:29:27 +0000]
0027530: Visualization - AIS_InteractiveContext::HilightNextDetected() doesn't work in Neutral Point

- added implementation of mentioned methods for neutral point;
- sequence of detected objects was replaced by sequence of detected owner's indexes in AIS_InteractiveContext;
- commands vselnext and vselprev were corrected to update viewer properly;
- test case for issue #27530.

4 years ago0028009: [Regression vs OCCT7.0.0] Cut produces not valid shape
nbv [Wed, 2 Nov 2016 12:39:37 +0000]
0028009: [Regression vs OCCT7.0.0] Cut produces not valid shape

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

Small correction of test cases for issue CR28009

4 years ago0027431: [Regression to 6.9.1] Huge tolerance obtained during intersection of cylinde...
nbv [Wed, 2 Nov 2016 12:25:36 +0000]
0027431: [Regression to 6.9.1] Huge tolerance obtained during intersection of cylinder and sphere

The root of the problem is incorrect processing of cases when intersection line goes through the apex(es) of sphere. The fix improves this situation. The algorithm is taken from DecomposeResult(...) function (see IntPatch_ImpPrmIntersection.cxx file). Before the fix, faltering steps were done to solve this problem. As result, it worked in some particular cases. Now, its possibilities have been extended significantly.

Following changes have been made in the fix:

1. Class IntPatch_ALineToWLine has been rewritten cardinally. It touches as interfaces of existing methods as adding/removing some methods/fields. Correction touches both cases: going through seam of Cone/Sphere and through pole(s) of sphere. Old interface did not allow making some actions with analytical line (ALine), e.g. splitting it on several Walking-lines (WLine).

2. Restriction-line support has been removed from Implicit-Implicit intersection result (see IntPatch_Intersection::GeomGeomPerfom(...) method). It connects with the fact that the intersection algorithm itself returns precise intersection line in analytical cases (in compare with parametric intersector). Therefore, we do not need in additional (restriction) line.

3. New class IntPatch_SpecialPoints has been added. This class contains methods to add some special points (such as apex of cone, pole of sphere, point on surface boundary etc.) in intersection line (IntPatch_PointLine). It is based on the static functions, which already exist in IntPatch_ImpPrmIntersection.cxx file (these functions have been moved to the new class).

4. Method IntPatch_WLineTool::ExtendTwoWlinesToEachOther(...) has been renamed to IntPatch_WLineTool::ExtendTwoWLines(...). It is connected with changing main idea of the method. Now it allows extending WLine to the surface boundary or to the singular point (if it is possible): cone apex, sphere pole etc. Interface of this method has been corrected, too. At that, old functionality (extending to each other) has been kept. For implementation of this algorithm, new enumeration "IntPatchWT_WLsConnectionType" has been created.

5. Method IntPatch_PointLine::CurvatureRadiusOfIntersLine(...) has been added. See IntPatch_PointLine.hxx for detail information. It allows correct step computing depended on the local curvature of the intersection line. This method uses geometrical properties of intersected surfaces to compute local curvature. Therefore, it can be applied in wide range of cases even if the intersection curve is not represented in explicit form (e.g. in case of param-param-intersection).

6. Method IntSurf::SetPeriod(...) has been created.

7. Additional check has been added in Draft_Modification::Perform() method for better choice of correct fragment of intersection line for processing DRAFT operation.

8. New overload method IntPatch_Point::SetValue() has been added.

9. Some refactoring of the code has been made.

Creation of test case for issue #27431.

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

It is really IMPROVEMENT (but fortuitous).

The behavior of this test has been reverted to the state before fixing the issue #27341. Main problem has not been fixed in #27341. It was fortuitous improvement.

Quality of intersection curve was not checked. And the curve is bad on both MASTER and FIX. Input data are really wrong: plane-like-cone. However, on the MASTER, four intersection curves (the quality is insignificant) are expected. On the fix, not empty intersection result is expected simply.

Differences in images and CPU is expected. Difference in images is expected to be fixed in the issue #26020. Now, we should apply this behavior.
Much CPU time is spent by IntTools_FaceFace::ComputeTolReached3d(...) and GeomInt_IntSS::BuildPCurves(...) methods calling. These methods are not touched by the algorithm. It is the result of change of intersection curve(s) form. However, the new Curve(s) seems to be valid and can be applied. As result, new behavior can be applied, too.

CPU difference is expected. Much CPU time is spent by IntPatch_PointLine::CurvatureRadiusOfIntersLine(...) method calling. This method is really new (it does not exist on the MASTER) and is really useful. Therefore, we should apply new behavior.

CPU difference is expected. Much CPU time is spent by IntTools_WLineTool::DecompositionOfWLine(...) and IntTools_FaceFace::ComputeTolReached3d(...) methods calling. These methods are not touched by the algorithm. It is the result of change of intersection curve(s) form. However, the new Curve(s) seems to be valid and can be applied. As result, new behavior can be applied, too.

Differences in images is expected. The test keeps its BAD status on the FIX. But the result on the fix is nearer to expected than on the MASTER. Issue #27014 is still actual. As before, it is not clear, why the number of entities is different. The number of section curves has not been changed. Interfered entities are the same as on the MASTER.

The reason is described in the issue #27896.

Small correction in the test case

4 years ago0027441: The method IntTools_Context::IsVertexOnLine incorrectly computes parameter...
emv [Wed, 27 Apr 2016 13:28:52 +0000]
0027441: The method IntTools_Context::IsVertexOnLine incorrectly computes parameter of the point on the curve

Choosing the closest bound to the checking point.

4 years ago0026524: ShapeFix_Face crash when performing fix on an invalid face.
gka [Wed, 26 Oct 2016 16:31:49 +0000]
0026524: ShapeFix_Face crash when performing fix on an invalid face.

Check that method Context() returns not null handle was added.

4 years ago0027975: Data Exchange - Add possibility to connect DGTs to vertices.
ika [Tue, 18 Oct 2016 10:24:29 +0000]
0027975: Data Exchange - Add possibility to connect DGTs to vertices.

Implement new function to find Product_Definition_Shape entity.
Modify Reader to add possibility of import DGTs, connected to some geometry items.
Update test cases.

4 years ago0027991: Modeling Algorithms - BRepGProp_Face crashes on face without geometric surface
kgv [Sat, 22 Oct 2016 15:37:51 +0000]
0027991: Modeling Algorithms - BRepGProp_Face crashes on face without geometric surface

BRepGProps now ignores faces without geometric surface to avoid access violation.
BRepExtrema_DistShapeShape::DistanceMapMap() now skips comparison between void bounding boxes.
BRepBndLib::Add() now ignores useTriangulation flag for faces without geometric surfaces, and uses triangulation if any for updating of the box.

4 years ago0027258: Configuration - generate built-in replacement for mandatory resource files
ski [Fri, 28 Oct 2016 11:29:58 +0000]
0027258: Configuration - generate built-in replacement for mandatory resource files

Generation of header files from resource files was added to CMake and genproj procedures.

Message_MsgFile has been extended with new method ::LoadFromString()
for loading messages from embedded resources.
Message_MsgFile::LoadFromString() is now a preferred way
for loading message resources by application
as alternative to environment variables.

TObje/TObj.msg is now embedded into TObj_Application.cxx.
TObj_Application now loads its global messages
on instantiation of the first class instance.

UnitsAPI/Lexi_Expr.dat now completely embedded into Units_Lexicon.cxx.
UnitsAPI/Units.dat now embedded into Units_UnitsDictionary.cxx
but can be regenerated from resource file.
The definition of the following units have been removed:
benne à charbon, calorie (diététique).

Unused message files XSMessage/IGES.us and IGES.fr have been removed.
Related code IGESData.cxx has been removed as well.

XSMessage/XSTEP.us is now embedded into Interface_StaticStandards.cxx
and used for fallback initialization in case when file resources
defined by CSF_XSMessage environment variable are missing.

SHMessage/SHAPE.us is now embedded into ShapeExtend.cxx
and used for fallback initialization in case when file resources
defined by CSF_XHMessage environment variable are missing.
Duplicating code has been removed from ShapeProcess_OperLibrary.cxx.

Shaders/Declarations.glsl and Shaders/DeclarationsImpl.glsl
are now embedded into OpenGl_ShaderProgram.cxx.
CSF_ShadersDirectory is no more required for using OCCT 3D Viewer.

Ray-Tracing GLSL programs from Shaders are now embedded into OpenGl_View_Raytrace.cxx.
File resources are still used instead of embedded programs
when CSF_ShadersDirectory is defined, but this functionality
is intended for OCCT development.

Enumeration Graphic3d_ShaderProgram::ShaderName_Phong
demonstrating custom GLSL program usage has been removed.

4 years agoModify test case v3d/raytrace/bug25221 according to new draw commands.
apn [Fri, 28 Oct 2016 11:28:42 +0000]
Modify test case v3d/raytrace/bug25221 according to new draw commands.

4 years ago0025792: BRepAlgoAPI_Section fails depending on order of arguments
apn [Thu, 27 Oct 2016 12:50:24 +0000]
0025792: BRepAlgoAPI_Section fails depending on order of arguments

Added test case bugs/modalg_6/bug25792

4 years ago0027244: Test case with workaround for issue 0027240
emv [Thu, 27 Oct 2016 14:52:34 +0000]
0027244: Test case with workaround for issue 0027240

Test case for issue CR27404

4 years ago0027404: Data Exchange - access violation when reading STEP file
gka [Thu, 27 Oct 2016 14:50:18 +0000]
0027404: Data Exchange - access violation when reading STEP file

Added check on the null handle
Test case for issue CR27404

4 years ago0027692: Visualization, AIS_AngleDimension - exterior angle and arrows visibility...
nds [Tue, 11 Oct 2016 05:46:42 +0000]
0027692: Visualization, AIS_AngleDimension - exterior angle and arrows visibility improvements

Extension of angle presentation to provide this functionality

4 years ago0028002: Invalid result of Boolean Fuse operation
emv [Wed, 26 Oct 2016 10:00:56 +0000]
0028002: Invalid result of Boolean Fuse operation

The SolidClassifier algorithm does not take into account the internal parts of the solid and its faces.
But in some cases the parts which are internal on some shapes can be included in other shapes of
the same solid with FORWARD/REVERSED orientation and therefore should be counted as well.

The fix changes the procedure of the initialization of the tree of bounding boxes of the edges and
vertices of the solid in SolidExplorer algorithm to treat the internal/external parts of the solid correctly.

4 years ago0027260: Incorrect result of General Fuse operation
apn [Thu, 27 Oct 2016 11:33:50 +0000]
0027260: Incorrect result of General Fuse operation

Added test case bugs/modalg_6/bug27260

4 years ago0027514: Configuration, CMake - relative paths are not correctly handled for INSTALL_DIR
ski [Wed, 15 Jun 2016 09:29:47 +0000]
0027514: Configuration, CMake - relative paths are not correctly handled for INSTALL_DIR

Relative path can be used as INSTALL_DIR.

4 years ago0027971: Intersection with degenerated conical surface is translated incorrectly
anv [Thu, 20 Oct 2016 15:04:55 +0000]
0027971: Intersection with degenerated conical surface is translated incorrectly

Added check for infinite surfaces.

4 years ago0027960: Configuration - fix compilation of OSD_Directory with MinGW-w64
kgv [Thu, 13 Oct 2016 17:22:27 +0000]
0027960: Configuration - fix compilation of OSD_Directory with MinGW-w64

Check _NATIVE_WCHAR_T_DEFINED only within _MSC_VER since it is msvc-specific.

4 years ago0025221: Visualization - Depth test errors in ray-tracing scene containing face outlines
duv [Tue, 25 Oct 2016 08:21:17 +0000]
0025221: Visualization - Depth test errors in ray-tracing scene containing face outlines

Calculation of polygon offset for ray tracing has been changed.
Issues with wrong data in FBO depth buffer has been resolved.

4 years ago0025214: HLR fails to project edges
msv [Tue, 25 Oct 2016 08:54:34 +0000]
0025214: HLR fails to project edges

Avoid getting non-existing array items. Use reference to array items instead of pointers where it is possible.
Create test case. Before the fix this test raised exception in debug mode.

4 years ago0027856: Regression vs 6.7.1: General Fuse operation fails to fuse the solids
nbv [Tue, 25 Oct 2016 13:39:13 +0000]
0027856: Regression vs 6.7.1: General Fuse operation fails to fuse the solids

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

4 years ago0027782: Hidden Line Removal algorithm fails to handle filleted edges
msv [Thu, 27 Oct 2016 14:24:20 +0000]
0027782: Hidden Line Removal algorithm fails to handle filleted edges

Create test case for the bug.
Minor correction in test case bugs/modalg_6/bug27782

4 years ago0027570: Unacceptable performance during reading large STEP files
ika [Wed, 15 Jun 2016 11:29:32 +0000]
0027570: Unacceptable performance during reading large STEP files

Improvement of translating assemblies from STEP.

4 years ago0027764: Visualization - add functionality for animation of 3D camera and interactive...
kgv [Thu, 27 Oct 2016 14:20:38 +0000]
0027764: Visualization - add functionality for animation of 3D camera and interactive objects

Added classes AIS_Animation, AIS_AnimationCamera, AIS_AnimationObjectLocatio.

Draw Harness command vanimation has been modified to manage animation timeline.
Command vfit has been extended with option -noupdate.
Formatting of vviewparams command output has been improved.
Functionality of commands vlocreset, vlocmove, vloctranslate, vlocrotate,
vlocmirror, vlocscale has been merged into vlocation/vsetlocation.
vlocation now can print the current local transformation of the object.

v3d/ivtk test group does not call vfit anymore.

Fixed misprint in test cases bugs/vis/bug24623_3 and bug25532.

4 years ago0025264: Mesh very slow for Revol shape
oan [Thu, 27 Oct 2016 14:18:18 +0000]
0025264: Mesh very slow for Revol shape

Test case added
Small corrections of test case

4 years ago0027339: CMake install should launch generation of overview documentation
ski [Thu, 27 Oct 2016 14:16:24 +0000]
0027339: CMake install should launch generation of overview documentation

Documentation is generated automatically during build/install process
Added check of minimum Doxygen version.
Install process causes generation of Overview documentation (unless already generated) by checking of occt_logo.png file existing.
Overview project regenerates documentation if any source file was changed.

4 years ago0028001: Remove unused BOPCol_VectorOfInteger.hxx
emv [Tue, 25 Oct 2016 05:33:17 +0000]
0028001: Remove unused BOPCol_VectorOfInteger.hxx

Removing the remains of BOPCol_Array1, which was replaced with BOPCol_NCVector.

4 years ago0027986: Visualization - AIS_InteractiveContext::SetLocation() does not update dynami...
kgv [Thu, 20 Oct 2016 09:07:54 +0000]
0027986: Visualization - AIS_InteractiveContext::SetLocation() does not update dynamic highlighting

AIS_InteractiveContext::SetLocation() - update HilightMode instead of DisplayMode.
PrsMgr_PresentationManager::UpdateHighlightTrsf() - fix accessing NULL presentation.

4 years ago0027988: Visualization, AIS_InteractiveContext - fix inconsistent methods for setting...
kgv [Fri, 21 Oct 2016 09:51:26 +0000]
0027988: Visualization, AIS_InteractiveContext - fix inconsistent methods for setting highlight styles

Methods ::ChangeHighlightStyle() and ::ChangeSelectionStyle() have been
replaced by ::SetHighlightStyle() and ::SetSelectionStyle().

4 years ago0027934: Data Exchange - implement STEP common labels
ika [Wed, 5 Oct 2016 14:02:27 +0000]
0027934: Data Exchange - implement STEP common labels
implement import/export
update test cases

4 years ago0027984: Volume maker algorithm is unable to build all possible solids from the given...
emv [Thu, 20 Oct 2016 08:31:19 +0000]
0027984: Volume maker algorithm is unable to build all possible solids from the given faces

While checking the section edges between pair of faces on coincidence with the
existing edges use for checking only the edges from these two faces.

Test case for the issue.

4 years ago0027969: Visualization - add interfaces to access selecting volumes from SelectMgr_Se...
vpa [Wed, 19 Oct 2016 16:11:23 +0000]
0027969: Visualization - add interfaces to access selecting volumes from SelectMgr_SelectingVolumeManager

- getter for selection frustum computed during last run of selection mechanism was added to SelectMgr_SelectingVolumeManager;
- getters for frustum planes were added to SelectMgr_BaseFrustum and its successors;
- an API for applications to get frustum planes was added to SelectBasics_SelectingVolumeManager.

4 years ago0027987: CellsBuilder algorithm does not find shared common parts of the arguments
emv [Fri, 21 Oct 2016 08:09:34 +0000]
0027987: CellsBuilder algorithm does not find shared common parts of the arguments

Treatment of the possibility of shared parts has been added.

4 years ago0027688: Visualization, AIS_Dimension - add possibility to set custom text value
nds [Thu, 27 Oct 2016 14:02:00 +0000]
0027688: Visualization, AIS_Dimension - add possibility to set custom text value

Implementation of custom text in dimension presentation;
Test case is provided for this bug;
Code correction: Standard_EXPORT is removed for inline method.
Minor correction of test case bugs/vis/bug27688

4 years ago0027769: BRepOffsetAPI_MakePipeShell produces a face based on degenerated toroidal...
aka [Thu, 27 Oct 2016 13:58:38 +0000]
0027769: BRepOffsetAPI_MakePipeShell produces a face based on degenerated toroidal surface

method BuildKPart was modified in order to create sphere instead of torus in case of major radius <= tolerance
Minor corrections in test case bugs/modalg_6/bug27769

4 years ago0024393: Visualization - objects position with enchanced precision
kgv [Sun, 2 Oct 2016 16:36:14 +0000]
0024393: Visualization - objects position with enchanced precision

Graphic3d_ZLayerSettings - public fields have been replaced by methods.
Added new property Origin defining local coordinate system for all Layer objects.

Syntax of Draw Harness command VZLayer has been redesigned.

Graphic3d_CStructure now stores bounding box with double precision floats.
OpenGl_BVHTreeSelector - frustum culling now works with double precision floats.

4 years ago0027972: Visualization - remove unused and not implemented property V3d_View::EnableG...
kgv [Mon, 17 Oct 2016 14:39:30 +0000]
0027972: Visualization - remove unused and not implemented property V3d_View::EnableGLLight()

4 years ago0027959: BRepMesh_Delaun produces mesh with gaps on internal edges
oan [Thu, 20 Oct 2016 13:34:29 +0000]
0027959: BRepMesh_Delaun produces mesh with gaps on internal edges

Check both sides of internal link for adjusted triangle and process left and right polygons separately
Small correction of test case for issue CR27959

4 years ago0024291: Visualization - move Z-buffer trihedron presentation from TKOpenGl to TKV3d
isk [Fri, 8 Jul 2016 12:01:08 +0000]
0024291: Visualization - move Z-buffer trihedron presentation from TKOpenGl to TKV3d

Not implemented method V3d_View::TriedronEcho() and enum Aspect_TypeOfTriedronEcho have been removed.
The following methods have been removed from Graphic3d_CView class:
TriedronDisplay(), TriedronErase(), ZBufferTriedronSetup(), TriedronEcho().
New class V3d_Trihedron has been added as replacement of removed OpenGl_Trihedron
which now uses Graphic3d_TMF_TriedronPers persistence.

4 years ago0027957: Visualization, AIS_InteractiveContext - protect from displaying the same...
kgv [Thu, 13 Oct 2016 12:24:23 +0000]
0027957: Visualization, AIS_InteractiveContext - protect from displaying the same AIS_InteractiveObject within multiple contexts

AIS_InteractiveContext methods adding object to the context now throws Standard_ProgramError exception
if object has been already displayed in another context.
AIS_InteractiveContext::Remove() now NULLifies context assigned to the object.

AIS_InteractiveContext now inherits from Standard_Transient instead of deprecated MMgt_TShared
and defines C++ destructor instead of method Delete().

AIS_InteractiveObject - undocumented property State() has been removed.
Undocumented property Users() has been moved to AIS_IdenticRelation.

Draw Harness command vclose now clear AIS_InteractiveContext content
before nullifying it to ensure that objects have been properly removed.

AIS_MultipleConnectedInteractive now overrides method ::SetContext()
to assign context for children objects.

4 years ago0027961: Visualization - remove unused and no more working OpenGl_AVIWriter
kgv [Thu, 13 Oct 2016 13:28:34 +0000]
0027961: Visualization - remove unused and no more working OpenGl_AVIWriter

4 years ago0027604: Application Framework - memory is not released after closing XBF file
mpv [Thu, 20 Oct 2016 13:26:25 +0000]
0027604: Application Framework - memory is not released after closing XBF file

Virtual method *BinLDrivers_DocumentRetrievalDriver::Clear* now allows to Clear a cash data accumulated during a reading.
So, in successor class BinDrivers_DocumentRetrievalDriver this allows clearing the BinMNaming_NamedShapeDriver set of stored shapes when it is not needed anymore.

4 years ago0027963: Coding Rules - eliminate GCC compiler warnings -Wmissing-field-initializers...
kgv [Thu, 13 Oct 2016 18:09:51 +0000]
0027963: Coding Rules - eliminate GCC compiler warnings -Wmissing-field-initializers in OSD_File.cxx

4 years ago0027945: Visualization - handle correctly view clipping planes within zoom-persistent...
kgv [Thu, 13 Oct 2016 11:01:19 +0000]
0027945: Visualization - handle correctly view clipping planes within zoom-persistent objects

OpenGl_Structure::Render() and SelectMgr_ViewerSelector::checkOverlap()
now clip entire zoom/rotate persistence object
by checking anchor point with global clipping planes.

Image dump has been added to the new test case.

4 years ago0027953: Foundation Classes - eliminate global variables from OSD_Path
kgv [Wed, 12 Oct 2016 09:46:48 +0000]
0027953: Foundation Classes - eliminate global variables from OSD_Path

4 years ago0027931: Coding rules, Precision.hxx - eliminate dead code within PRECISION_OBSOLETE...
kgv [Wed, 5 Oct 2016 10:52:48 +0000]
0027931: Coding rules, Precision.hxx - eliminate dead code within PRECISION_OBSOLETE macros

4 years ago0027943: Visualization - fix broken shading by positional light for object with local...
kgv [Sun, 9 Oct 2016 17:48:16 +0000]
0027943: Visualization - fix broken shading by positional light for object with local transformation

OpenGl_ShaderManager, per-vertex shading and per-fragment shading
- do not apply model-world matrix on the light sources
which are expected to be defined in world-space.

4 years ago0027952: Visualization - do not reset RayTracing state on changing structures not...
kgv [Tue, 11 Oct 2016 19:55:25 +0000]
0027952: Visualization - do not reset RayTracing state on changing structures not in main Z-Layer

4 years ago0027824: Parasolid XMT file generates weird geometry after conversion
anv [Thu, 1 Sep 2016 09:29:14 +0000]
0027824: Parasolid XMT file generates weird geometry after conversion

Fixed adapter for C0 offsets

4 years ago0027930: XMT file conversion loops infinitely
nbv [Thu, 13 Oct 2016 10:48:38 +0000]
0027930: XMT file conversion loops infinitely

Now, the algorithm tries to estimate U- and V-ranges of future intersection curve(s) on the surface. This information is used in stop-criterium of the algorithm instead of full surface range used earlier. It allows reducing dependencies of intersection result on the surface ranges.
Tuning of test case bugs/modalg_6/bug27937_1

4 years ago0027946: Out of range item access in offset algorithm
aml [Mon, 10 Oct 2016 12:08:36 +0000]
0027946: Out of range item access in offset algorithm

Out of range access has been eliminated.

4 years ago0027619: Improvement on manuipulation of data by TPrsStd_AISPresentation attribute.
vro [Thu, 13 Oct 2016 10:45:03 +0000]
0027619: Improvement on manuipulation of data by TPrsStd_AISPresentation attribute.

The branch contains a fix on Undo/Redo for TPrsStd_AISPresentation attribute + an opportunity to deselect the interactive object.
getData() now calls FindAttribute(), the field myData disappeared.
A fix of regression on bug73_3.
A cleverer solution.

4 years ago0027941: Foundation Classes - NCollection_Vector::Iterator::offsetV() does not match...
kgv [Fri, 7 Oct 2016 15:49:34 +0000]
0027941: Foundation Classes - NCollection_Vector::Iterator::offsetV() does not match NCollection_Vector::end()

4 years ago0027944: Visualization, V3d_View - filter duplicates within ::AddClipPlane()
kgv [Sun, 9 Oct 2016 18:59:58 +0000]
0027944: Visualization, V3d_View - filter duplicates within ::AddClipPlane()

4 years ago0027915: Foundation Classes - remove the class NCollection_QuickSort
ski [Thu, 13 Oct 2016 10:40:12 +0000]
0027915: Foundation Classes - remove the class NCollection_QuickSort

Class NCollection_QuickSort was removed.
Direction of dimension lines was corrected.
Updated porting notes.
minor corrections in AIS

4 years ago0027895: Correction in the constructor Extrema_ExtElC::Extrema_ExtElC (const gp_Lin...
nbv [Thu, 22 Sep 2016 13:46:12 +0000]
0027895: Correction in the constructor Extrema_ExtElC::Extrema_ExtElC (const gp_Lin&,const gp_Lin&,const Standard_Real)

Computation algorithm is made more simple.
Adjusting some test cases according to their new behavior.

4 years ago0026894: Intersection algorithm between two infinite cylinders is hanging
nbv [Thu, 6 Oct 2016 09:57:25 +0000]
0026894: Intersection algorithm between two infinite cylinders is hanging

1. VRange of intersection curve has been limited. As result, too oblong intersection curve(s) will be never returned.
2. Now, purger algorithm is not called for lines obtained by Geom-Geom intersection method.
3. New statuses are entered in IntPatch_ImpImpIntersection class. It makes intersection algorithm more informative and flexible for using.
4. Method IntPatch_ImpImpIntersection::GetStatus() has been created.

Tuning of test case bugs modalg_6/bug26894

4 years ago0023178: Intersection of cylinders fails to produce results
nbv [Fri, 23 Sep 2016 14:24:52 +0000]
0023178: Intersection of cylinders fails to produce results

1. Unification of trimmed and not-trimmed cylinders processing (IntPatch_Intersection::GeomGeomPerfomTrimSurf() method has been removed).
2. Interface of IntPatch_ImpImpIntersection::Perform(...) method has been changed.
3. Now, WLine purging is forbidden for Geom-Geom-Intersection.
4. Bnd_Range class has been created. See Bnd_Range.hxx for detail information.
5. Algorithm of AddBoundaryPoint function has been improved in order to obtain intersection points in both boundaries (VFirst and VLast of every surface).
6. Earlier, method Geom2dConvert::ConcatG1(...) increased resulted B-spline degree (in case of not succession of previous iteration). Now increased value has been limited by Geom2d_BSplineCurve::MaxDegree() value (max degree = 25).
7. Algorithm of B-spline closure definition has been changed in the methods Geom2dConvert::C0BSplineToC1BSplineCurve(...) and Geom2dConvert::C0BSplineToArrayOfC1BSplineCurve(...).

Creation of test case for this issue.
Adjusting test cases according to their new behavior.

Small correction in the code according to KGV's remark.

4 years ago0027864: Add vtk support to products java wrapper
apn [Thu, 6 Oct 2016 09:36:21 +0000]
0027864: Add vtk support to products java wrapper

Fix redefinition IVtkVTK_EXPORT error during wrapping in products java sample.
Removed defined(__WIN32__)

4 years ago0027905: Visualization - randomly directed arrows on test v3d mesh B7
ski [Thu, 6 Oct 2016 09:30:39 +0000]
0027905: Visualization - randomly directed arrows on test v3d mesh B7

Behaviour of command meshvectors is stable now.
It shows default vector (0,0,1) for volume entity.
Test cases for issue CR27905

4 years ago0027929: Methods D0 and D1 for trimmed offset surface return different values if...
nbv [Wed, 5 Oct 2016 11:36:33 +0000]
0027929: Methods D0 and D1 for trimmed offset surface return different values if the surface has osculating surface

Using osculating surface is forbidden if the normal to basis (for offset) surface is well defined.

Creation of test case for this issue.
Adjusting test cases according to their new behavior.

4 years ago0027892: Construction error in offset
nbv [Tue, 27 Sep 2016 14:46:14 +0000]
0027892: Construction error in offset

Provide proper initialization of the field Geom_OffsetSurface::myOscSurf in the case of creating offset of the trimmed bspline surface.

Adjusting test cases according to their new behavior.

4 years ago0024073: Coding rules - ambiguous call to overloaded function StdPrs_DeflectionCurve...
kgv [Mon, 3 Oct 2016 11:41:12 +0000]
0024073: Coding rules - ambiguous call to overloaded function StdPrs_DeflectionCurve::Match()

Default parameters values have been removed from the methods
StdPrs_DeflectionCurve::Match() to avoid ambiguity.

4 years ago0027899: Visualization, Ray Tracing - Provide ability to rebuild GLSL shaders on...
dbp [Sat, 24 Sep 2016 17:26:33 +0000]
0027899: Visualization, Ray Tracing - Provide ability to rebuild GLSL shaders on the fly

For debugging and performance analysis it is reasonable to provide the ability to rebuild ray tracing shaders on the fly.
In this way, it will be possible to analyze the impact of different shaders modifications
without re-launching and re-configuring the scene.
This functionality can be used in the following way:
> vrenderparams -rebuild

4 years ago0027916: Visualization - access violation occurs within AIS_ColoredShape::Compute...
kgv [Wed, 28 Sep 2016 17:18:25 +0000]
0027916: Visualization - access violation occurs within AIS_ColoredShape::Compute() for NULL shape

AIS_ColoredShape::Compute() - add NULL shape check.
AIS presentations - removed redundant clearance within ::Compute()
which is always done in advance within PrsMgr_PresentationManager::Update().

4 years ago0027883: Visualization - AIS_InteractiveContext::IsDisplayed() does not return displa...
kgv [Mon, 26 Sep 2016 08:38:18 +0000]
0027883: Visualization - AIS_InteractiveContext::IsDisplayed() does not return displayed state for specified Display Mode

4 years ago0027890: BndLib_Add2dCurve::Add(..) works incorrect on some curves
isn [Thu, 22 Sep 2016 11:42:30 +0000]
0027890: BndLib_Add2dCurve::Add(..) works incorrect on some curves

Draw Harness command 'gbounding' has been extended
to pass curve2d as args and to work in two modes ('normal' and optimal).

4 years ago0027888: Fuse of valid untouched solids leads to result with faulties
msv [Wed, 21 Sep 2016 08:02:25 +0000]
0027888: Fuse of valid untouched solids leads to result with faulties

Incorrect result of classification of a point relatively a solid leads to faulty interferences between vertices/edges of one solid with another one.  Classification has been corrected by checking if an auxiliary point in face coincides with the face boundary in 3D space.

Test case has been created.

4 years ago0027713: Test bugs fclasses bug6143 is not OK for Linux after restoring FPS (26329)
abv [Mon, 29 Aug 2016 03:22:44 +0000]
0027713: Test bugs fclasses bug6143 is not OK for Linux after restoring FPS (26329)

Use standard (POSIX-compliant) macro "__linux__" for identification of Linux system, instead of obsolete "linux"

4 years ago0026329: Restore floating point signals handling in DRAW
ifv [Tue, 9 Aug 2016 17:12:59 +0000]
0026329: Restore floating point signals handling in DRAW

Added DRAW command dsetsignal, resetting OSD signal handler with either armed or disabled FPE handler, according to an option.
If called without arguments, it sets FPE handler only if environment variable OSD_FPE is defined (with value different from 0).
On start, DRAW calls dsetsignal to set FPE signal if CSF_FPE is defined.
Test bugs fclasses bug6143 uses dsetsignal to set FPE handler unconditionally before the test command, and resets it to default at the end.

A number of changes in the code have been done in order to fix floating point exceptions that became generated after enabling signals:

- Global functions Sinh() and Cosh() defined in Standard_Real.hxx are improved to raise Standard_NumericError exception if argument is too big (greater than 710.47586), instead of relying on system treatment of floating point overflow. These functions are used instead of sinh and cosh in ElCLib.cxx.

- Maximal value of parameter on hyperbola is restricted by 23 (corresponding to ~1e10 in 3d) in order to avoid FP overflow in Extrema_GenExtCS.cxx, ShapeFix_EdgeProjAux.cxx.

- Interface of the root curve adaptor class Adaptor3d_Curve has been updated to add new virtual methods BasisCurve and OffsetValue. They complement the adaptor for the case of offset curves. These methods are used in Extrema_GenExtCS.cxx to restrict domain search in the case of offset of hyperbola, in order to get rid of floating point overflow. All classes inheriting Adaptor3d_Curve have been changed to implement the new virtual methods.

- Protection against division by zero has been implemented in ApproxInt_KnotTools.cxx, BRepClass3d_SClassifier.cxx, BRepGProp_Face.cxx, BRepMesh_FastDiscretFace.cxx, Geom2dGcc_Circ2d2TanOnIter.cxx, Geom2dInt_Geom2dCurveTool.cxx, IntPolyh_MaillageAffinage.cxx.

- Protection against calling of math functions of infinite arguments has been added in BRepCheck_Edge.cxx, BRepLib.cxx, CSLib_NormalPolyDef.cxx, Extrema_FuncExtPC.gxx, Extrema_GExtPC.gxx, Extrema_GLocateExtPC.gxx, Intf_InterferencePolygonPolyhedron.gxx, ShapeAnalysis_Surface.cxx, ShapeAnalysis_TransferParametersProj.cxx, ShapeAnalysis_Wire.cxx, math_FunctionSetRoot.cxx.

- Proper initialization of local variables is done in BOPAlgo_PaveFiller_6.cxx, XSDRAWSTLVRML.cxx.

- Inconsistent usage of Standard_Boolean* to access integer data in HLR (caused by #27772) is corrected

Some test cases have been updated to actual state.

4 years ago0027901: Resource_Manager creates directory with insufficient permissions
abv [Sun, 25 Sep 2016 20:05:00 +0000]
0027901: Resource_Manager creates directory with insufficient permissions

Resource_Manager now sets RWXD permissions for the created directories to allow their removal.
Method OSD_Directory::Build() will create intermediate directories if they do not exist yet.
OSD_FileNode is corrected to clean error status in method Exists().

Tests bugs fclasses bug181_1 and bug181_2 are corrected to check existence of generated resource files; TODO is removed in bug181_2.

4 years ago0027794: A danger in the 'NCollection_Vector' initializing constructor
abv [Mon, 26 Sep 2016 07:32:06 +0000]
0027794: A danger in the 'NCollection_Vector' initializing constructor

Constructor of NCollection_Vector is now declared as explicit.

4 years ago0027893: Visualization - AIS_InteractiveContext::SetSelected does not work
vpa [Mon, 26 Sep 2016 17:34:26 +0000]
0027893: Visualization - AIS_InteractiveContext::SetSelected does not work

- misprint in check of global selection existence in AIS_InteractiveContext::SetSelected was corrected;
- remaining occurrencies of hard-coded 0 selection mode were corrected to use global selection mode;
- test case for issue #27983

4 years ago0027891: Visualization - FPE due to use of DBL_MAX in SelectMgr_SelectingVolumeManage...
kgv [Fri, 23 Sep 2016 12:33:40 +0000]
0027891: Visualization - FPE due to use of DBL_MAX in SelectMgr_SelectingVolumeManager::DetectedPoint()

4 years ago0027818: Visualization - provide an interface to define highlight presentation properties
vpa [Wed, 28 Sep 2016 09:43:17 +0000]
0027818: Visualization - provide an interface to define highlight presentation properties

- introduces a wrapper for setting up highlight properties: Graphic3d_HighlightStyle;
- API of all methods that use highlight or selection color is changed to deal with Graphic3d_HighlightStyle;
- highlight in shading mode now supports transparency, implemented via blending;
- transparency for selection can also be set, but implementing custom entity owners with additional presentation on application level;
- methods PrsMgr_PresentationManager::Highlight, PrsMgr_PresentationManager::BoundBox that highlight object with hard-coded color are removed;
- deprecated methods of SelectMgr_EntityOwner, that use presentation manager's highlight method, were removed;
- methods of IsHilighted AIS context with selection color checks were replaced;
- added API to store dynamic and selection highlight to Prs3d_Drawer class;
- customization of dynamic and selection highlight for particular objects is now available through SelectMgr_SelectableObject::HilightAttributes();
- AIS_InteractiveContext highlight methods were updated to support individual highlight styles of interactive objects;
- introduced new command - vselprops, that allows to customize global selection and highlight properties like autoactivation, pixel tolerance and colors;
- Draw Harness commands vautoactivatesel and vselprecision were removed, use vselprops instead;
- fixed bug in command's parser;
- test case for issue #27818

4 years ago0027607: Visualization - Implement adaptive screen space sampling in path tracing
dbp [Wed, 13 Jul 2016 09:19:27 +0000]
0027607: Visualization - Implement adaptive screen space sampling in path tracing

This commit provides useful functionality for path tracing rendering core.

1) Graphic3d_RenderingParams class was extended with additional AdaptiveScreenSampling option (disabled by default).
   If this option is enabled, path tracing tries to adjust the number of samples for different screen areas.

   In this way, the more complex areas (from the point of light conditions) are sampled more intensively,
   while the simple areas are sampled very rarely.
   For example, caustics and glossy reflections are typical candidates for more precise sampling.

   In general, this allows to equalize image convergence and not to waste resources for already converged areas.
   It is also possible to visualize sampling densities by enabling ShowSamplingTiles option
   (activating and deactivating this option does not affect on the accumulated image).

2) Mixing OpenGL and ray-tracing output has been changed.
   Now blending is performed using OpenGL functionality, while ray-tracing shaders only output correct Z-value.

Test case bugs vis bug27083 has been updated,
since the alpha value is now correctly set by Ray-Tracing to 1, opaque.

4 years ago0027900: Coding rules - drop redundant Name parameter from V3d_Viewer constructor
kgv [Sun, 25 Sep 2016 16:09:23 +0000]
0027900: Coding rules - drop redundant Name parameter from V3d_Viewer constructor

Removed unused methods AIS_InteractiveContext::DomainOfMainViewer(),
V3d_Viewer::NextName(), V3d_Viewer::Domain().

NCollection_List - added methods Contains() and Remove() taking item value.
V3d_Viewer now uses V3d_ListOfLight, V3d_ListOfView instead of V3d_ListOfTransient.

Removed obsolete ZFit workaround from command XShow.

4 years ago0025180: Visualization - Homogeneous transformation API in TKV3d
kgv [Sat, 17 Sep 2016 16:33:53 +0000]
0025180: Visualization - Homogeneous transformation API in TKV3d

PrsMgr_PresentableObject, Graphic3d_Structure now consistently
take and store Handle(Geom_Transformation) instead of
TColStd_Array2OfReal / Graphic3d_Mat4.
Low-level advanced methods have been modified to pass Handle(Geom_Transformation).
High-level methods have been preserved accepting old syntax taking gp_Trsf.

Geom_Transformation now inlines most methods.

4 years ago0027860: Visualization - clean up Transformation Persistence API
kgv [Mon, 19 Sep 2016 20:40:07 +0000]
0027860: Visualization - clean up Transformation Persistence API

Graphic3d_Camera::TransformMatrices redundant NCollection_Handle usage
has been replaced with validity flags.

Graphic3d_TransModeFlags now defined as enumeration, not integer bit flags.
Graphic3d_TMF_PanPers and Graphic3d_TMF_FullPers have been removed.
Graphic3d_TMF_ZoomRotatePers has been introduced.

Graphic3d_TransformPers is now inherits Standard_Transient.
Graphic3d_TransformPers now defines dedicated constructors
for 3D persistence (zoom / rotate) and 2D persistence (2d / trihedron).
2D persistence now supports dedicated values for X and Y offsets.
The corner is now specified by enumeration Aspect_TypeOfTriedronPosition
instead of indirect interpretation of anchor point values.
Fixed handling of Graphic3d_TMF_ZoomRotatePers (combination of Graphic3d_TMF_RotatePers + Graphic3d_TMF_ZoomPers).

PrsMgr_PresentableObject, Graphic3d_CStructure now hold
Handle(Graphic3d_TransformPers) instead of a value.
Method ::SetTransformPersistence(), ::TransformPersistence()
now works with Handle(Graphic3d_TransformPers).
Old methods have been marked deprecated.

4 years ago0027696: Return max distance in xdistcs Draw command
aml [Thu, 22 Sep 2016 05:12:16 +0000]
0027696: Return max distance in xdistcs Draw command

Message about maximal distance is added for xdistcs command.

4 years ago0027873: Exception is raised in BRepFill_Filling::FindExtremitiesOfHoles()
nbv [Thu, 15 Sep 2016 14:53:06 +0000]
0027873: Exception is raised in BRepFill_Filling::FindExtremitiesOfHoles()

The reason of exception has been eliminated.
Creation of the test case for this issue.
Correction of unstable test case.

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

4 years ago0027879: Draw Harness, ViewerTest - disable VSync within non-interactive mode
kgv [Sat, 17 Sep 2016 10:32:13 +0000]
0027879: Draw Harness, ViewerTest - disable VSync within non-interactive mode

4 years ago0027720: HLRBrep_Algo BSpline missing edges
jgv [Thu, 8 Sep 2016 15:20:57 +0000]
0027720: HLRBrep_Algo BSpline missing edges

    The algorithm that builds outlines ("silhouettes") makes an outline in 2d parametric space of the surface starting from some previously detected point where normal is orthogonal to direction of view. So, the surface is previously discretized into (m*n) sample points and some of them become starting points for future outlines.

    If the surface has non-uniform parametrization and/or some local extremums of curvature, the outlines can not be built without breaks, so there are several groups of consequent outlines in this case. Unfortunately, it leads to the situation when current number of sample points becomes insufficient to build all the parts of outlines.

    The idea is to detect the "holes" between already constructed parts of outlines and then complete the construction.

New auxiliary draw command for testing of HLR.

Correction according to the remarks.

Update of test case according to the developer's directive

4 years ago0006768: Problems making offset wires
bugmaster [Fri, 16 Sep 2016 11:49:16 +0000]
0006768: Problems making offset wires

Adding testing case

4 years ago0027880: Samples - fix handling of Unicode paths within MFC import/export sample
kgv [Mon, 19 Sep 2016 08:46:27 +0000]
0027880: Samples - fix handling of Unicode paths within MFC import/export sample

4 years ago0014531: Boolean Operation Algorithm fails
bugmaster [Wed, 21 Sep 2016 07:58:11 +0000]
0014531: Boolean Operation Algorithm fails

Adding test case

4 years ago0027875: GeomFill_NSections constructor crash on sequence of curve containing only...
nbv [Fri, 16 Sep 2016 12:31:29 +0000]
0027875: GeomFill_NSections constructor crash on sequence of curve containing only one curve

The GeomFill_NSections algorithm cannot create any surface from sequence with single curve only.

Therefore, return is provided in corresponding place of the code. Additionally, some public methods of GeomFill_NSections class checks if the surface has been created earlier.

Test case for this issue has been created.

Compiler error has been eliminated.

4 years ago0027822: Exception access violation is raised in BRepOffsetAPI_MakePipeShell during...
jgv [Mon, 19 Sep 2016 10:35:27 +0000]
0027822: Exception access violation is raised in BRepOffsetAPI_MakePipeShell during of build

New public method IsDone() is added to abstract class BRepFill_SectionLaw.

Test case bugs/modlag_6/bug27822 has been added

4 years ago0010234: BRepOffsetAPI_MakePipeShell fails on spine built from two segments and profi...
bugmaster [Mon, 19 Sep 2016 12:03:00 +0000]
0010234: BRepOffsetAPI_MakePipeShell fails on spine built from two segments and profile built from arc

Adding testing case