View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0023203||Community||OCCT:Data Exchange||public||2012-06-14 22:09||2013-04-29 15:21|
|Reporter||Roman Lygin||Assigned To||bugmaster|
|Target Version||6.6.0||Fixed in Version||6.6.0|
|Summary||0023203: STEP import produces wrong surface of revolution on ellipse|
|Description||Regression caused by #952|
STEP importer produces wrong surface of revolution when axis of revolution goes through the ellipse location and lies in its plane.
The root-cause is fix #952 in StepToGeom_MakeSurfaceOfRevolution::Convert() which produces a regression on ellipses.
Apparently the fix 952 was wrong:
- it changes the ellipse axes
- even on circles (when geometrical form is preserved) it may change original surface parametrization and hence pcurve in the file will be imported wrong.
With that, the fix #952 needs to be revisited and at best removed. It should be Shape Healing's job, not particular format importer's, to handle such case. Perhaps this is not needed at all - perhaps OCC can handle such revolved surfaces well (note that face f from reproducer is valid and displayed well, and checkshape claims valid).
P.S. Public access to #952 is denied, so it is impossible to verify original model that required that fix.
|Steps To Reproduce||ellipse e 0 0 0 0 -1 0 1 0 -1 10 5|
revsurf re e 0 0 0 0 0 1
mkface f re 0 3 0 1
stepwrite 0 f f.stp
stepread . res *
explode res_1 f
mksurf ress1 res_1_1
# above is synthetic reproducer. attached test.brep is original one.
|Tags||No tags attached.|
|Test case number||bugs step(008) bug23203|
test.zip (1,206 bytes)
||Reassigned to skl, the author of #952|
The branch CR23203 created and pushed into the repository. Please review.
(Please note that the fix has not been compiled or tested explicitly - only manually verified in debugger, by skipping the code of fix952)
||I believe that fix for bug 952 is useful for circles and cases for ellipses which axis coincided with axis of rotation. Therefore I think that existed fix for bug 952 needs to change but not remove.|
Checking of ellipse axis was added.
Git branch CR23203_1 is ready to be reviewed.
bug23203 (1,647 bytes)
Branch CR23203_1 (and products from GIT master) was compiled on Linux and
Windows platforms and tested.
bugs step(008) bug23203 - OK.
occt: master 41b3001c
2012-12-21 12:01:04Details Diff
|0023203: STEP import produces wrong surface of revolution on ellipse
Checking of XAsis for ellipses was added.
Adding test case for this fix
|mod - src/StepToGeom/StepToGeom_MakeSurfaceOfRevolution.cxx||Diff File|
|add - tests/bugs/step/bug23203||Diff File|
|2012-06-14 22:09||Roman Lygin||New Issue|
|2012-06-14 22:09||Roman Lygin||Assigned To||=> gka|
|2012-06-14 22:09||Roman Lygin||File Added: test.zip|
|2012-06-14 22:11||Roman Lygin||Assigned To||gka => skl|
|2012-06-14 22:12||Roman Lygin||Note Added: 0020698|
|2012-06-14 22:26||Roman Lygin||Note Added: 0020699|
|2012-06-14 22:26||Roman Lygin||Status||new => resolved|
|2012-06-22 10:04||skl||Note Added: 0020759|
|2012-06-22 10:05||skl||Assigned To||skl => dbv|
|2012-06-28 17:25||abv||Status||resolved => assigned|
|2012-10-24 17:55||abv||Assigned To||dbv => ika|
|2012-10-24 17:55||abv||Target Version||=> 6.6.0|
||Note Added: 0022682|
||Assigned To||ika => skl|
||Status||assigned => resolved|
|2012-12-17 12:10||skl||Note Added: 0022717|
|2012-12-17 12:11||skl||Status||resolved => reviewed|
|2012-12-17 12:12||skl||Assigned To||skl => mkv|
||File Added: bug23203|
||Note Added: 0022773|
||Test case number||=> bugs step(008) bug23203|
||Assigned To||mkv => bugmaster|
||Status||reviewed => tested|
|2012-12-24 17:23||bugmaster||Changeset attached||=> occt master 41b3001c|
|2012-12-24 17:23||bugmaster||Status||tested => verified|
|2012-12-24 17:23||bugmaster||Resolution||open => fixed|
||Status||verified => closed|
||Fixed in Version||=> 6.6.0|