View Issue Details

IDProjectCategoryView StatusLast Update
0026925Open CASCADEOCCT:Data Exchangepublic2020-12-02 17:11
ReportergkaAssigned Todpasukhi  
PrioritynormalSeveritymajor 
Status closedResolutionfixed 
Product Version6.9.1 
Target Version7.5.0Fixed in Version7.5.0 
Summary0026925: Data Exchange - Exception can be raised if assembly is empty
DescriptionException can be raised if assembly is empty. See comment to issue 0026314:
This fix broke the following logic that used to work in pre-7.0.

"In pre-7.0 one could create a structure of labels using plain OCAF/XDE API:
-assembly
 -instance
  -part (with shape)
and call ShapeTool::UpdateAssembly (asm_label). The latter would create a valid shape to be attached to asm_label.

Current implementation throws an exception:
  TopoDS_Shape aShape = GetShape(L); //<-- returns null
  Standard_Boolean isFree = aShape.Free(); //exception is thrown

The implementation assumes that the asm_label always contains some non-null shape."
Steps To ReproduceBugs is not reproduced in current OCCT version
TagsNo tags attached.
Test case numberbugs/step/bug26925

Attached Files

  • 26925.tcl (208 bytes)

Relationships

child of 0026314 closedgka Method XCAFDoc_ShapeTool::SetShape() works not correctly. 

Activities

gka

2019-09-12 17:12

developer   ~0087026

Last edited: 2019-09-12 17:13

In the current development version this scenario works correctly.
Method XCAFDoc_ShapeTool::UpdateAssembly to increase performance was replaced on the method XCAFDoc_ShapeTool::UpdateAssemblies which updated whole document.
At current OCCT version described scenario can be reproduced in Test Harness with following sequence of commands shown in log hereafter (see attached script 26925.tcl)


Draw[3]> XNewDoc D1
Draw[4]> XNewShape D1
0:1:1:1
Draw[7]> DFB D1
browser_D1
Draw[8]> box a 0 0 0 10 10 10
Draw[9]> XAddShape D1 a 0
0:1:1:2
Draw[10]> ttranslate a 10 0 0
Draw[11]> XAddComponent D1 0:1:1:1 a
0:1:1:1
Draw[11]> XGetShape s0 D1 0:1:1:1
Draw[12]> dump s0


*********** Dump of s0 *************
Shape : 1, FORWARD

Dump of 1 TShapes

-----------------

Flags : Free, Modified, Checked, Orientable, Closed, Infinite, Convex, Locked

TShape # 1 : COMPOUND 11000000 000001D69C4A9550



 -------
Dump of 0 Curve2ds
 -------


 -------
Dump of 0 Curves
 -------

 -------
Dump of 0 Polygon3Ds
 -------
 -------
Dump of 0 PolygonOnTriangulations
 -------

 -------
Dump of 0 surfaces
 -------

 -------
Dump of 0 Triangulations
 -------



 -------
 Dump of 0 Locations
 -------



Draw[20]>Draw[21]> he XUpd*
XUpdateAssemblies : Doc : updates assembly compounds
Draw[21]> XUpdateAssemblies D1
Draw[22]> XGetShape s1 D1 0:1:1:1
Draw[23]> nbs s1
Number of shapes in s1
 VERTEX : 8
 EDGE : 12
 WIRE : 6
 FACE : 6
 SHELL : 1
 SOLID : 1
 COMPSOLID : 0
 COMPOUND : 1
 SHAPE : 35

gka

2019-09-12 17:13

developer  

26925.tcl (208 bytes)

abv

2019-09-14 08:25

manager   ~0087080

Proposed DRAW commands do not allow reproducing the situation described in the issue since command XNewShape creates empty compound, i.e. the shape on the assembly root label always exists. By looking in the code I suspect that if the assembly label has no shape attached, it will not be updated with UpdateAssemblies() (i.e. no exception but also no action). To be checked

git

2020-05-12 19:50

administrator   ~0092080

Branch CR26925 has been created by dpasukhi.

SHA-1: 72e6020d50e6d78474fbd4dfb420db28ff92caa0


Detailed log of new commits:

Author: dpasukhi
Date: Tue May 12 19:51:12 2020 +0300

    0026925: Data Exchange - Exception can be raised if assembly is empty
    
    - Issue is done in previous version OCCT
    - Add test to verify the creating of the label structure using plain OCAF/XDE API
    - Situation when the assembly label has no shape attached is correct

ika

2020-05-13 13:45

developer   ~0092086

Only one test case was added, it passed successfully:
http://occt-tests/CR26925-master-dpasukhi-OCCT/Windows-64-VC14/bugs/step/bug26925.html
http://occt-tests/CR26925-master-dpasukhi-OCCT/Debian80-64/bugs/step/bug26925.html

No remarks.

ika

2020-05-13 13:46

developer   ~0092087

Dear bugmaster,

please integrate:
OCCT - CR26925
PRODUCTS - NOT

bugmaster

2020-05-15 11:06

administrator   ~0092129

Combination -
OCCT branch : WEEK-20
master SHA - d9eb6aca8789e6b8ff6742e3ad60a4dadf0cd37e
a206de37fbfa0bf71bd534ae47192bbec23b8522
Products branch : WEEK-20 SHA - eb85b0feb479f67f3a505bcb285922a288149f74
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 17146.6200000001 / 17177.020000000186 [-0.18%]
Products
Total CPU difference: 11178.21000000009 / 11272.470000000078 [-0.84%]
Windows-64-VC14:
OCCT
Total CPU difference: 18609.671875 / 18646.015625 [-0.19%]
Products
Total CPU difference: 13007.71875 / 13180.21875 [-1.31%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

git

2020-05-16 14:21

administrator   ~0092184

Branch CR26925 has been deleted by inv.

SHA-1: 72e6020d50e6d78474fbd4dfb420db28ff92caa0

Related Changesets

occt: master 72e6020d

2020-05-12 16:51:12

dpasukhi

Details Diff
0026925: Data Exchange - Exception can be raised if assembly is empty

- Issue is done in previous version OCCT
- Add test to verify the creating of the label structure using plain OCAF/XDE API
- Situation when the assembly label has no shape attached is correct
Affected Issues
0026925
add - tests/bugs/step/bug26925 Diff File

Issue History

Date Modified Username Field Change
2015-11-25 11:04 gka New Issue
2015-11-25 11:04 gka Assigned To => gka
2015-11-25 11:05 gka Relationship added child of 0026314
2016-02-17 10:56 abv Target Version 7.0.0 => 7.1.0
2016-10-26 11:39 gka Target Version 7.1.0 => 7.2.0
2017-07-27 09:24 abv Target Version 7.2.0 => 7.4.0
2019-09-04 17:41 kgv Summary Exception can be raised if assembly is empty => Data Exchange - Exception can be raised if assembly is empty
2019-09-12 12:43 gka Status new => assigned
2019-09-12 17:12 gka Note Added: 0087026
2019-09-12 17:13 gka File Added: 26925.tcl
2019-09-12 17:13 gka Note Edited: 0087026
2019-09-12 17:14 gka Assigned To gka => abv
2019-09-12 17:14 gka Status assigned => resolved
2019-09-12 17:14 gka Steps to Reproduce Updated
2019-09-14 08:25 abv Note Added: 0087080
2019-09-14 08:25 abv Assigned To abv => gka
2019-09-14 08:25 abv Status resolved => assigned
2019-09-14 08:25 abv Target Version 7.4.0 => 7.5.0
2020-04-15 21:27 gka Assigned To gka => dpasukhi
2020-05-12 19:50 git Note Added: 0092080
2020-05-12 19:53 dpasukhi Status assigned => resolved
2020-05-13 13:45 ika Note Added: 0092086
2020-05-13 13:46 ika Note Added: 0092087
2020-05-13 13:46 ika Assigned To dpasukhi => bugmaster
2020-05-13 13:46 ika Status resolved => reviewed
2020-05-15 11:06 bugmaster Note Added: 0092129
2020-05-15 11:06 bugmaster Status reviewed => tested
2020-05-15 11:07 bugmaster Test case number => bugs/step/bug26925
2020-05-16 14:07 dpasukhi Changeset attached => occt master 72e6020d
2020-05-16 14:07 dpasukhi Assigned To bugmaster => dpasukhi
2020-05-16 14:07 dpasukhi Status tested => verified
2020-05-16 14:07 dpasukhi Resolution open => fixed
2020-05-16 14:21 git Note Added: 0092184
2020-12-02 16:40 emo Fixed in Version => 7.5.0
2020-12-02 17:11 emo Status verified => closed