-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
UVIndex not handled #5
Comments
Thanks. Do you happen to have a fbx with this data in it so I can make sure it's correct? |
https://github.com/klaszlo8207/Flutter-FBX-3D-Viewer/tree/master/assets Here it is some fbx files, version 7.4.0 ASCII Brendan I had to update some BUGS in your code and I had to upload to my package: https://github.com/klaszlo8207/Flutter-FBX-3D-Viewer/tree/master/lib/fbx_parser If you FIX those all I can use your library then later I found 3 BUGS (mentioned here in the issues as well), you can find in my repo with searching for: "//TODO" or just compare the codes https://github.com/klaszlo8207/Flutter-FBX-3D-Viewer/search?q=%2F%2FTODO&unscoped_q=%2F%2FTODO Thanks. |
More on the BUGS: line 175-184 line 390 null check (crash was) line 414-424 |
That's great, I'll add those fixes and make sure those fbx files work. I did add a couple fixes last night to get an fbx file exported out of blender to load. You're the first person I know of to have used this library, so much appreciated sharing your fixes. I haven't even used it for anything, I was just curious what was inside fbx files. |
that issues already fixed? or not yet? @klaszlo8207 I updated some code and already pull request to your repo package. i hope this package will be work again. |
You assume that the UvIndices is in order but not, you have to handle that and order manually
in
void _loadUvs(FbxElement e) { //fbx_mesh.dart
add this
then in generateDisplayMeshes: (to the last line add)
The text was updated successfully, but these errors were encountered: