DiscussionsIssue archiveOCCT:Data Exchange

Archived issue #0030402

Data Exchange - exported VRML file has broken location information

Open CASCADEOCCT:Data Exchangeclosed20 public notes

Search issues

Description

Exporting into VRML format produces broken result in case if original model contains (nested) transformations and system units are different from VRML format (meters).

Steps to reproduce

pload ALL
param xstep.cascade.unit MM
ReadStep D Attachment 1 (STP)
XGetOneShape a D
incmesh a 0.001
WriteVrml D tmp.wrl
loadvrml b tmp.wrl
vinit
vdisplay b
vfit

Public activity

20 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: Mon Dec 10 14:08:09 2018 +0300

    0030402: exported VRML file has broken location information
02Commenter 2
03Commenter 3
Sergey,

there is no test case in branch (which you have talked about).
Are you sure you have pushed the latest version to git?
04Commenter 2
Branch [archived branch] has been updated by Commenter 1.

[revision removed]


Detailed log of new commits:

Author: Commenter 1
Date: Tue Dec 11 12:25:12 2018 +0300

    //Test case bugs/xde/bug30402 were added

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

[revision removed]
06Commenter 1
Combination -
OCCT branch : [archived branch] SHA - [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

Test case:
bugs xde bug30402 - OK

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16379.8 / 16366.920000000002 [+0.08%]
Products
Total CPU difference: 7077.670000000036 / 7064.160000000029 [+0.19%]
Windows-64-VC14:
OCCT
Total CPU difference: 17769.265625 / 17885.453125 [-0.65%]
Products
Total CPU difference: 8459.96875 / 8548.140625 [-1.03%]

Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
07Commenter 3
+set filename bug26657
+param xstep.cascade.unit MM
+ReadStep D [locate_data_file ${filename}.stp]
+XGetOneShape a D
+incmesh a 0.001
+
+file delete -force ${imagedir}/${filename}.wrl
+WriteVrml D ${imagedir}/${filename}.wrl
+loadvrml b ${imagedir}/${filename}.wrl
+
+checkview -display b -3d -path ${imagedir}/${test_image}.png

Test case is incomplete - it doesn't check transformations in the document, hence even if image is correct, the result might be actually wrong.

And for image comparison, test case might try dumping original and generated VRML files to check if they produce the same image.
08Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
09Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
10Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
11Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
12Commenter 1
Images for original shape and generated VRML file were added:
http://occt-tests/CR30402-master-SKL-OCCT/Debian80-64/bugs/xde/bug30402.html

Dear skl, could you please provide additional check transformations in the document.
13Commenter 3
+set filename bug26657
+ReadStep D [locate_data_file ${filename}.stp]

I'm not sure it is a good idea splitting the file name from extension - this would prevent a straightforward seeking in test base usage of "bug26657.stp".
14Commenter 2
Branch [archived branch] has been updated by Commenter 1.

[revision removed]


Detailed log of new commits:

Author: Commenter 1
Date: Wed Dec 12 11:28:17 2018 +0300

    //Correct test case according to remarks

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

[revision removed]


Detailed log of new commits:

Author: Commenter 1
Date: Wed Dec 12 12:14:58 2018 +0300

    //Modify loadvrml command
    //Update test case

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

[revision removed]
17Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
18Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
19Commenter 2
Branch [archived branch] has been updated forcibly by Commenter 1.

[revision removed]
20Commenter 2
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]