Archived issue #0025167
STEP Reader - allow NextAssemblyUsageOccurance to accept ProductDefinitionShape as child instead of ProductDefinition
Description
The STEP files exported by PRO/ENGINEER can contain NextAssemblyUsageOccurance entities, which have ProductDefinitionShape entities as children, while the standard prescribes ProductDefinition.
But ProductDefinitionShape is always coupled with ProductDefinition instance, therefore a workaround can be provided
But ProductDefinitionShape is always coupled with ProductDefinition instance, therefore a workaround can be provided
Steps to reproduce
Unfortunately, the STEP file is classified, I can only place a small snippet of it:
#1858356=SHAPE_DEFINITION_REPRESENTATION(#1858357,#1858358);
#1858357=PRODUCT_DEFINITION_SHAPE('','SHAPE FOR',#1858426);
#1858358=SHAPE_REPRESENTATION('',(),#1858417);
#1858359=NEXT_ASSEMBLY_USAGE_OCCURRENCE('526','Next assembly relationship',
'SOMETHING',#1858357,#1858332,$);
...
#1858426=PRODUCT_DEFINITION('part definition','',#1858425,#1858327);
#1858356=SHAPE_DEFINITION_REPRESENTATION(#1858357,#1858358);
#1858357=PRODUCT_DEFINITION_SHAPE('','SHAPE FOR',#1858426);
#1858358=SHAPE_REPRESENTATION('',(),#1858417);
#1858359=NEXT_ASSEMBLY_USAGE_OCCURRENCE('526','Next assembly relationship',
'SOMETHING',#1858357,#1858332,$);
...
#1858426=PRODUCT_DEFINITION('part definition','',#1858425,#1858327);
Public activity
5 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
This branch includes the following new commits:
new 0a50d79 0025167: Workaround to allow NextAssemblyUsageOccurance accepting ProductDefinitionShape as child instead of ProductDefinition
Detailed log of new commits:
[revision removed]
Author: razmyslovich
Date: Tue Aug 19 13:49:37 2014 +0200
0025167: Workaround to allow NextAssemblyUsageOccurance accepting ProductDefinitionShape as child instead of ProductDefinition
[revision removed]
This branch includes the following new commits:
new 0a50d79 0025167: Workaround to allow NextAssemblyUsageOccurance accepting ProductDefinitionShape as child instead of ProductDefinition
Detailed log of new commits:
[revision removed]
Author: razmyslovich
Date: Tue Aug 19 13:49:37 2014 +0200
0025167: Workaround to allow NextAssemblyUsageOccurance accepting ProductDefinitionShape as child instead of ProductDefinition
The workaround is committed, please, review it.
Branch CR21567 is ready to be tested
Dear Commenter 1,
Branch CR25167 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]
Number of compiler warnings:
occt component:
Linux: 15 (15 on master)
Windows: 0 (0 on master)
products component:
Linux: 11 (11 on master)
Windows: 1 (1 on master)
Regressions/Differences:
Not detected
Testing case:
bugs step bug25167 - OK
http://occt-tests/CR25167-master-occt/Debian60-64/bugs/step/bug25167.html
http://occt-tests/CR25167-master-occt/Windows-32-VC10/bugs/step/bug25167.html
Testing on Linux:
Total MEMORY difference: 352295748 / 351913792
Total CPU difference: 44175.61999999997 / 45954.130000000136
Testing on Windows:
Total MEMORY difference: 239643372 / 239473840
Total CPU difference: 40999.234375 / 32460.796875
Branch CR25167 (and products from GIT master) was compiled on Linux and Windows platforms and tested.
[revision removed]
Number of compiler warnings:
occt component:
Linux: 15 (15 on master)
Windows: 0 (0 on master)
products component:
Linux: 11 (11 on master)
Windows: 1 (1 on master)
Regressions/Differences:
Not detected
Testing case:
bugs step bug25167 - OK
http://occt-tests/CR25167-master-occt/Debian60-64/bugs/step/bug25167.html
http://occt-tests/CR25167-master-occt/Windows-32-VC10/bugs/step/bug25167.html
Testing on Linux:
Total MEMORY difference: 352295748 / 351913792
Total CPU difference: 44175.61999999997 / 45954.130000000136
Testing on Windows:
Total MEMORY difference: 239643372 / 239473840
Total CPU difference: 40999.234375 / 32460.796875
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Related records