4 years ago0026450: Boolean operation is running endless
apn [Wed, 23 Nov 2016 15:07:21 +0000 (18:07 +0300)]
0026450: Boolean operation is running endless

Added test case bugs/modalg_6/bug26450

4 years ago0028147: Data Exchange - NULL dereference while reading color from STEP file
kgv [Wed, 23 Nov 2016 12:53:48 +0000 (15:53 +0300)]
0028147: Data Exchange - NULL dereference while reading color from STEP file

StepVisual_StyledItem::Item() now does not call IsKind() on NULL object.
Add several checks for NULL to import/export.

Add test case bugs/step/bug28147

4 years ago0025920: BRepAlgoAPI_Fuse makes invalid result
msv [Mon, 21 Nov 2016 08:24:23 +0000 (11:24 +0300)]
0025920: BRepAlgoAPI_Fuse makes invalid result

The command OCC426 for test case bugs modalg_2 bug426 has been modified in order to make the test passed. The modification concerns only the old Boolean operation branch (BRepAlgo_Fuse).

4 years ago0028143: In the method ShapeFix_Edge::FixAddPCurve initialized by face location of...
gka [Tue, 22 Nov 2016 13:20:36 +0000 (16:20 +0300)]
0028143: In the method ShapeFix_Edge::FixAddPCurve initialized by face location of face is not taking into account.

Class ShapeAnalysis::Surface is initialized by transformed surface in the method
ShapeFix_Edge::FixAddPCurve (const TopoDS_Edge& edge,const TopoDS_Face& face, const Standard_Boolean isSeam, const Standard_Real prec) in the case when face has location.

4 years ago0028077: Add option for genproj to generate UWP projects
ski [Fri, 11 Nov 2016 14:00:25 +0000 (17:00 +0300)]
0028077: Add option for genproj to generate UWP projects

Ability to generate UWP solution was added to genproj command, applied when VC version set to "vc14-uwp".
Environment variables are used to find installed Microsoft SDKs.

Visual Studio project option "Use precompiled headers" is set to "Not Using" by default.
Character set option of VS project Unicode is used instead of preprocessor definitions for UWP projects.

4 years ago0028146: Data files with the same content in test cases
mkv [Tue, 22 Nov 2016 16:22:56 +0000 (19:22 +0300)]
0028146: Data files with the same content in test cases

Names of used shapes were corrected in test cases

4 years ago0028038: Incorrect documentation generated for "user guides/BRep Format"
aiv [Mon, 21 Nov 2016 15:23:28 +0000 (18:23 +0300)]
0028038: Incorrect documentation generated for "user guides/BRep Format"

References to external MathJax.js library were changed from HTTP to HTTPS alternative.

4 years ago0028105: HLR rendering crash in MFC sample
ski [Fri, 18 Nov 2016 11:31:58 +0000 (14:31 +0300)]
0028105: HLR rendering crash in MFC sample

User_Cylinder class was updated according to work as single object in Prs3d_Presentation.

4 years ago0028127: Visualization - transparent object breaks Z-Layer depth buffer clear
kgv [Fri, 18 Nov 2016 13:00:59 +0000 (16:00 +0300)]
0028127: Visualization - transparent object breaks Z-Layer depth buffer clear

OpenGl_LayerList::Render() now calls glDepthMask(GL_TRUE) before clearing depth buffer.
Depth buffer is now cleared even if ZLayer with this command has no structures.

4 years ago0028134: TDataStd_Name attribute gets null ID.
szy [Mon, 21 Nov 2016 11:14:03 +0000 (14:14 +0300)]
0028134: TDataStd_Name attribute gets null ID.

'myID' field is initialized in constructor now.

4 years ago0028046: Incomplete result of offset operation in mode complete intersection
emv [Mon, 14 Nov 2016 07:19:15 +0000 (10:19 +0300)]
0028046: Incomplete result of offset operation in mode complete intersection

1. Special usage of the common edges not belonging to any invalidity while rebuilding the invalid faces (method IntersectFaces() in BRepOffset_MakeOffset_1.cxx).
Such edges could only be extended to the valid faces boundaries, thus could only split the face and not create any new part.

2. Test cases for the issue.

4 years ago0027849: ResourceManager path computations fail for the folders containing dots
razmyslovich [Thu, 8 Sep 2016 08:09:12 +0000 (10:09 +0200)]
0027849: ResourceManager path computations fail for the folders containing dots

Treatment of paths in Resource_Manager is corrected to handle properly paths with dots inside.
Extraction of extension from path in DOS mode when working on Linux is corrected in OSD_Path.
Test case added.

4 years ago0025560: Documentation - describe changes in AIS_InteractiveConnect and AIS_MultipleC...
ysn [Wed, 23 Nov 2016 09:57:32 +0000 (12:57 +0300)]
0025560: Documentation - describe changes in AIS_InteractiveConnect and AIS_MultipleConnected introduced within 6.8.0

 New section about scene-graph hierarchy and instancing.
 Proofreading of extended section about z-layers.
 Removal of obsolete information.

Remarks taken into account.

4 years ago0028130: Spelling error in description of class BOPAlgo_Section
abv [Sat, 19 Nov 2016 14:21:59 +0000 (17:21 +0300)]
0028130: Spelling error in description of class BOPAlgo_Section

Spelling error is corrected

4 years ago0028103: Foundation Classes - extend NCollection_Map to check whether two given maps...
abk [Mon, 14 Nov 2016 15:08:44 +0000 (18:08 +0300)]
0028103: Foundation Classes - extend NCollection_Map to check whether two given maps are intersected

Mehtod HasIntersection added in NCollection_Map to check whether two given maps contain at least one same item.
'Draw' command and test to check the method were created.

4 years ago0027256: Visualization, Path Tracing - add support of alpha-channel of texture
dbp [Thu, 17 Nov 2016 15:51:38 +0000 (18:51 +0300)]
0027256: Visualization, Path Tracing - add support of alpha-channel of texture

Regenerate resource files.

4 years ago0028114: Visualization, Path tracing - Make path tracing mode interactive in high...
dbp [Wed, 16 Nov 2016 12:44:56 +0000 (15:44 +0300)]
0028114: Visualization, Path tracing - Make path tracing mode interactive in high resolutions

4 years ago0028101: Visualization, Select3D_SensitiveSet - fix NULL dereference on re-adding...
kgv [Thu, 17 Nov 2016 12:51:44 +0000 (15:51 +0300)]
0028101: Visualization, Select3D_SensitiveSet - fix NULL dereference on re-adding the same sensitivity

Select3D_SensitiveSet now stores BVH_PrimitiveSet as class field (no dynamic allocation),
and BVH_PrimitiveSet subclass now stores raw pointer to Select3D_SensitiveSet (no smart pointer).

Select3D_BVHPrimitiveContent definition has been moved into Select3D_SensitiveSet
class definition to avoid confusion.

Unused field Select3D_SensitiveSet::myIsLeftChildQueuedFirst has been removed.

4 years ago0027992: Modeling Algorithms - Extrema_ExtPS crashes on face without geometric surface
msv [Thu, 17 Nov 2016 12:50:16 +0000 (15:50 +0300)]
0027992: Modeling Algorithms - Extrema_ExtPS crashes on face without geometric surface

The algorithm BRepExtrema_DistShapeShape has been protected against exceptions when non-geometric shape data is given on input, like a face containing triangulation only or an edge containing polygon only. Such faces/edges are ignored by the algorithm.

BRepGProps::VolumeProperties() now ignores faces without geometric surface to avoid access violation.

4 years ago0028115: Visualization, Ray tracing - Fix crash on NVIDIA GPUs with driver version...
dbp [Wed, 16 Nov 2016 14:47:07 +0000 (17:47 +0300)]
0028115: Visualization, Ray tracing - Fix crash on NVIDIA GPUs with driver version 373.06 when window is resized

4 years ago0028095: Draw Harness, ViewerTest - use RGBA format instead of BGRA within vreadpixel
kgv [Thu, 17 Nov 2016 12:39:52 +0000 (15:39 +0300)]
0028095: Draw Harness, ViewerTest - use RGBA format instead of BGRA within vreadpixel

OpenGl_Workspace::BufferDump() now implicitly converts RGBA dump
into requested BGR, BGRA and RGB image.

DRAW command dversion is improved to report OpenGL variant used (desktop or ES); reporting of version of MSVC is corrected for VC14 and above; reporting of HAVE_OPENCL option is removed.

Usage of command vdump is corrected in some tests to specify extension .png for an image file.

Compiler warning is eliminated in OpenGl_Text.cxx (OpenGL ES mode only).

4 years ago0025144: Coding rules - consider rules for formatting pointers and constants placemen...
kgv [Wed, 13 Aug 2014 06:19:55 +0000 (10:19 +0400)]
0025144: Coding rules - consider rules for formatting pointers and constants placement in expressions

4 years ago0028094: Shape obtained after DRAW command "splitshape" has unnecessary high tolerance.
gka [Fri, 11 Nov 2016 14:08:08 +0000 (17:08 +0300)]
0028094: Shape obtained after DRAW command "splitshape" has unnecessary high tolerance.

Check of the gap between ends of the p-curves having common vertex in the parametric space of face similar as in the class BRepCheck_Wire was added.
Check that face lies on the periodic surface was added.
Added check between ends of 2D curves in the 3D space and vertex the same which used in the BRepCheck_Wire::SelfIntersect
Test case for issue CR28094 has been created.

4 years ago0028093: Visualization - access violation on removal of dynamic highlighting after...
kgv [Fri, 11 Nov 2016 15:20:49 +0000 (18:20 +0300)]
0028093: Visualization - access violation on removal of dynamic highlighting after device loss

OpenGl_Layer::Remove() now tries to remove object from myAlwaysRenderedMap
if it was not removed from other groups.

4 years ago0028079: Proofreading of updates section and possible documentation changes since 7.0
ysn [Wed, 16 Nov 2016 09:07:55 +0000 (12:07 +0300)]
0028079: Proofreading of updates section and possible documentation changes since 7.0

Added some content from the bugtracker to porting notes.
Reviewed some other recent changes.

4 years ago0027828: Reference documentation of AIS_Trihedron contains invalid information about...
ysn [Wed, 16 Nov 2016 14:08:24 +0000 (17:08 +0300)]
0027828: Reference documentation of AIS_Trihedron contains invalid information about default size.

Fixed the description of AIS_Trihedron

4 years ago0028099: Visualization, OpenGl_Text - handle DIMENSION and SUBTITLE styles within...
kgv [Mon, 14 Nov 2016 10:14:04 +0000 (13:14 +0300)]
0028099: Visualization, OpenGl_Text - handle DIMENSION and SUBTITLE styles within Core Profile

OpenGl_Text now creates VBO for drawing background rectangle.

4 years ago0028096: VC14 compiler warnings of 7.1.0.beta
abv [Thu, 17 Nov 2016 12:14:19 +0000 (15:14 +0300)]
0028096: VC14 compiler warnings of 7.1.0.beta

Warnings are eliminated
// minor corrections in headers

4 years ago0028092: Wrong statement in reference manual for Geom_ToroidalSurface::Bounds
msv [Fri, 11 Nov 2016 12:49:41 +0000 (15:49 +0300)]
0028092: Wrong statement in reference manual for Geom_ToroidalSurface::Bounds

Description has been corrected.

4 years ago0028061: Samples - Highlighting of any object with "Hidden On" option leads to crash...
vpa [Wed, 9 Nov 2016 14:55:51 +0000 (17:55 +0300)]
0028061: Samples - Highlighting of any object with "Hidden On" option leads to crash in mfc Modeling

Graphic3d_Structure::Highlight() does not call UnHighlight() anymore.

4 years ago0028081: Fix regression in HLR introduced by the fix for 28053
msv [Wed, 9 Nov 2016 15:27:47 +0000 (18:27 +0300)]
0028081: Fix regression in HLR introduced by the fix for 28053

The whole piece of the walking line between two vertices is no more checked for having length greater than Precision::Confusion(). Instead, only full duplicate points (comparing with gp::Resolution()) are removed, allowing the resulting line have any non-null length.

The test bugs modalg_6 bug27720_5 has been reverted to its original state.

4 years ago0028076: Test bugs moddata_2 bug469 fails when FPE are enabled
nbv [Wed, 9 Nov 2016 08:22:12 +0000 (11:22 +0300)]
0028076: Test bugs moddata_2 bug469 fails when FPE are enabled

The reason of FPE exception has been eliminated.

4 years ago0028056: Draw Harness, ViewerTest - correct vanimation command to produce stable...
kgv [Wed, 9 Nov 2016 13:20:32 +0000 (16:20 +0300)]
0028056: Draw Harness, ViewerTest - correct vanimation command to produce stable timestamp on seek

4 years ago0027575: Reading STEP file gives an empty result
gka [Thu, 17 Nov 2016 11:44:59 +0000 (14:44 +0300)]
0027575: Reading STEP file gives an empty result

Exception is raised during translation BSpline curve having only one unique knot.
Protection for such case was added
Added protection against exception during reading BSpline curve with number of control points equal to 0
Added protection against exception due to using null binder during writing XDE document

Test case for issue #27575

4 years ago0023639: Exception on attempt to save to XmlOcaf format
abv [Tue, 8 Nov 2016 02:08:22 +0000 (05:08 +0300)]
0023639: Exception on attempt to save to XmlOcaf format

Variant FULL is removed from the options of DRAW command pload; ALL can be used in all cases

4 years ago0027428: DRAW command "bsection" did not built section line between two faces (one...
emv [Mon, 7 Nov 2016 22:09:33 +0000 (01:09 +0300)]
0027428: DRAW command "bsection" did not built section line between two faces (one face based on the plane)

Adjusting test case for the issue.

4 years agoIncrementing OCCT version up to 7.1.0 beta V7_1_0_beta
apn [Wed, 9 Nov 2016 09:12:43 +0000 (12:12 +0300)]
Incrementing OCCT version up to 7.1.0 beta

4 years agoUpdate test case offset/shape_type_i_c/Q9 according to new behavior
apn [Wed, 9 Nov 2016 09:08:50 +0000 (12:08 +0300)]
Update test case offset/shape_type_i_c/Q9 according to new behavior

4 years agoUpdated dependencies in mfc OCAF sample
apn [Wed, 9 Nov 2016 09:07:07 +0000 (12:07 +0300)]
Updated dependencies in mfc OCAF sample

4 years ago0028040: Foundation Classes, Message_Printer - handle Standard_CString messages as...
kgv [Wed, 2 Nov 2016 09:56:30 +0000 (12:56 +0300)]
0028040: Foundation Classes, Message_Printer - handle Standard_CString messages as UTF-8

4 years ago0027678: Draw User's guide contains wrong example for for-loop
ysn [Thu, 3 Nov 2016 11:29:56 +0000 (14:29 +0300)]
0027678: Draw User's guide contains wrong example for for-loop

Signs have been restored in draw UG as per its DOC version.

4 years ago0028062: Samples - Shading model functionality does not work in mfc Viewer3d
kgv [Tue, 8 Nov 2016 07:14:49 +0000 (10:14 +0300)]
0028062: Samples - Shading model functionality does not work in mfc Viewer3d

Unsupported shading models have been removed from the sample dialog.

4 years ago0027923: Environment variable INSTALL_DIR_LAYOUT is not documented
apn [Tue, 8 Nov 2016 14:12:48 +0000 (17:12 +0300)]
0027923: Environment variable INSTALL_DIR_LAYOUT is not documented

Added description of INSTALL_DIR_LAYOUT variable

4 years ago0027932: Improvement of standard attributes usability.
szy [Thu, 27 Oct 2016 14:55:43 +0000 (17:55 +0300)]
0027932: Improvement of standard attributes usability.

OCAF attributes TDataStd_AsciiString, TDataStd_Integer, TDataStd_Name, TDataStd_Real are extended by possibility to use custom GUID.

Now multiple attributes of any of these types can be placed at the same label using different user-defined GUIDs.
For this new "Set" methods were added into each attribute, which takes this custom GUID as an argument.
Other aspects of management of attributes on labels remain the same.

Version number of persistent OCAF documents is incremented.
However, the attributes are stored in the same way unless non-standard GUID is used for particular attribute.
Previously saved documents are fully supported, but the new documents with this extension used will be non-readable by the previous version of OCAF libraries.

4 years ago0028036: Visualization, AIS_ColoredShape - handle correctly nested compounds within...
kgv [Tue, 1 Nov 2016 18:36:24 +0000 (21:36 +0300)]
0028036: Visualization, AIS_ColoredShape - handle correctly nested compounds within Shaded display mode

AIS_ColoredShape::Compute() now parses nested compounds in two passes
to handle complex cases with compounds used for grouping styles.

4 years ago0028053: Regressions in HLR when FPE signals are enabled
msv [Mon, 7 Nov 2016 12:49:54 +0000 (15:49 +0300)]
0028053: Regressions in HLR when FPE signals are enabled

Protect Contap_Contour from creation of null-length curves.

4 years ago0028054: Regression: Class BRepProj_Projection gives invalid result after projection...
nbv [Mon, 7 Nov 2016 15:48:37 +0000 (18:48 +0300)]
0028054: Regression: Class BRepProj_Projection gives invalid result after projection edge based on the line on the conical surface

Problem of projection of line on the cone in the special case when starting point of the line coincides with the cone apex is fixed (by shifting this point along the source line).

4 years ago0028072: Samples - remove deprecated vector export from MFC sample
kgv [Tue, 8 Nov 2016 09:30:31 +0000 (12:30 +0300)]
0028072: Samples - remove deprecated vector export from MFC sample

4 years ago0028068: Samples - Add dimension functionality shows incorrect tip for an Edge in...
ski [Tue, 8 Nov 2016 14:26:23 +0000 (17:26 +0300)]
0028068: Samples - Add dimension functionality shows incorrect tip for an Edge in mfc Viewer3d

Warning message was updated to show actual information.
Additional checking of type of shape was added to prevent exceptions for Add Dimension functionality.

4 years ago0028066: Samples - mfc ocaf sample could not read/save any document without environme...
abv [Tue, 8 Nov 2016 04:22:06 +0000 (07:22 +0300)]
0028066: Samples - mfc ocaf sample could not read/save any document without environment variables

Persistence libs are loaded using DefineFormat() in OCAF sample

4 years ago0028065: [Regression to 7.0] The method HLRAppli_ReflectLines::GetResult must be...
msv [Mon, 7 Nov 2016 17:05:58 +0000 (20:05 +0300)]
0028065: [Regression to 7.0] The method HLRAppli_ReflectLines::GetResult must be const

4 years ago0028064: Samples - Raytracing does not work in Qt samples
ski [Tue, 8 Nov 2016 11:57:46 +0000 (14:57 +0300)]
0028064: Samples - Raytracing does not work in Qt samples

QAction signal activated() was changed to signal triggered() to support Qt5.

4 years ago0028060: Samples - Objects in Geometry/Viewer2d have incorrect color
kgv [Tue, 8 Nov 2016 09:30:31 +0000 (12:30 +0300)]
0028060: Samples - Objects in Geometry/Viewer2d have incorrect color

Broken custom presentations Sample2D_Text and ISession_Text
have been replaced by AIS_TextLabel.

ISession2D_Curve::Compute() - fixed defining the group without presentation aspects.
GeomSources - fixed assigning custom attributes.

4 years ago0028059: Correction of upgrade guide for 7.1
msv [Tue, 8 Nov 2016 15:43:09 +0000 (18:43 +0300)]
0028059: Correction of upgrade guide for 7.1

Correction of upgrade guide for 7.1
// minor corrections

4 years ago0028050: Empty result of offset operation in mode complete join type intersection
emv [Thu, 3 Nov 2016 09:58:25 +0000 (12:58 +0300)]
0028050: Empty result of offset operation in mode complete join type intersection

1. BRepOffset_Inter3d::ConnexIntByInt
Create unique offset edge for each localized part shared between pair of faces of the input shape.

2. Test cases for the issue.

4 years ago0026917: 3D Offset algorithm produces incorrect result
emv [Tue, 8 Nov 2016 15:04:28 +0000 (18:04 +0300)]
0026917: 3D Offset algorithm produces incorrect result

Extension on the 3D Offset algorithm (Mode="Complete", Join Type = "Intersection")
for colliding cases to add support for new configurations of the shapes.
In the previous approach the result of the offset operation was build from
the offset faces using MakerVolume algorithm, without checking of the validity of these faces.
The new extension is based on this approach, but now the offset faces are being checked
on invalidity and rebuild in case of any. This allows (in case of successful rebuilding) to avoid creation
of the unforeseen parts such as dangling parts, spikes, inverted faces in the result of offset operation.
The main criteria for the validity of the faces is the coincidence of the normal
direction of the offset face with the normal direction of the original face.
Check for removal of invalid faces has been removed as obsolete.

BRepOffset_Inter2D: Avoid excess trimming of the edges due to coincidence with other edges.
BRepOffset_Inter3D: Careful treatment of the intersection of the faces connected only through vertices.

Eliminating the compiler warning.

Small corrections of test cases for issue CR26917

4 years ago0027974: Visualization, ray tracing - Improve ray tracing engine
dbp [Thu, 20 Oct 2016 09:10:47 +0000 (12:10 +0300)]
0027974: Visualization, ray tracing - Improve ray tracing engine

* Multiple importance sampling for path tracing
* Improved light sources sampling (better handling several light sources)
* Fixed issues in light source intersection (light distance is taken into account)
* Add new TCL sample - OCCT Ball model for demonstrating physically-based materials
* Fix potential issue on NVIDIA GPUs ("Error: Failed to upload light source buffer")
* Path tracing materials reviewed; directional light source was smoother by default

4 years ago0026738: Make Boolean operations safely treating arguments when running with fuzzy...
msv [Tue, 8 Nov 2016 12:20:42 +0000 (15:20 +0300)]
0026738: Make Boolean operations safely treating arguments when running with fuzzy option

When fuzzy option is in force prevent increasing tolerance of input shapes. Instead pass increased by fuzzy value the tolerances of sub-shapes everywhere where it is needed by intersection algorithms.

The following changes in API have been made:

- The methods SetFuzzyValue and FuzzyValue have been moved from the classes BOPAlgo_ArgumentAnalyzer, BOPAlgo_Builder, and BOPAlgo_PaveFiller to the base class BOPAlgo_Algo.
- The public method BOPDS_DS::VerticesOnIn has been renamed to SubShapesOnIn, and the new output parameter theCommonPB has been added.
- In BOPTools_AlgoTools, a new argument "theFuzzyValue" has been added in the methods ComputeVV and AreFacesSameDomain.
- In IntTools_Context, a new argument "theFuzzyValue" has been added in the methods ComputeVE and ComputeVF.
- The methods SetFuzzyValue and FuzzyValue have been added in the classes IntTools_EdgeEdge, IntTools_FaceFace.
- In the class IntTools_EdgeFace, the methods SetTolE, SetTolF, TolE, TolF have been removed, and the methods SetFuzzyValue, FuzzyValue have been added.
- The new argument "theTol" has been added in the method IntTools_WLineTool::DecompositionOfWLine.

Some improvements in algorithms have been made during fighting with regressions:

- Correct initialization of pave blocks for degenerated edges.
- In BOPAlgo_PaveFiller::MakeBlocks(), filter out paves on intersection curve that were put on the curve accidentally due to wide range of E-F intersection vertex.
- In the method IntTools_Tools::ComputeTolerance the margin added to the computed tolerance has been increased up to 0.001%.
- The method BOPAlgo_PaveFiller::PutPaveOnCurve has been corrected in order to use the original vertex tolerance instead of the value increased during putting it on other curves.
- The new method BOPDS_PaveBlock::RemoveExtPave has been added.
- The vertex tolerance computation in BOPTools_AlgoTools::CorrectCurveOnSurface has been improved, taking into account intersection segments between p-curves (to avoid regression on "bugs modalg_6 bug22794").
- Improve IsExistingPaveBlock to make more stable catching of coincidence of common block with section curve (against regression "bugs modalg_4 bug697_2" on Linux).

Test case for the bug has been added.

The following test cases have been updated as improvements:
boolean gdml_private ZH2 ZI7 ZJ7
boolean volumemaker C4

The test case bugs/modalg_4/pro19653 has been corrected to make it stable. See comment inside the script for details.

The test case bugs/modalg_6/bug25880 has been corrected to suppress wrong bfuse commands.

The test bugs/modalg_6/bug26954_3 has been corrected to compare the result with more precise reference value.

The "faulty" TODO in boolean/volumemaker/A8 has been made actual for Linux as well.

//Eliminate compilation error on Linux.

4 years ago0028027: Configuration - option BUILD_RESOURCES does not work on CMake 2.8
ski [Mon, 31 Oct 2016 08:54:54 +0000 (11:54 +0300)]
0028027: Configuration - option BUILD_RESOURCES does not work on CMake 2.8

CMake command "string (APPEND ...)" was replaced by SET equivalent to support CMake 2.8

4 years ago0027802: [Regression to 7.0] Solid classifier raises exception ConstructionError
msv [Wed, 2 Nov 2016 07:59:39 +0000 (10:59 +0300)]
0027802: [Regression to 7.0] Solid classifier raises exception ConstructionError

Test case has been created.

4 years ago0024788: Foundation Classes - remove Dico_Dictionary
ski [Wed, 12 Oct 2016 15:02:55 +0000 (18:02 +0300)]
0024788: Foundation Classes - remove Dico_Dictionary

Class Dico_Dictionary was replaced by NCollection_DataMap/NCollection_IndexedDataMap classes.
Changed argument name in template class NCollection_Datamap to eliminate redefinitions of static members in dependent code.

4 years ago0028039: Coding rules, TObj - pass file name by reference
kgv [Wed, 2 Nov 2016 09:50:02 +0000 (12:50 +0300)]
0028039: Coding rules, TObj - pass file name by reference

4 years ago0028028: Solid becomes invalid after scaling
azv [Sat, 29 Oct 2016 08:23:05 +0000 (11:23 +0300)]
0028028: Solid becomes invalid after scaling

The shape after scaling contains PCurves with incorrect parametrization, because of too large tolerance given.

1. Set Precision::PConfution() as a tolerance for GeomLib::SameRange.
2. Do not take into account tolerances of vertices while calculating tolerance of transformed PCurve for edge.
3. Test case bugs/modalg_6/bug28028

4 years ago0028035: Visualization - V3d_Trihedron::compute() endlessly creates new graphic groups
kgv [Tue, 1 Nov 2016 12:14:35 +0000 (15:14 +0300)]
0028035: Visualization - V3d_Trihedron::compute() endlessly creates new graphic groups

V3d_Trihedron::compute() now reuses existing groups in the structure
and resets the flag myToCompute.

4 years ago0028030: Algorith GeomLib_CheckCurveOnSurface takes too much time for Bspline curves...
ifv [Mon, 7 Nov 2016 13:38:53 +0000 (16:38 +0300)]
0028030: Algorith GeomLib_CheckCurveOnSurface takes too much time for Bspline curves with big number of knots

Checking of number of knots is added. If number of knots more then allowed value, algorithm does not search distance between curves for each knot interval, but uses predefined sample point distribution.

This change was born in the scope of work on the issue 26329, but does not have direct connection to it. Therefore it has been postponed to be integrated later in its own context.

Test case for issue #28030

4 years ago0028034: Application Framework - stack overflow in LDOM destructor
pdn [Tue, 1 Nov 2016 10:43:56 +0000 (13:43 +0300)]
0028034: Application Framework - stack overflow in LDOM destructor

4 years ago0027896: Prm-Prm intersection algo returns wrong result if it is called with start...
nbv [Fri, 28 Oct 2016 06:37:38 +0000 (09:37 +0300)]
0027896: Prm-Prm intersection algo returns wrong result if it is called with start intersection point, which lies in the domain boundary

Purger has been disabled (in IntPatch_PrmPrmIntersection algorithm) if some points have been added in the Walking line (it is stupidly, first, to insert some points in the line and, after that, to delete points from this line).

Some improvements in IntWalk_PWalking::SeekPointOnBoundary(...) method have been made (see comments in the code for detail information).

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

4 years ago0027868: Visualization - AIS_InteractiveContext::Remove() might keep removed object...
vpa [Mon, 31 Oct 2016 13:34:15 +0000 (16:34 +0300)]
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 (14:20 +0300)]
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 (13:59 +0300)]
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 (13:49 +0300)]
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 (15:51 +0300)]
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 (17:08 +0300)]
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 (09:17 +0300)]
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 (15:03 +0300)]
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 (15:46 +0300)]
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 (18:20 +0300)]
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 (15:43 +0300)]
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 (14:29 +0300)]
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 (15:39 +0300)]
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 (15:25 +0300)]
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 (16:28 +0300)]
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 (19:31 +0300)]
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 (13:24 +0300)]
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 (18:37 +0300)]
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 (14:29 +0300)]
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 (14:28 +0300)]
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 (15:50 +0300)]
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 (17:52 +0300)]
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 (17:50 +0300)]
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 (08:46 +0300)]
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 (13:00 +0300)]
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 (14:33 +0300)]
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 (12:29 +0300)]
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 (18:04 +0300)]
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 (20:22 +0300)]
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 (11:21 +0300)]
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 (11:54 +0300)]
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 (16:39 +0300)]
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 (17:24 +0300)]
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