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
try/catch suggestion #1
Comments
hi eric, thank you for the suggestion! why, though? the whole operation is wrapped in an exception handler anyway. do you have an example where some individual faces may fail, and you still want to retain the rest? if so, i'll add this right away. cheers jeremy |
Here is an OBJ file with errors in it: https://drive.google.com/file/d/0B_UPhfVzWDkxMWJEVnMtdWN0N28/view?usp=sharing |
what exact exception is thrown, please? you should never add a catch-all handler if you can avoid it. |
i do not see the sample OBJ file you refer to. where is it? |
i updated the code with a catch-all exception handler. i would like to narrow it down to the specific exception thrown, though. |
I think this OBJ tries to make 2-vertex faces https://drive.google.com/file/d/0B_UPhfVzWDkxMWJEVnMtdWN0N28/view?usp=sharing |
thx, i'll take a look. |
nope, if you simply look at the OBJ in a text editor, you see that all faces have four vertices. maybe some of the four vertices list the same one or two vertices twice? i'll test... you should be able to debug this also, man :-) |
Even MeshLab complains about that OBJ: |
hi eric, thank you for the high_ball_glass.obj sample OBJ. i added the exception handler you suggest, and also added a result log in the debug console window reporting the number of faces that succeeded and failed on each shape. |
closed with release 2015.0.0.17. |
I suggest adding try/catch around builder.AddFace
The text was updated successfully, but these errors were encountered: