DiscussionsIssue archiveOCCT:Application Framework

Archived issue #0031350

Application Framework - Dump improvement for OCAF classes

Open CASCADEOCCT:Application Frameworkclosed8 public notes

Search issues

Description

- Move dump of TDF_Attribute ID into base class;
- extend TDF_Attribute, TDocStd_Document with parameters;
- improve dump of some XCAFDoc/XCAFDimTolObjects instances.

Public activity

8 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Wed Dec 16 12:08:49 2020 +0300

    0031350: Inspectors - property view in DFBrowser for DumpJson
    
    - do not dump document in CDM_MetaData, just pointer text;
    - move ID dump in to TDF_Attribute;
    - extend TDF_Attribute with other important states(obtained by methods);
    - extend TDocStd_Document with other important states;
    - avoid dumping of shapes in TNaming_UsedShapes (it takes much time);
    - extend XCAFDoc, XCAFDimTolObjects dump to json;
    - correct XCAFDoc_ColorTool, XCAFDoc_MaterialTool, XCAFDoc_LayerTool to dump only connected to the object information;
    - correct XCAFDoc_Material to dump name and density value only if it exists;
    - add check on correct direction in gp_Ax3.
02Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
03Author
Dear Kirill,
could you please, review the patch.
Jenkins job is: http://vm-jenkins-test-12.nnov.opencascade.com:8080/view/CR31350-master-nds/

Thank you, Natalia
04Commenter 3
+  if (!Direction().IsEqual (aDir, Precision::Confusion()))

Shouldn't it be Precision::Angular() here?

+  .
...
+//purpose  : .

It would be nice avoiding redundant trailing spaces in the patch.
05Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Author
Corrected, please check.
07Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
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: 17709.570000000054 / 17744.450000000124 [-0.20%]
Products
Total CPU difference: 12258.480000000121 / 12330.210000000125 [-0.58%]
Windows-64-VC14:
OCCT
Total CPU difference: 19272.796875 / 19429.1875 [-0.80%]
Products
Total CPU difference: 13712.859375 / 13755.5 [-0.31%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
08Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records