Archived issue #0026989
[Regression in 6.9.0] Export of a reversed face leads to wrong data in 6.9.0
Description
Apparently another adverse side effect of #25632 in addition to issue #0026419.
A reversed face during export to IGES cannot be imported back - the resulting face contains only 2 edges, is 0-area and tpstat reports issues:
Draw[164]> tps c
TransferRead :
*******************************************************************
****** Statistics on Transfer Process (Read) ******
****** Check messages ******
*******************************************************************
Count Check ** TRANSFER READ CHECK **
----- -----------
1 W:142 0 CurveOnSurface : Edges were intersecting, corrected
1 W:142 0 CurveOnSurface : Null area wire detected, wire skipped
Nb Total:2 for 2 items
The same face with Forward orientation gracefully produces correct result.
Given several regressions discovered in GeomToIGES_GeomSurface as of 6.9.0 (see also #0026931), perhaps the whole algorithm deserves careful review.
A reversed face during export to IGES cannot be imported back - the resulting face contains only 2 edges, is 0-area and tpstat reports issues:
Draw[164]> tps c
TransferRead :
*******************************************************************
****** Statistics on Transfer Process (Read) ******
****** Check messages ******
*******************************************************************
Count Check ** TRANSFER READ CHECK **
----- -----------
1 W:142 0 CurveOnSurface : Edges were intersecting, corrected
1 W:142 0 CurveOnSurface : Null area wire detected, wire skipped
Nb Total:2 for 2 items
The same face with Forward orientation gracefully produces correct result.
Given several regressions discovered in GeomToIGES_GeomSurface as of 6.9.0 (see also #0026931), perhaps the whole algorithm deserves careful review.
Steps to reproduce
restore C:/temp/f_2.brep
whatis f_2 #REVERSED face
newmodel
brepiges f_2
igesbrep . a *
numshape a
tpstat c
#wrong result
orient f_2 F
newmodel
....
#correct result
Test case: bugs iges bug26989
whatis f_2 #REVERSED face
newmodel
brepiges f_2
igesbrep . a *
numshape a
tpstat c
#wrong result
orient f_2 F
newmodel
....
#correct result
Test case: bugs iges bug26989
Public activity
5 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Commenter 3.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Tue Jan 12 18:08:26 2016 +0300
0026989: [Regression in 6.9.0] Export of a reversed face leads to wrong data in 6.9.0
Add range for degenerated edges to mirrored pcurves during exporting of reversed faces.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Tue Jan 12 18:08:26 2016 +0300
0026989: [Regression in 6.9.0] Export of a reversed face leads to wrong data in 6.9.0
Add range for degenerated edges to mirrored pcurves during exporting of reversed faces.
The problem with this face was in wrong new pcurve of degenerated edge. The range to it should be set manually after pcurve mirror in algorithm of exporting reversed face, because there is no 3D curve with range for UpdateEdge() method.
Dear GKA,
could you please review branch CR26989?
Dear GKA,
could you please review branch CR26989?
Branch CR26989 is ready to be tested
Dear Commenter 1,
Branch CR26989 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]
Number of compiler warnings:
occt component:
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MasOS: 134 (134 on master)
products component:
Linux: 37 (37 on master)
Windows: 0 (0 on master)
Regressions/Differences:
Not detected
Testing cases:
bugs iges bug26989 - OK
http://occt-tests/CR26989-master-occt-64/Debian70-64/bugs/iges/bug26989.html
http://occt-tests/CR26989-master-occt-64/Windows-64-VC10/bugs/iges/bug26989.html
Testing on Linux:
Total MEMORY difference: 90162439 / 89888003 [+0.31%]
Total CPU difference: 19106.03999999998 / 19263.980000000094 [-0.82%]
Testing on Windows:
Total MEMORY difference: 57257555 / 57526605 [-0.47%]
Total CPU difference: 18153.539968098856 / 18866.152536099093 [-3.78%]
Branch CR26989 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
[revision removed]
Number of compiler warnings:
occt component:
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MasOS: 134 (134 on master)
products component:
Linux: 37 (37 on master)
Windows: 0 (0 on master)
Regressions/Differences:
Not detected
Testing cases:
bugs iges bug26989 - OK
http://occt-tests/CR26989-master-occt-64/Debian70-64/bugs/iges/bug26989.html
http://occt-tests/CR26989-master-occt-64/Windows-64-VC10/bugs/iges/bug26989.html
Testing on Linux:
Total MEMORY difference: 90162439 / 89888003 [+0.31%]
Total CPU difference: 19106.03999999998 / 19263.980000000094 [-0.82%]
Testing on Windows:
Total MEMORY difference: 57257555 / 57526605 [-0.47%]
Total CPU difference: 18153.539968098856 / 18866.152536099093 [-3.78%]
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Related records