Data Exchange - Lost colors for IGES export

Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: ink
Date: Fri Apr 24 11:04:32 2015 +0300

    0026047: Lost colors for IGES export

Branch CR26047 is ready to test

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

Dear Commenter 2,
Branch CR26047 was rebased on current master of occt git-repository.
[revision removed]

Dear Commenter 2,
Branch CR26047 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 15 (15 on master)
Windows: 0 (0 on master)

products component :
Linux: 39 (39 on master)
Windows: 0 (0 on master)

Regressions/Differences/Improvements:
http://occt-tests/CR26047-master-products-64/Debian70-64/summary.html
http://occt-tests/CR26047-master-products-64/Windows-64-VC10/summary.html
sat doc_1 E9
sat doc_4 E7
sat doc_6 D3, H5

Testing cases:


Testing on Linux:
occt component :
Total MEMORY difference: 92039529 / 91192751 [+0.93%]
Total CPU difference: 17622.739999998987 / 17162.109999999382 [+2.68%]
products component :
Total MEMORY difference: 25141649 / 25144817 [-0.01%]
Total CPU difference: 7069.739999999988 / 7020.5499999999965 [+0.70%]

Testing on Windows:
occt component :
Total MEMORY difference: 57097354 / 57089920 [+0.01%]
Total CPU difference: 16345.098375598987 / 16439.931383498915 [-0.58%]
products component :
Total MEMORY difference: 16167781 / 16167831 [-0.00%]
Total CPU difference: 5356.341135299979 / 5316.560880299963 [+0.75%]

There are no differences in images found by testdiff.

Dear ink,
could you please write tcl-script for reproduce this bug.

Branch [archived branch] has been created by Commenter 4.

[revision removed]


Detailed log of new commits:

Author: Commenter 4
Date: Wed Oct 21 15:15:36 2015 +0300

    Add test case bugs/xde/bug26047

Author: ink
Date: Fri Apr 24 11:04:32 2015 +0300

    0026047: Lost colors for IGES export

Test case has been added to the patch in branch CR26047_1.

Dear gka,

could you please review test case and confirm tests reported as regression due to deviation from reference data?

Branch [archived branch] has been deleted by Commenter 4.

[revision removed]

Branch [archived branch] has been created by Participant.

[revision removed]


Detailed log of new commits:

Author: ink
Date: Fri Apr 24 11:04:32 2015 +0300

    0026047: Lost colors for IGES export
    
    Method XSAlgo_AlgoContainer::MergeTransferInfo() is corrected to keep locations of sub-shapes when recording transfer results in the map.
    
    Add test case bugs/xde/bug26047

The fix leads to loss of some colors in ACIS tests:
sat doc_1 E9
sat doc_4 E7
sat doc_6 D3 H5

Branch [archived branch] has been created by Commenter 1.

[revision removed]


Detailed log of new commits:

Author: Commenter 1
Date: Wed Nov 11 14:23:30 2020 +0300

    0026047: Data Exchange - Lost colors for IGES export
    
    - Fix problem with write shape with locations
    - Add support for adding sub-shapes (labels) together with sub-shapes without location (only for IGES)

Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]

Dear ika, please review CR26047_3
The bug was incorrect writing and reading of the shapes with locations.
- I update ShapeTools for adding subshape reference by no-locations(new class field, that enable before IGES import and disabled after it)
- Add support of find shape, that was edit(remove location), while export an IGES file

ALL tests are OK:
http://occt-tests/CR26047_3-master-Commenter 1-OCCT/Debian80-64/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-OCCT/Windows-64-VC14/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-Products/Debian80-64/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-Products/Windows-64-VC14/diff_summary.html

Could you please remove new redundant flags from XCAFDoc_ShapeTool and add comments

Branch [archived branch] has been created by Commenter 1.

[revision removed]


Detailed log of new commits:

Author: Commenter 1
Date: Wed Nov 11 14:23:30 2020 +0300

    0026047: Data Exchange - Lost colors for IGES export
    
    - Fix problem with write shape with locations
    - Add support for adding sub-shapes (labels) together with sub-shapes without location

Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]

Dear gka, please review CR26047_4
The bug was incorrect writing and reading of the shapes with locations.
- I update ShapeTools for adding subshape reference by no-locations
- Add support of find shape, that was edit(remove location), while export an IGES file

All tests are OK:
http://occt-tests/CR26047_3-master-Commenter 1-OCCT/Debian80-64/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-OCCT/Windows-64-VC14/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-Products/Debian80-64/diff_summary.html
http://occt-tests/CR26047_3-master-Commenter 1-Products/Windows-64-VC14/diff_summary.html

Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]

Branch CR26047_4 was reviewed

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: 18049.94000000008 / 18024.44000000013 [+0.14%]
Products
Total CPU difference: 12325.960000000132 / 12167.95000000011 [+1.30%]
Windows-64-VC14:
OCCT
Total CPU difference: 19703.46875 / 19697.21875 [+0.03%]
Products
Total CPU difference: 13792.6875 / 13647.203125 [+1.07%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

Branch [archived branch] has been deleted by Participant.

[revision removed]

Branch [archived branch] has been deleted by Participant.

[revision removed]

Branch [archived branch] has been deleted by Participant.

[revision removed]

Branch [archived branch] has been deleted by Participant.

[revision removed]