-
Notifications
You must be signed in to change notification settings - Fork 6
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
Cannot convert .stp file to .h5m #88
Comments
Thanks for letting me know, I shall keep an eye out for similar behaviour |
So, I think I made it work for my case, although I will admit I am making a lot of guesses of what might have been wrong and how the code works. I disabled all checks regarding the size of the
were adding the same vertices repeatedly to the moab_core object (because it is inside two for loops), and so I modified the function for this not to happen, ie I commented out lines 145 and 146 and added After this change the _vertices_to_h5m very quickly generated an .h5m file that I mbconverted to a.vtk that looks like my original step file at first glance. |
Nice work. The other thing I occasionally try is to manually mesh with gmsh manually meshing the step file gives good control over the mesh parameters 1 load up gmsh GUI then continue from line 21 of this example |
I wonder if the step file has something inside that I've not considered. Some sort of nested comp solid or something else, original_ids and scrambled_ids should be the same length so I guess something is getting expanded in the process |
Hello,
Thank you for creating this library!
I have been trying to use it to convert a .stp file to .h5m as described in the examples ( surface_mesh / single_stp_file.py). However it seems the process fails for me because
original_ids
andscrambled_ids
(line 565-566 of core.py) end up having a different size and the comments in the code tell me that this should not be the case.Unfortunately I cannot share the .stp file I am using, however if there are some things I can try to investigate this issue, please let me know.
The text was updated successfully, but these errors were encountered: