STeP Importer stuck and hogging memory

Hello, your problem similar with 0032127: Data Exchange - faceted STEP file will not open - MantisBT (opencascade.org)

I see that you have a lot of shells within single solid. That is not expected input for current algo.

The fast workaround is disable that type of healing - if you are using XSTEP shape healing file, please turn off next setting (FromSTEP.FixShape.FixShellOrientationMode). Make it equal with "0"

Usual path: "occt\occt\install\src\XSTEPResource\STEP"

If you don't use a file or not delivery, please update that parameter using the next code.

STEPControl_Controller::Init();
Interface_Static::SetIVal ("FromSTEP.FixShape.FixShellOrientationMode", 0);

Please check with JeMalloc memory manager. It can not help, but at least can be tested. If you can share your file, it will be helpful for us to reproduce especially on your problem case.

Best regards, Commenter-1.

Thanks for answering.

I fail to see the similarity with the bug you linked: the problem there, IIUC, is there's no structure in the file, but only a lot of free faces; here there are breps. In any case, I'll trust you and keep an eye on that issue.

I don't even know what "XSTEP shape healing file" is (sorry, I feel documentation is the worst part of OC :).
I'm using the following code:
Handle(XCAFApp_Application) App{XCAFApp_Application::GetApplication()};
Handle(TDocStd_Document) Doc;
App->NewDocument("BinXCAF",Doc);
>>> STEPControl_Controller::Init();
>>> Interface_Static::SetIVal("FromSTEP.FixShape.FixShellOrientationMode",0);
STEPCAFControl_Reader;
if (reader.ReadFile(FileName)!=IFSelect_RetDone) throw std::runtime_error{"OpenCASCADE STEP reading failed"};
if (!reader.Transfer(Doc)) throw std::runtime_error{"OpenCASCADE transfer failed"};

As you can see, I added the code you suggested (>>>), but nothing changed.
Perhaps I've done it wrong?

I've tried this on FreeBSD, which, AFAICT uses jemalloc by default.
In cmake config I have USE_MMGR_TYPE=NATIVE.
Do you still suggest I try USE_MMGR_TYPE=JEMALLOC?

I wish I could share this file. Alas, as I said previously, I don't have clearance. It comes from a big well-known company who doesn't like to share their "secrets".

bye & Thanks
av.

Yes, needs to update the CMake variable to use another manager.

As for a parameter. This parameter will have no affect if resource is exist on the your env. please copy a resource file from "occt\occt\install\src\XSTEPResource\STEP" to some location.

And specify the next env variable before calling your app or inside your app. "CSF_STEPDefaults" to the folder which contains "STEP" file (no extension).

And please update the parameter that i describe before.

_

Yes, documentation for OCCT is not well structured. We want to improve userability, but it is difficult for application from 1990x :) With a gigantic size of functionality.

Soon we simplify DE processes to have any needed API in one place.

Best regards, Commenter-1.

Thanks.
Using the file and the environment variable it works perfecly.
However, this is a bit of a hassle (we'd need to modify installers, etc...), so if I could achieve this through code, it would be a lot simpler.
Consider:
_ I don't have the "STEP" file anywhere (except in OC sources);
_ I don't have the "CSF_STEPDefaults" environment variable;
_ IIUC you're saying in such a situation the code I posted should work? Or, otherwise, how should it be modified?

Thanks in advance.

bye
av.

Hello, it is for sure possible from your source code update.

You need to make specific OCCT's static variable as empty string. I do not remember which one. I will return to you when found.

Anyway I guess it will be better to improve out documentation in the same time :)

Best regards, Commenter-1.

STEPControl_Controller::Init();
Interface_Static::SetIVal ("FromSTEP.FixShape.FixShellOrientationMode", 0);
Interface_Static::SetCVal ("read.step.resource.name", "");

That should work :0

Best regards, Commenter-1.

Works perfectly.
Thanks a lot.

Hello.
Sorry to revive an old thread.
I'v updated to 7.9.0 and the problem came back: setting the above parameter now doesn't prevent OpenCASCADE from allocating several GBs of RAM and consequently getting killed.
Has anyhing changed in this area?
Are there new settings that I should consider?
Thanks.

Hello. The ShapeHealing parameters handling was changed. Please check https://dev.opencascade.org/doc/overview/html/occt__upgrade.html#upgrade...

For now you need to apply parameters as in sample or use new method to apply structure with parameters.

Best regards, Commenter-1.

Hello and thanks for answering.

I modified my code to be:

STEPCAFControl_Reader reader;
DE_ShapeFixParameters FixParameters; FixParameters.FixShellOrientationMode=DE_ShapeFixParameters::FixMode::NotFix;
reader.SetShapeFixParameters(FixParameters);
reader.ReadFile(filename);
reader.Transfer(Doc);

However OC still hangs hogging memory until it's killed, so I guess FixShellOrientationMode is not taken into account.

I tried changing it like:

STEPCAFControl_Reader reader;
reader.ReadFile(filename);
DE_ShapeFixParameters FixParameters; FixParameters.FixShellOrientationMode=DE_ShapeFixParameters::FixMode::NotFix;
reader.SetShapeFixParameters(FixParameters);
reader.Transfer(Doc);

but then nothing is imported.

I guess I'm doing it wrong, but I find it very hard to use the documentation.
Sorry.
Any further hint?

Thanks and Best regards.

All of the current option must to work. I check both of them locally and all of them are working fine (the parameters are going inside correctly)

In case of STEP you need to use as original - default one:

DE_ShapeFixParameters aParameters = DESTEP_Parameters::GetDefaultShapeFixParameters();

Could you please check what happened on these line? 
https://github.com/dpasukhi/OCCT/blob/38cd8081f17219cd1064deedf6f07173b0...

Best regards, Commenter-1.

New bug: STEP file with a lot of faces are stack during fixing · Issue #417 · Open-Cascade-SAS/OCCT

Additionally, if still not working with your code, please let me know - i will check one more time shape healing propagation.

Hello.

Calling reader.SetShapeFixParameters before reader.ReadFile(filename) does not work.
However it works if I call reader.SetShapeFixParameters between reader.ReadFile(filename) and reader.Transfer.

(The fact that nothing was imported was due to another bug, i.e. using "-Wl,-Bsymbolic" with LLVM completely breaks RTTI).

Thanks.

Oh, unexpected. I will validate.

As for setting up the parameters, thank you. Will be validated. For some reason, OCCT is tested on Clang and didn't face issues. Thank you for the research.

I will revert this linker flag. If issue is critical, then need to check options with hot path.

Could you please share how it was detected?

Best regards, Commenter-1.

PR with removing symbolic: Configuration - Remove -symbolic linker flag

Original issue was the try to link "static" and "dynamic" libs with fpic and without. Original fix were incorrect.

Best regards, Dmitri.