OCCT icon indicating copy to clipboard operation
OCCT copied to clipboard

Published 20 hours ago • verified publisher icon Open-Cascade-SAS

Revolved shape in STEP file is imported inverted

Open elias-bananaz opened this issue 9 months ago • 7 comments

Description

The attached file is imported incorrectly. It appears as if the revolved shape is imported with the complementary angles instead.

Expected Behavior

Shape should look like this:

Image

Actual Behavior

It looks like this instead:

Image

Sample Code or DRAW Tcl Script

pload XDE OCAF VISUALIZATION MODELING

# read STEP file into XDE document using STEPCAFControl_Reader
jsupload /models/occ/as1-oc-214.stp
ReadStep D as1-oc-214.stp

# display the document in 3D viewer using XCAFPrs_AISObject
vinit View1
XDisplay -dispMode 1 D -explore
vfit

(sample from https://draw.sview.ru/xde-import-step/)

Operating System

Windows

Compiler

MSVC

Bitness

64-bit

OCCT Version

latest

Additional Files

STEP file to reproduce: b15.zip

elias-bananaz avatar Feb 17 '25 15:02 elias-bananaz

7.6.3 it was correct. 7.7.0 - already broken. Regression. But unfortunately, can't be done for 7.9.0. Will be planned for the next release

dpasukhi avatar Feb 17 '25 15:02 dpasukhi

Thanks. Can it be fixed at least in the development branch?

elias-bananaz avatar Feb 17 '25 16:02 elias-bananaz

Yes, if the fix will be ready, it will be available in dev

dpasukhi avatar Feb 17 '25 16:02 dpasukhi

I have another file that has a problem with a revolved surface. Not sure if it's the same problem, but I'm sharing here nonetheless.

Blades lock.zip

You can see a sort of "blemish" at the center of the shape, when the surface is triangulated. Image

elias-bananaz avatar Mar 03 '25 14:03 elias-bananaz

The regression commit: b0f92d3d5c78548a1e5d14f7b7c28463a3981092

dpasukhi avatar Mar 04 '25 21:03 dpasukhi

I saw the fix in the draft PR, but you wrote that the regression is deeper. Would taking this commit be enough for fixing the case I posted? Or would I need to wait for a different PR?

elias-bananaz avatar Mar 30 '25 09:03 elias-bananaz

@elias-bananaz the fix is planned to be done. But case is fixed. Need more changes for general case

dpasukhi avatar Mar 30 '25 10:03 dpasukhi

Original issue resolved. In case of the keeping the problem, reopen the issue or create a new one.

dpasukhi avatar Sep 04 '25 16:09 dpasukhi

The above fix worked well for the torus example, but I just tried testing the above PR with more complex geometry and found certain torodial surfaces continue to have inverted faces.

wrench.stp.zip

Image

mf-hpollock avatar Sep 04 '25 16:09 mf-hpollock

Please report it separately. It is new dateset. Please create a new bug report

dpasukhi avatar Sep 04 '25 16:09 dpasukhi

Please report it separately. It is new dateset. Please create a new bug report

Will do, thank you!

mf-hpollock avatar Sep 04 '25 17:09 mf-hpollock

Please report it separately. It is new dateset. Please create a new bug report

Will do, thank you!

No needs for any info to reproduce,you can create a bug report with empty details and these dataset

dpasukhi avatar Sep 06 '25 08:09 dpasukhi