Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IGES support #184

Open
fidoriel opened this issue Nov 16, 2024 · 4 comments · May be fixed by #186
Open

IGES support #184

fidoriel opened this issue Nov 16, 2024 · 4 comments · May be fixed by #186

Comments

@fidoriel
Copy link
Contributor

It would be very nice, if the OCCT iges importer would be supported within the high opencascade api abstraction like step is.

https://dev.opencascade.org/doc/overview/html/occt_user_guides__iges.html

@bschwind
Copy link
Owner

Yes, this can be added back in now that we have more upload space on crates.io for occt-sys

@fidoriel
Copy link
Contributor Author

Oh, it was in there but has been removed, so everything is already done? Great.

@bschwind
Copy link
Owner

Oh, it was in there but has been removed, so everything is already done?

Not quite - The C++ code was excluded via a Cargo.toml exclude path to get the crate size down, but it should now be included.

So the bindings were never there, but it should be straightforward to follow the bindings for STEP and STL import/export in order to add support for IGES.

@fidoriel fidoriel linked a pull request Nov 20, 2024 that will close this issue
@fidoriel
Copy link
Contributor Author

fidoriel commented Nov 20, 2024

Oh, it was in there but has been removed, so everything is already done?

Not quite - The C++ code was excluded via a Cargo.toml exclude path to get the crate size down, but it should now be included.

So the bindings were never there, but it should be straightforward to follow the bindings for STEP and STL import/export in order to add support for IGES.

Bindings were not difficult. Difficult was to understand which functions to call in the c++ classes. And to figure out why the linker was not working. Spoiler: IGES was not compiled ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants