occt.git
5 years ago0030636: Configuration, qmake - unable to build using VS2017
kgv [Mon, 8 Apr 2019 13:55:12 +0000 (16:55 +0300)]
0030636: Configuration, qmake - unable to build using VS2017

5 years ago0030629: Modeling Algorithms - Invalid result of fuse operation
emv [Thu, 4 Apr 2019 06:59:24 +0000 (09:59 +0300)]
0030629: Modeling Algorithms - Invalid result of fuse operation

Plane/Torus intersection: take into account the Plane-Torus distance sign to compute the correct location of the section curve.

Test case for the issue.

5 years ago0030618: Modeling Algorithms, BOPTools_Parallel - avoid using map for thread-local...
kgv [Thu, 28 Mar 2019 21:14:36 +0000 (00:14 +0300)]
0030618: Modeling Algorithms, BOPTools_Parallel - avoid using map for thread-local contexts without TBB

OSD_Parallel::ToUseOcctThreads() - new flag allowing to use OCCT threads implementation even when compiled with TBB (for testing).
Added new command dparallel for managing default Thread Pool.
OSD_Parallel::For() now avoid creation of universal iterator in simplest case.

BOPTools_Parallel - eliminated redundant typedefs/explicit instantiations of templates.
Added functor using array of per-thread context instead of a map.

5 years ago0030339: Configuration - add project files for qmake IR-2019-04-05
kgv [Mon, 1 Apr 2019 20:44:20 +0000 (23:44 +0300)]
0030339: Configuration - add project files for qmake

Added adm/qmake/Occt.pro for building OCCT using Qt Creator.
genproj/genconf scripts have been extended with an option defining project format.

Draw_Appli now tries resolving CASROOT/DRAWDEFAULT/CSF_OCCTResourcePath/CSF_OCCTSamplesPath
locations within usual OCCT development/installation environment.
Added missing TKShHealing dependency to TKDraw/DRAWEXE (inherited from TKMesh).
OSD_Process::ExecutableFolder() - added method returning executable folder.

5 years ago0030626: Visualization - manipulator crashes when is attached with scaling mode disabled
iko [Tue, 2 Apr 2019 10:58:32 +0000 (13:58 +0300)]
0030626: Visualization - manipulator crashes when is attached with scaling mode disabled

AIS_Manipulator crashes on SetPart method when 2 inputs (AIS_ManipulatorMode, Standard_Boolean) has been fixed up.
Test case for this bug has been added.

5 years ago0030616: Crash on reading STEP file
gka [Thu, 28 Mar 2019 15:25:20 +0000 (18:25 +0300)]
0030616: Crash on reading STEP file

Added protections to avoid exceptions for null objects.
For reading entity StepShape_ToleranceValue was added reading lower and upper bounds represented by entities "StepRepr_ReprItemAndMeasureWithUnit" or "StepRepr_MeasureRepresentationItem"

5 years ago0030622: Visualization, TKOpenGl - Path Tracing fails on macOS
kgv [Fri, 29 Mar 2019 17:46:39 +0000 (20:46 +0300)]
0030622: Visualization, TKOpenGl - Path Tracing fails on macOS

Fixed usage of non-existing function texture2D().

5 years ago0030619: Draw Harness, ViewerTest - add continuous rendering option to vrepaint command
kgv [Thu, 28 Mar 2019 15:36:54 +0000 (18:36 +0300)]
0030619: Draw Harness, ViewerTest - add continuous rendering option to vrepaint command

Aspect_Window::InvalidateContent() - added new virtual method for invalidating window content using platform-specific API.

TKDraw, tkLoop() on Window platform has been changed so that to prevent
continuous input window events blocking terminal input
(e.g. in case if processing events is not fast enough or
if another continuously sends new events).

TKViewerTest, on X11 platform has been fixed a message processing
so that to avoid messages being not processed.
Added aggregation of Exposer and ConfigureNotify events.
Fixed aggregation MotionNotify events.

5 years ago0030611: Coding Rules - eliminate GCC compiler warnings -Wcatch-value
kgv [Sun, 31 Mar 2019 20:38:14 +0000 (23:38 +0300)]
0030611: Coding Rules - eliminate GCC compiler warnings -Wcatch-value

Add missing const& to catch statements.

5 years ago0030579: Draw Harness, Draw_Interpretor - catch exceptions other than Standard_Failure IR-2019-03-29
kgv [Fri, 15 Mar 2019 10:15:18 +0000 (13:15 +0300)]
0030579: Draw Harness, Draw_Interpretor - catch exceptions other than Standard_Failure

5 years ago0030574: Configuration, genproj - CSF_DEFINES is not reset at the beginning of env.bat
kgv [Thu, 14 Mar 2019 19:17:19 +0000 (22:17 +0300)]
0030574: Configuration, genproj - CSF_DEFINES is not reset at the beginning of env.bat

env.bat now resets %CSF_DEFINES% variable to empty state
and append optional %CSF_DEFINES_EXTRA% external value.

5 years ago0030556: Coding - IGESData_GlobalSection missing member initialization in default...
kgv [Tue, 12 Mar 2019 01:51:41 +0000 (04:51 +0300)]
0030556: Coding - IGESData_GlobalSection missing member initialization in default constructor

IGESData_GlobalSection now initializes class fields.

5 years ago0030505: Volume Rendering - Deferred lighting
iko [Wed, 20 Mar 2019 09:30:48 +0000 (12:30 +0300)]
0030505: Volume Rendering - Deferred lighting

Missing getter for OpenGl_Texture::mySizedFormat has been added.

5 years ago0030522: Modeling Algorithms - BRepBuilderAPI_MakeWire produces different wires depen... IR-2019-03-22
emv [Tue, 26 Feb 2019 11:06:30 +0000 (14:06 +0300)]
0030522: Modeling Algorithms - BRepBuilderAPI_MakeWire produces different wires depending on the order of parameters

The following improvements have been implemented in BRepLib_MakeWire class to make it more reliable:
1. Quit adding edges of a wire into result if the current edges have not been added. Return NotDone status.
2. Allow merging not only bounding vertices (connected to only one edge) but also the middle ones, so the multi-connected vertices may appear in the result.

Test case for the issue.

5 years ago0030582: Coding - avoid defining references to properties of NULL objects
kgv [Fri, 15 Mar 2019 15:06:04 +0000 (18:06 +0300)]
0030582: Coding - avoid defining references to properties of NULL objects

Approx_SweepApproximation::Perform() now creates empty arrays.
GeomAdaptor_SurfaceOfRevolution::UTrim() - fixed No_Exception misuse.
StdPrs_ShadedShape - fixed defining an invalid reference to Poly_Triangulation::UVNodes().

BSplCLib::MovePoint() and BSplCLib::MovePointAndTangent() now take optional
weights parameter as pointer consistent to other methods like BSplCLib::BuildEval().

5 years ago0030489: Modeling Algorithms - BRepBuilderAPI_GTransform hangs
ifv [Tue, 12 Mar 2019 13:47:21 +0000 (16:47 +0300)]
0030489: Modeling Algorithms - BRepBuilderAPI_GTransform hangs

Approx_ComputeCLine.gxx - criterium to stop interval cutting is increased.
ProjLib_ProjectedCurve.cxx - fix regression for bugs modalg_5 bug25886
Other tests are modified according to current state of algorithm

5 years ago0030593: Modeling Algorithms - ShapeUpgrade_ShapeDivide::Perform() throws Standard_No...
kgv [Tue, 19 Mar 2019 08:24:21 +0000 (11:24 +0300)]
0030593: Modeling Algorithms - ShapeUpgrade_ShapeDivide::Perform() throws Standard_NoMoreObject

Added missing TopExp_Explorer::More() checks to ShapeUpgrade_ShapeDivide::Perform() and BRepCheck_Analyzer::Perform().

5 years ago0027316: Draw Harness - access violation on passing non-existing shape to XAddShape
kgv [Wed, 20 Mar 2019 05:53:41 +0000 (08:53 +0300)]
0027316: Draw Harness - access violation on passing non-existing shape to XAddShape

Add NULL check to XAddShape command.
Removed broken test case bugs/step/bug27316.

5 years ago0030583: Draw Harness - ignore cpulimit when Debugger is attached on Windows platform
kgv [Sat, 16 Mar 2019 08:25:47 +0000 (11:25 +0300)]
0030583: Draw Harness - ignore cpulimit when Debugger is attached on Windows platform

cpulimit now omits ExitProcess() when IsDebuggerPresent() returns TRUE.

5 years ago0030594: Data Exchange - access violation within IGESGeom_BSplineCurve constructor
kgv [Wed, 20 Mar 2019 06:50:52 +0000 (09:50 +0300)]
0030594: Data Exchange - access violation within IGESGeom_BSplineCurve constructor

Added NULL check.

5 years ago0023838: Modeling Data - Standard_OutOfRange in Geom_OffsetSurface::LocalDN (called...
kgv [Sat, 16 Mar 2019 13:08:27 +0000 (16:08 +0300)]
0023838: Modeling Data - Standard_OutOfRange in Geom_OffsetSurface::LocalDN (called by BRepFill_PipeShell)

Fixed incorrect array allocation within GeomEvaluator_OffsetSurface::CalculateDN()
and protection against assigning out-of-range values in case of non-square
number of U and V derivatives on surface.

5 years ago0030588: Coding - avoid calling operator->() on NULL Handle
kgv [Sat, 16 Mar 2019 15:55:36 +0000 (18:55 +0300)]
0030588: Coding - avoid calling operator->() on NULL Handle

Use .get() method instead of operator->() where result can be NULL.
STEPCAFControl_Reader::ReadLayers(), XCAFDoc_ColorTool,
XCAFDoc_LayerTool::SetVisibility() - fixed static functions syntax.

5 years ago0030558: Coding - replace math_SingleTab with NCollection_LocalArray
kgv [Tue, 12 Mar 2019 03:05:21 +0000 (06:05 +0300)]
0030558: Coding - replace math_SingleTab with NCollection_LocalArray

math_Vector::Multiplied() - fixed modification of original array.

math_SVD::Solve(), math_SVD::PseudoInverse(), GeomFill_LocationGuide::InitX() - removed incorrect constness.

math_Vector, math_IntegerVector - math_SingleTab has been replaced by NCollection_LocalArray+NCollection_Array1.
Added accessors returning const value.

5 years ago0030536: Foundation Classes - TCollection_ExtendedString::StartsWith() and EndsWith...
vro [Mon, 18 Mar 2019 07:16:40 +0000 (10:16 +0300)]
0030536: Foundation Classes - TCollection_ExtendedString::StartsWith() and EndsWith() have a mistake

Draw-commands StartsWith and EndsWith are renamed to QAStartsWith and QAEndsWith.
The test script bug30536 was corrected.

5 years ago0030544: Data Exchange - IGES import fails on specific file
abv [Mon, 11 Mar 2019 18:28:52 +0000 (21:28 +0300)]
0030544: Data Exchange - IGES import fails on specific file

Method IGESToBRep_BRepEntity::TransferEdge() is fixed to record result even if it is wire rather than edge.
This allows translation of EdgeLoop entities referring to CompositeCurve items in EdgeList entities.
Curves that fail to translate are recorded in result as Null shapes (rather than skipping them) to keep indexation of curves in EdgeList valid for remaining ones.

Method IGESToBRep_BRepEntity::TransferLoop() is improved to skip failed edges thus avoiding exception and allowing translation to complete for transferable entities.

Warning messages are corrected to be more meaningful.
Added test bugs iges bug30544

5 years ago0030569: Foundation Classes - NCollection_Shared lacks inclusion of NCollection_Defin...
kgv [Wed, 13 Mar 2019 13:53:16 +0000 (16:53 +0300)]
0030569: Foundation Classes - NCollection_Shared lacks inclusion of NCollection_DefineAlloc.hxx

Add missing includes.

5 years ago0030584: Modeling Algorithms, BRepFill_OffsetWire - access violation on Face having...
kgv [Sat, 16 Mar 2019 10:22:47 +0000 (13:22 +0300)]
0030584: Modeling Algorithms, BRepFill_OffsetWire - access violation on Face having no boundaries

Added missing NULL and empty result checks.
Updated test cases using offset_wire_092.brep shape containing all Edges
with TopAbs_INTERNAL orientation (hense no boundary to make offset).

5 years ago0030581: Modeling Data - Standard_OutOfRange within Geom_BSplineSurface::LocateV()
kgv [Sat, 16 Mar 2019 06:33:37 +0000 (09:33 +0300)]
0030581: Modeling Data - Standard_OutOfRange within Geom_BSplineSurface::LocateV()

Geom2d_BSplineCurve::LocateU(),Geom_BSplineCurve::LocateU, Law_BSpline::LocateU()
Geom_BSplineSurface::LocateU() and Geom_BSplineSurface::LocateV()
have been corrected with missing range checks.
BSplCLib::Hunt() documentation has been corrected to reflect its actual implementation.

5 years ago0030571: Visualization, TKOpenGl - mapped texture is ignored while drawing points IR-2019-03-15
kgv [Sat, 16 Mar 2019 13:47:33 +0000 (16:47 +0300)]
0030571: Visualization, TKOpenGl - mapped texture is ignored while drawing points

OpenGl_AspectsTextureSet now appends Sprite texture to the end of texture set.
OpenGl_Context::init() - fixed usage of GL_MAX_COMBINED_TEXTURE_IMAGE_UNITS on too old GL context.

Default sampler names occSampler0, occActiveSampler and occSamplerBaseColor
are now excluded from header with common declarations for build-in GLSL programs.

OpenGl_ShaderObject::LoadAndCompile() and OpenGl_ShaderProgram::Link()
now includes program Id within error log.

OpenGl_ShaderManager now supports mapping texture onto point primitive within built-in programs
(texture transformation is ignored).

5 years ago0030511: Mesh - too long meshing of assembly containing single solid (shared)
oan [Mon, 4 Mar 2019 06:41:34 +0000 (09:41 +0300)]
0030511: Mesh - too long meshing of assembly containing single solid (shared)

Added test case.

5 years ago0030573: OSD_Parallel_TBB: limit number of execution threads using settings of OSD_Th...
oan [Thu, 14 Mar 2019 16:22:04 +0000 (19:22 +0300)]
0030573: OSD_Parallel_TBB: limit number of execution threads using settings of OSD_ThreadPool::DefaultPool()

Add tbb::task_scheduler_init to OSD_Parallel::forEach().

5 years ago0030575: Tests - v3d/materials unstable image screenshots of alpha-blending on GeForc...
kgv [Thu, 14 Mar 2019 21:12:12 +0000 (00:12 +0300)]
0030575: Tests - v3d/materials unstable image screenshots of alpha-blending on GeForce with MSAA

testdiff now looks for "IMAGE_COLOR_TOLERANCE: Tol" statements in log
to execute diffimage with -toleranceOfColor Tol parameter.

Image difference is first performed with 0 tolerance and then optionally with non-zero tolerance,
so that the log always includes information that some images are different,
but with two different messages:
- "IMAGE differs" for images failed comparison;
- "IMAGE is similar" for images passed comparison only with non-zero tolerance.

v3d/materials - disabled MSAA and removed duplicate.
demo/samples/materials - compare screenshots with 0.8% color tolerance (2/255).

5 years ago0030580: Configuration, CMake - introduce BUILD_RELEASE_DISABLE_EXCEPTIONS option...
kgv [Fri, 15 Mar 2019 11:24:15 +0000 (14:24 +0300)]
0030580: Configuration, CMake - introduce BUILD_RELEASE_DISABLE_EXCEPTIONS option managing No_Exception macros

5 years ago0030560: Modeling Algorithms - Improvement of the method BRepTools::DetectClosedness
jgv [Thu, 14 Mar 2019 09:38:09 +0000 (12:38 +0300)]
0030560: Modeling Algorithms - Improvement of the method BRepTools::DetectClosedness

1. Rewrite the method BRepTools::DetectClosedness.
2. Correct the method BRepOffset_Tool::EnLargeFace to process properly non-periodic but closed faces.

5 years ago0030570: Visualization, TKOpenGl - OpenGl_AspectFace ignores texture revision on...
kgv [Wed, 13 Mar 2019 18:46:24 +0000 (21:46 +0300)]
0030570: Visualization, TKOpenGl - OpenGl_AspectFace ignores texture revision on recompute

Added revision check to one more branch within OpenGl_AspectsTextureSet::build().

5 years ago0030564: Coding - math_Gauss uninitialized 'Singular' member variable
kgv [Wed, 13 Mar 2019 04:49:59 +0000 (07:49 +0300)]
0030564: Coding - math_Gauss uninitialized 'Singular' member variable

Removed unused class field Singular.

5 years ago0030562: Coding - TopOpeBRepBuild_Builder use of null pointer
kgv [Tue, 12 Mar 2019 17:22:26 +0000 (20:22 +0300)]
0030562: Coding - TopOpeBRepBuild_Builder use of null pointer

TopOpeBRepBuild_Builder::Clear() - added NULL check.

5 years ago0030124: Visualization, AIS_InteractiveObject - clean up confusing Presentation inval...
mnv [Thu, 7 Mar 2019 17:45:58 +0000 (20:45 +0300)]
0030124: Visualization, AIS_InteractiveObject - clean up confusing Presentation invalidation logic

Removed excess mechanism for invalidating presentations AIS_InteractiveObject::myRecomputeEveryPrs.
PrsMgr_PresentableObject::Update() has been deprecated and replaced by PrsMgr_PresentableObject::UpdatePresentations()
recomputing only explicitly invalidated presentations.

5 years ago0030554: Coding - ChFiDS_CommonPoint uninitialized member traarc
kgv [Mon, 11 Mar 2019 20:35:14 +0000 (23:35 +0300)]
0030554: Coding - ChFiDS_CommonPoint uninitialized member traarc

Added missing initialization to ChFiDS_CommonPoint constructor.

5 years ago0030553: Coding - TopOpeBRepDS_Surface missing default initalizations
kgv [Mon, 11 Mar 2019 20:20:44 +0000 (23:20 +0300)]
0030553: Coding - TopOpeBRepDS_Surface missing default initalizations

All class fields are now initialized within every TopOpeBRepDS_Surface constructor.

5 years ago0030545: Documentation - methods marked with Standard_DEPRECATED has ill-formed Doxyg...
kgv [Fri, 8 Mar 2019 12:52:08 +0000 (15:52 +0300)]
0030545: Documentation - methods marked with Standard_DEPRECATED has ill-formed Doxygen output

Doxygen option PREDEFINED has been extended by Standard_DEPRECATED declaration.

5 years ago0030547: Foundation Classes - NCollection_DoubleMap lacks Seek() method
kgv [Fri, 8 Mar 2019 23:32:11 +0000 (02:32 +0300)]
0030547: Foundation Classes - NCollection_DoubleMap lacks Seek() method

Added Seek1() and Seek2() methods to NCollection_DoubleMap class.

5 years ago0030149: BRepMesh_Delaun: free triangle is left in the mesh after cleanup of auxiliar... IR-2019-03-07
oan [Wed, 6 Mar 2019 11:29:57 +0000 (14:29 +0300)]
0030149: BRepMesh_Delaun: free triangle is left in the mesh after cleanup of auxiliary items

Remove free triangles connected to triangles over nodes of super triangle.
Modification of improved test case.

5 years ago0029570: Visualization, Graphic3d_Aspect - merge Graphic3d_Group aspects
kgv [Sun, 3 Mar 2019 18:07:55 +0000 (21:07 +0300)]
0029570: Visualization, Graphic3d_Aspect - merge Graphic3d_Group aspects

Graphic3d_AspectFillArea3d, Graphic3d_AspectLine3d, Graphic3d_AspectMarker3d
and Graphic3d_AspectText3d have been merged into new class Graphic3d_Aspects.
The old classes are preserved as dummy sub-classes of Graphic3d_Aspects
preserving different per-aspect defaults.

Methods IsGroupPrimitivesAspectSet(), GroupPrimitivesAspect(), FillAreaAspect(),
LineAspect() and MarkerAspect() have been removed from Graphic3d_Group.
Instead, a new method Graphic3d_Group::ReplaceAspects() has been introduced
for replacing existing group aspects.

AIS_Shape now uses new method AIS_InteractiveObject::replaceAspects()
for updating computed groups with new aspects without presentation recomputation
in places where SynchronizeAspects() is not applicable.

OpenGl_AspectFace, OpenGl_AspectLine, OpenGl_AspectMarker
and OpenGl_AspectText have been merged into new class OpenGl_Aspects.

ViewerTest::parseColor() - fix uninitialized alpha component.
Graphic3d_AspectText3d/Prs3d_TextAspect - removed unused properties Space, ExpansionFactor, Angle.
Remove getters Values() deprecated since OCCT 7.1.0.

5 years ago0030470: Modeling Algorithms - Invalid result of offset operation in mode "Complete...
emv [Tue, 5 Feb 2019 11:58:41 +0000 (14:58 +0300)]
0030470: Modeling Algorithms - Invalid result of offset operation in mode "Complete" with join type "Intersection"

The fix is intended to obtain the correct result of offset operation in which the result contained only the small inverse part of the shape, which clearly should not have been included in the result at all.

The fix adds treatment of the inverted edges while removing the inside faces (collapsed ones) - RemoveInsideFaces method:
- Removing the invalid hanging blocks containing the inverted edges.
- Considering the face containing the inverted edges as the invalid one.

Test cases for the issue.

5 years ago0030523: Visualization - Highlighting does not work anymore
Benjamin Bihler [Wed, 6 Mar 2019 12:02:29 +0000 (15:02 +0300)]
0030523: Visualization - Highlighting does not work anymore

Added hints about ZLayer consideration and changing highlight colors to documentation
of AIS_InteractiveContext::SetHighlightStyle() method.

AIS_InteractiveContext::MoveTo() now checks if highlighting style does not use immediate layer
and performs full Redraw() instead of RedrawImmediate().

Improved parsing of ZLayer names within vdisplay, vzlayer and vselprops commands.
vreadpixel now reports syntax errors on wrong input.

5 years ago0030535: Draw Harness - add size argument to vgrid command
vro [Wed, 6 Mar 2019 09:33:53 +0000 (12:33 +0300)]
0030535: Draw Harness - add size argument to vgrid command

vgrid command has been revised and extended with new argument -size.

5 years ago0030532: Visualization - Manipulator for dragging within a plane
anv [Thu, 21 Feb 2019 09:32:33 +0000 (12:32 +0300)]
0030532: Visualization - Manipulator for dragging within a plane

Added dragging sectors to Manipulator presentation.
Test added.

5 years ago0030490: Modeling Algorithms - Incomplete result of Cut operation
emv [Mon, 25 Feb 2019 12:53:27 +0000 (15:53 +0300)]
0030490: Modeling Algorithms - Incomplete result of Cut operation

The problem is fixed by the fix for the issue #0027448, thus integrating the test case only.

5 years ago0030182: Visualization, Image_AlienPixMap - support reading encoded image from memory... IR-2019-03-01
kgv [Thu, 28 Feb 2019 22:26:20 +0000 (01:26 +0300)]
0030182: Visualization, Image_AlienPixMap - support reading encoded image from memory buffer

Added two new Image_AlienPixMap::Load() methods, taking std::istream
and memory buffer (pointer, length) arguments.
This allows reading image from application memory or from file with non-zero offset.

Added Image_AlienPixMap::IsTopDownDefault() static property allowing to query rows order used by linked image library,
so that application might generate UV texture coordinates accordingly.

Added missing Release() to IWICImagingFactory instance.

5 years ago0030533: Data Exchange - Crash during STEP import.
ika [Thu, 28 Feb 2019 14:56:47 +0000 (17:56 +0300)]
0030533: Data Exchange - Crash during STEP import.

Add Null checks to avoid crash during reading faces without bounds.

5 years ago0024437: Visualization - silhouette edges based on OpenGL
asl [Wed, 27 Feb 2019 22:36:32 +0000 (01:36 +0300)]
0024437: Visualization - silhouette edges based on OpenGL

Added new flag Graphic3d_AspectFillArea3d::ToDrawSilhouette() activating silhouette (outline) rendering.

The new feature can simulate fake HLR look-n-feel,
when combined with Aspect_IS_HIDDENLINE interior style (filling object with background color),
face boundary edges (with most continuity flag set to c2 or lower).

Silhouette GLSL program is very simple - it displaces model alongside vertex normal,
so that it is applicable only to smooth surfaces and closed volumes,
and produces visual artifacts at sharp corners, especially when face boundary is disabled.

OpenGl_SetOfShaderPrograms has been modified so that to reduce
dimensions of the grid of static size based on amount of program combinations.

OpenGl_PrimitiveArray no more allocates VBO resources if primitive array
is marked to be not drawn via interior style / line type / marker type.

5 years ago0030497: [REGRESSION] Mesh - wrong Poly_Polygon3D within local selection of located...
nds [Thu, 28 Feb 2019 06:55:53 +0000 (09:55 +0300)]
0030497: [REGRESSION] Mesh - wrong Poly_Polygon3D within local selection of located shape

The previous code has a condition to avoid processing the same faces if the face has a location.
The similar condition should be applied during the edges processing.
If not doing this, in the previous implementation, IMeshData_Edge instances are created for all edges(even located), but edges of faces located are not filled with curves.
As a result we had wrong local selection of edges.

Limit addition of edges to data model by ones with unique TShape and location using edges map already available in BRepMesh_ShapeVisitor.

5 years ago0030356: Data Exchange - IGES model after importing into document has location issues
pdn [Wed, 27 Feb 2019 14:02:37 +0000 (17:02 +0300)]
0030356: Data Exchange - IGES model after importing into document has location issues

5 years ago0029787: Visualization - Avoid in presentation edges of certain continuity class
asl [Wed, 27 Feb 2019 16:39:08 +0000 (19:39 +0300)]
0029787: Visualization - Avoid in presentation edges of certain continuity class

A new flag Prs3d_Drawer::FaceBoundaryUpperContinuity() has been introduced
handled by StdPrs_ShadedShape::FillFaceBoundaries() method to exclude edges
of higher continuity class (e.g. to skip seam edges).

Draw Harness command vshowfaceboundary has been replaced by vaspects:
> vaspects -setFaceBoundaryDraw 1 -setFaceBoundaryColor RED -setFaceBoundaryType DASH.

5 years ago0030448: Coding - add typo detection to derivation creation methods using Standard_NO...
Benjamin Bihler [Mon, 25 Feb 2019 09:52:01 +0000 (12:52 +0300)]
0030448: Coding - add typo detection to derivation creation methods using Standard_NODISCARD attribute

Added macro Standard_NODISCARD equivalent to C++17 attribute [[nodiscard]] for compilers that support this.
Using Standard_NODISCARD macro for methods that create new object in gp, math, Geom, Bnd packages.
Marked equivalent operators with Standard_NODISCARD, if they are defined close to relevant methods.

Corrected code where warnings on unused result of calls to methods creating new objects are generated.
In most cases it looks like spelling errors (e.g. Normalised() instead of Normalise())

5 years ago0030493: Draw, ViewerTest - minor improvement of vdisplay command
kgv [Tue, 12 Feb 2019 09:22:26 +0000 (12:22 +0300)]
0030493: Draw, ViewerTest - minor improvement of vdisplay command

vdisplay no longer opens BREP file in case if Draw variable is not found.

Draw::Get() unused Complain argument has been removed from method definition.
Added Draw::GetExisting() and DBRep::GetExisting() commands never performing picking.

5 years ago0029810: Visualization - Tool for debugging shaders
asl [Tue, 26 Feb 2019 10:56:03 +0000 (13:56 +0300)]
0029810: Visualization - Tool for debugging shaders

A new tool for debugging shaders has been introduced.
The new method OpenGl_ShaderProgram::UpdateDebugDump() allows dynamically
dumping/restoring certain shader program (vertex and fragment shaders) to/from external file.

The file name is generated from the program's name with suffix ".vs" or ".fs" for shader type.
The environment variable CSF_ShadersDirectoryDump specifies the folder for dumps.

If the file does not exist (first frame), then it is automatically saved.
When the file date/time is changed in comparison with recent cached date,
then the file will be automatically loaded from external file,
thus this file can be modified in any editor.

OpenGl_ShaderManager now generates a human-readable resource ids for standard GLSL programs.

Draw Harness command vshader has been extended with arguments -list,-dump and -reload
for debugging shaders within OpenGl_Context.

5 years ago0030520: VIS - IVtkTools_ShapePicker::GetPickPosition() returns incorrect point
kgv [Tue, 26 Feb 2019 05:17:19 +0000 (08:17 +0300)]
0030520: VIS - IVtkTools_ShapePicker::GetPickPosition() returns incorrect point

vmoveto and ivtkmoveto commands now print topmost picked 3D point.

5 years ago0029996: Porting to VC 2017: memory leak reported in visualisation
abv [Tue, 26 Feb 2019 10:07:09 +0000 (13:07 +0300)]
0029996: Porting to VC 2017: memory leak reported in visualisation

Tests for memory leaks on display / erase cycles are performed with VBO disabled to avoid false memory leaks reported on NVidia graphics

5 years ago0029076: Visualization - implement element shrinking Shader IR-2019-02-22
mnv [Wed, 1 Aug 2018 15:09:37 +0000 (18:09 +0300)]
0029076: Visualization - implement element shrinking Shader

Aspect_IS_HOLLOW now an alias to Aspect_IS_EMPTY and Aspect_IS_HIDDENLINE does not implicitly enables mesh edges,
so that Graphic3d_AspectFillArea3d::SetDrawEdges() should be set independently.

OpenGl_ShaderManager now provides built-in GLSL programs for drawing mesh edges
in single pass (and on OpenGL ES which does not provide glPolygonMode()).

Graphic3d_RenderingParams::ToEnableAlphaToCoverage is now enabled by default
and properly handled at TKOpenGl level - enables coverage for Graphic3d_AlphaMode_Mask primitives.

OpenGl_PrimitiveArray now uses GLSL programs instead of glPolygonMode() by default,
which can be managed by OpenGl_Caps::usePolygonMode flag (desktop OpenGL only).
glPolygonMode() is also used as fallback regardless OpenGl_Caps::usePolygonMode flag
when GLSL programs are not supported (Geometry Shaders are required)
or stipple line style is required (not implemented within Face GLSL).

vaspects command has been extended by -setInterior -setDrawEdges -setEdgeColor -setEdgeType -setEdgeWidth
arguments replacing vsetinteriorstyle/vsetedgetype/vunsetedgetype commands.
vaspects now accepts arguments without "set" prefix.
ViewerTest::ParseColor() now parses RGBA color.

Redundant command BUC60738 has been removed.
AIS_ColorScale - fixed usage of uninitialized FillArea aspects.

5 years ago0030510: Application Framework - add missing NULL check within TDataStd_RealArray...
jfa [Thu, 21 Feb 2019 09:49:05 +0000 (12:49 +0300)]
0030510: Application Framework - add missing NULL check within TDataStd_RealArray::ChangeArray()

5 years ago0030384: Visualization - Won't display line with dot and dash style on Android
kgv [Wed, 20 Feb 2019 21:23:19 +0000 (00:23 +0300)]
0030384: Visualization - Won't display line with dot and dash style on Android

Fix misprint leading to double semicolon in GLSL code.

5 years ago0030498: Warnings of building documentation
apn [Mon, 18 Feb 2019 11:59:32 +0000 (14:59 +0300)]
0030498: Warnings of building documentation

Fix warning 'Reached end of file while still insided a (nested) comment in Markdown' for old versions of doxygen

5 years ago0030476: Visualization, Path Tracing - Adaptive Screen Sampling leads to unstable... IR-2019-02-15
kgv [Mon, 11 Feb 2019 15:00:35 +0000 (18:00 +0300)]
0030476: Visualization, Path Tracing - Adaptive Screen Sampling leads to unstable results

OpenGl_View::runPathtrace() has been extended with alternative multi-pass
Adaptive Screen Sampling mode, not relying on atomic floating point operations.
Although atomic operations on floats allows single-pass rendering,
such operations leads to instability in case of different calculation order.
Atomic float operations are also currently supported only by single GPU vendor.

Fixed GLSL compilation on Intel drivers (follow ARB_shader_image_load_store
specs rather than EXT_shader_image_load_store).

Graphic3d_RenderingParams::AdaptiveScreenSamplingAtomic option has been added
to activate 1-pass Adaptive Screen Sampling mode when supported by hardware.

vfps command has been extended with -duration argument allowing to limit command execution time.
vactivate command has been extended with -noUpdate argument.

5 years ago0030483: Visualization, Path Tracing - make Tile Size configurable
kgv [Wed, 6 Feb 2019 16:21:23 +0000 (19:21 +0300)]
0030483: Visualization, Path Tracing - make Tile Size configurable

OpenGl_TileSampler has been refactored to better describe its logic:
- Offset image now defines tile index instead of offset to tile origin.
- Added 2D array defining the number of times to sample tile for straight-forward debugging.

Graphic3d_RenderingParams has been extended with property
RayTracingTileSize for testing various tile configurations.
Default behavior is the following:
- Target number of tiles (e.g. upper limit per frame): 256
- Tile size: 32x32.

OpenGl_View::runPathtrace() has been split into two methods per rendering stage.
OpenGl_Texture::Init() now returns FALSE immediately on 0 input dimensions.

Added Image_PixMapTypedData template class allowing to work with image data of known pixel format.

5 years ago0028488: VIS - fix compilation with VTK 8.2
kgv [Fri, 15 Feb 2019 08:27:15 +0000 (11:27 +0300)]
0028488: VIS - fix compilation with VTK 8.2

Occurrences of removed method vtkDataArray::InsertNextTupleValue() have been replaced by InsertNextTypedTuple().
Fixed misprint in vtkTypeMacro usage for class IVtkTools_ShapeObject.
Patch #0030452 (SelectMgr_ViewerSelector::Deactivate() raises exception when called twice) has been propagated to IVtkOCC_ViewerSelector.

5 years ago0030486: Visualization - Rubber-band selection does not work with disabled overlap...
kgv [Fri, 8 Feb 2019 22:38:28 +0000 (01:38 +0300)]
0030486: Visualization - Rubber-band selection does not work with disabled overlap detection

SelectMgr_RectangularFrustum::Overlaps() now treats degenerated polygon as point instead of returning FALSE.

5 years ago0030477: Visualization - crash in AIS_Manipulator::ObjectTransformation after using...
kgv [Mon, 11 Feb 2019 09:01:40 +0000 (12:01 +0300)]
0030477: Visualization - crash in AIS_Manipulator::ObjectTransformation after using V3d_View::SetProj(V3d_Zpos)

5 years ago0030488: Visualization, Ray Tracing - empty error message on GLSL program compilation
kgv [Thu, 7 Feb 2019 14:24:47 +0000 (17:24 +0300)]
0030488: Visualization, Ray Tracing - empty error message on GLSL program compilation

OpenGl_ShaderObject::LoadAndCompile() - added method combining Load() and Compile() with optional error logging.
OpenGl_ShaderProgram::Link() now logs failures by default.

5 years ago0030478: Foundation Classes, NCollection_Array2 - provide Resize/Move methods consist...
kgv [Wed, 6 Feb 2019 09:29:21 +0000 (12:29 +0300)]
0030478: Foundation Classes, NCollection_Array2 - provide Resize/Move methods consistent to NCollection_Array1

5 years ago0030494: Coding Rules - eliminate declaration of OpenGl_Workspace::IsCullingEnabled()
kgv [Tue, 12 Feb 2019 06:37:30 +0000 (09:37 +0300)]
0030494: Coding Rules - eliminate declaration of OpenGl_Workspace::IsCullingEnabled()

5 years ago0030129: With newest doxygen 1.8.14 generation of overview gives warnings
abv [Fri, 8 Feb 2019 14:29:55 +0000 (17:29 +0300)]
0030129: With newest doxygen 1.8.14 generation of overview gives warnings

Doxygen configuration scripts are corrected to avoid '\n' symbols in the end of the @figure alias that caused multiple warning messages.
Other Doxygen warnings in Markdown files are eliminated.
In CMake builds the format of Doxygen warnings is set to use the style recognized by Visual Studio when compiler is MSVC.

5 years ago0030480: Visualization - Clear of Select3D_SensitiveGroup does not update internal... IR-2019-02-08
nds [Wed, 6 Feb 2019 17:20:47 +0000 (20:20 +0300)]
0030480: Visualization - Clear of Select3D_SensitiveGroup does not update internal container

5 years ago0028936: Draw freezes while making a 3d representation
bugmaster [Wed, 6 Feb 2019 12:02:48 +0000 (15:02 +0300)]
0028936: Draw freezes while making a 3d representation

Adding testing case

5 years ago0030329: Move BRepMesh_IncAllocator to NCollection package
kgv [Mon, 4 Feb 2019 09:21:14 +0000 (12:21 +0300)]
0030329: Move BRepMesh_IncAllocator to NCollection package

NCollection_IncAllocator has been extended with optional mutex allocation (disabled by default).

5 years ago0030474: Visualization - fallback font is ignored for missing font alias within Font_...
kgv [Mon, 4 Feb 2019 11:04:21 +0000 (14:04 +0300)]
0030474: Visualization - fallback font is ignored for missing font alias within Font_FontMgr::FindFont()

5 years ago0030469: Visualization - Draw crashes when 3D view is minimized
kgv [Fri, 1 Feb 2019 13:20:24 +0000 (16:20 +0300)]
0030469: Visualization - Draw crashes when 3D view is minimized

OpenGl_View::Redraw() - avoid rendering into window of zero size (leading to FPE and other issues).

5 years ago0030366: Coding Rules - GeometryTest_API2dCommands.cxx seems to be excess
kgv [Fri, 1 Feb 2019 14:49:08 +0000 (17:49 +0300)]
0030366: Coding Rules - GeometryTest_API2dCommands.cxx seems to be excess

5 years ago0030464: Visualization - unable to set sub-shape transparency using vaspects command IR-2019-02-01 IR-2019-02-02
kgv [Thu, 31 Jan 2019 11:52:19 +0000 (14:52 +0300)]
0030464: Visualization - unable to set sub-shape transparency using vaspects command

Added AIS_ColoredShape::SetCustomTransparency() for simple way for assigning sub-shape transparency.

5 years ago0030452: Visualization - SelectMgr_ViewerSelector::Deactivate() raises exception...
kgv [Mon, 21 Jan 2019 12:31:02 +0000 (15:31 +0300)]
0030452: Visualization - SelectMgr_ViewerSelector::Deactivate() raises exception when called twice

Removed unused SelectMgr_SOS_Sleeping enumeration value.
SelectMgr_ViewerSelector::Deactivate() now checks activation status before updating tolerance map.

5 years ago0030450: Visualization - AIS_InteractiveContext::MoveTo() never uses V3d_View::Immedi...
kgv [Mon, 21 Jan 2019 08:57:22 +0000 (11:57 +0300)]
0030450: Visualization - AIS_InteractiveContext::MoveTo() never uses V3d_View::ImmediateRedraw()

AIS_InteractiveContext::MoveTo() now calls V3d_View::ImmediateRedraw() when applicable.
AIS_Trihedron::HilightOwnerWithColor() - added missing ZLayer setup.

5 years ago0030405: Presentations of PMI are incorrectly scaled after change of units IR-2019-01-25
gka [Wed, 23 Jan 2019 15:37:23 +0000 (18:37 +0300)]
0030405: Presentations of PMI are incorrectly scaled after change of units

Corrected search of the entity StepShape_ShapeDimensionRepresentation defining used units in the method readConnectionPoints

5 years ago0030451: Selection mode of TPrsStd_AISPresentation attribute is restricted to one...
vro [Tue, 22 Jan 2019 10:47:09 +0000 (13:47 +0300)]
0030451: Selection mode of TPrsStd_AISPresentation attribute is restricted to one value

Two classes TDataXtd_Presentation and TPrsStd_AISPresentation were improved so that they accept a list of selection modes.
A new non-regression test is added: caf presentation N1

5 years ago0030429: Samples - add simple glfw 3D Viewer sample
mahaidong [Fri, 18 Jan 2019 08:00:30 +0000 (11:00 +0300)]
0030429: Samples - add simple glfw 3D Viewer sample

Aspect_DisplayConnection now provides constructor wrapping existing X Display connection.

5 years ago0030439: Visualization - extend fonts search within Font_FontMgr::FindFont() on Linux
kgv [Thu, 10 Jan 2019 20:10:51 +0000 (23:10 +0300)]
0030439: Visualization - extend fonts search within Font_FontMgr::FindFont() on Linux

Font_FontMgr has been redesigned to:
- Store fonts in a map instead a list.
- Allow mapping multiple fonts to a single alias.
- Log informative message about usage of non-requested font (fallback).
- Register all font files within standard folders on Linux when "fonts.dir" is not found.
- Prefer specific alias ("serif") as default fallback font instead of arbitrary one in a system.

A couple of obsolete and broken font aliases have been removed;
instead, new aliases of fonts popular on Linux platform have been added.

Font_NameOfFont.hxx has been extended with more neutral aliases
"monospace", "serif", "sans-serif", "cjk" and "korean".

Font_FontAspect enumeration values have been renamed Font_FA_ -> Font_FontAspect_
with old values preserved as alias.

Font_SystemFont has been extended with a list of paths to Font_FontAspect styles,
so that entire Font Family is now defined within a single Font_SystemFont instance.
Non-resizable fonts are now ignored by Font Manager.

5 years ago0029670: Draw Harness - vtrihedron xaxis zaxis is wrong
kgv [Thu, 10 Jan 2019 12:36:23 +0000 (15:36 +0300)]
0029670: Draw Harness - vtrihedron xaxis zaxis is wrong

Fixed gp_Ax2 initialization by two vectors.
Fixed -labels arguments number check.

5 years ago0030437: Visualization, TKV3d - add Draw command to print rendering statistics IR-2019-01-11
osa [Thu, 10 Jan 2019 14:25:48 +0000 (17:25 +0300)]
0030437: Visualization, TKV3d - add Draw command to print rendering statistics

Add new command "vstatprofiler" to manage rendering parameters and print them.
If there are some input parameters - print corresponding statistic counters values,
else - print all performance counters set previously.

5 years ago0030434: Visualization, TKV3d - add "NoUpdate" state of frustum culling optimization
osa [Wed, 9 Jan 2019 14:01:18 +0000 (17:01 +0300)]
0030434: Visualization, TKV3d - add "NoUpdate" state of frustum culling optimization

Frustum culling is now managed by Graphic3d_RenderingParams::FrustumCullingState flag
and can be switched into Graphic3d_RenderingParams::FrustumCulling_NoUpdate state
useful for debugging the algorithm.

Draw Harness command vrustumculling has been replaced by vrenderparams -frustumCulling.

5 years ago0030412: Visualization, TKV3d - add presentation of camera frustum
osa [Mon, 17 Dec 2018 09:01:17 +0000 (12:01 +0300)]
0030412: Visualization, TKV3d - add presentation of camera frustum

1) Added method Graphic3d_Camera::FrustumPoints() returning corner points of camera frustum.
2) Refactored methods OpenGl_BVHTreeSelector::isFullOut(...) and OpenGl_BVHTreeSelector::CacheClipPtsProjections()
3) Changed computation algorithm of frustum planes (build them by corner points)
4) Added interactive object AIS_CameraFrustum to draw camera frustum.
5) Extended Draw command "vcamera" with option displaying camera frustum.

5 years ago0030349: Coding - add usage of Standard_EXPORT for some not inline methods in Select3D
nds [Wed, 7 Nov 2018 15:30:21 +0000 (18:30 +0300)]
0030349: Coding - add usage of Standard_EXPORT for some not inline methods in Select3D

5 years ago0030430: Draw - command testgrid in parallel mode hangs if DRAW is launched without GUI
abv [Thu, 10 Jan 2019 05:17:04 +0000 (08:17 +0300)]
0030430: Draw - command testgrid in parallel mode hangs if DRAW is launched without GUI

Ensure that initialization of Tcl interpretor is performed in the same thread where commands are evaluated.

Added test demo draw bug30430

5 years ago0030377: DRAW, Windows - command executed via option -c fails on puts
abv [Sun, 18 Nov 2018 16:53:10 +0000 (19:53 +0300)]
0030377: DRAW, Windows - command executed via option -c fails on puts

When DRAW on Windows is launched with option -c, the command is now properly transferred to Tcl thread (separate thread that runs Tcl interpretor on Windows except when DRAW is run in batch mode) for execution, instead of being evaluated in the main thread.

Execution of DRAW in batch mode (option -b) is fixed by enabling proper initialization of the Tcl interpretor and replacement of backslashes in path to startup script by straight slashes on Windows in that mode.

Declaration of global variables used for communication of console command between threads is moved to Draw_Window.hxx to ensure consistency.
Function wscpy_s is used instead of memcpy to avoid possible buffer overrun.

5 years ago0030428: Modeling Algorithms - BRepBndLib::AddOBB() throws exception on empty Compound
kgv [Sat, 29 Dec 2018 12:41:54 +0000 (15:41 +0300)]
0030428: Modeling Algorithms - BRepBndLib::AddOBB() throws exception on empty Compound

Added missing Bnd_Box::IsVoid() check.

5 years ago0030424: C# wrapper - implement IEnumerable interface by NCollection classes IR-2018-12-29
kgv [Tue, 25 Dec 2018 17:25:55 +0000 (20:25 +0300)]
0030424: C# wrapper - implement IEnumerable interface by NCollection classes

TopExp_Explorer::Value() - added Current() alias for providing interface consistent to other OCCT Iterators.

5 years ago0030346: Modeling Algorithms - BRepPrimAPI_MakeRevol throws "BRepSweep_Translation...
ifv [Mon, 10 Dec 2018 12:48:55 +0000 (15:48 +0300)]
0030346: Modeling Algorithms - BRepPrimAPI_MakeRevol throws "BRepSweep_Translation::MakeEmptyVertex"

Implementation of method "IsDeleted(...)" for MakeRevol and MakePrism algorithms.

Problem (exception) occurs during history building and was caused by an attempt to obtain generated shape for subshape, which was really "deleted" by algorithm - this input subshape and its possible generated shape was not used in result.

5 years ago0030364: Visualization, TKOpenGl - allow initializing a Surface-less EGL context
kgv [Mon, 12 Nov 2018 12:46:40 +0000 (15:46 +0300)]
0030364: Visualization, TKOpenGl - allow initializing a Surface-less EGL context

OpenGl_Context::MakeCurrent()/OpenGl_Context::IsCurrent() have been modified to NOT fail
in case if myWindow is EGL_NO_SURFACE (valid off-screen rendering case within EGL).
OpenGl_GraphicDriver::InitEglContext() now finds EGL surface config in case if it has not been passed by argument.

OpenGl_Window constructor now allows wrapping an off-screen rendering surface EGL_NO_SURFACE.
However, it still creates a dummy surface eglCreatePbufferSurface() to workaround bugs in some GLES drivers (Vivante GC2000).

V3d_View::SetWindow()/V3d_View::MustBeResized()/V3d_Viewer::SetViewOn() have been modified
to avoid implicit View redraw (leading to undefined behavior/crashes in case if rendering
should be done into default FBO defined right after V3d_View initialization).

5 years ago0030396: Infinite recursion during ShapeFix after BRepAlgoAPI_Cut IR-2018-12-21
msv [Thu, 20 Dec 2018 14:10:00 +0000 (17:10 +0300)]
0030396: Infinite recursion during ShapeFix after BRepAlgoAPI_Cut

Two test cases are added. One for Boolean operation (it is good, because BO has been fixed already), and another for fixshape operation (it causes Draw crash).

5 years ago0029964: Bad arc discretization
oan [Tue, 13 Nov 2018 10:08:37 +0000 (13:08 +0300)]
0029964: Bad arc discretization

Added test cases.

5 years ago0027976: Unstable work of 2d intersection algorithm for two lines
nbv [Thu, 29 Nov 2018 07:02:14 +0000 (10:02 +0300)]
0027976: Unstable work of 2d intersection algorithm for two lines

1. Test case for the issue has been created.
The problem is still reproduced on MASTER.

2. Some test cases have been moved to lowalgos/2dinter grid.