You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As I tried this snippet, I found out the format support is very limited, as well as various bugs are present in the code. If it's still supported, here's the small list of issues I found:
Empty file without the data crashes the code (i.e. blender's empty scene PLY export).
Various exporters use vertex_indices line even if they export in ascii mode. Only vertex_index is hardcoded to the switch though, meaning that perfectly fine ascii models that can be parsed will fail to do so, unless added extra cases. Correct way to implement this would be to actually parse this property line.
case "property list uchar int vertex_index":
case "property list uchar int vertex_indices":
case "property list uchar uint vertex_indices":
Wrong indexing in this line of code, requires -1 on both params to work properly. Didn't check the exact reason why, but other combinations crashed the code.
As I tried this snippet, I found out the format support is very limited, as well as various bugs are present in the code. If it's still supported, here's the small list of issues I found:
The text was updated successfully, but these errors were encountered: