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

Potential invalid fbx or wrong conversion #36

Closed
Apidcloud opened this issue Apr 14, 2023 · 6 comments
Closed

Potential invalid fbx or wrong conversion #36

Apidcloud opened this issue Apr 14, 2023 · 6 comments

Comments

@Apidcloud
Copy link

Hello!

A user uploaded a fbx over the last weekend and it created all sort of problems (see CesiumGS/gltf-pipeline#608 and google/draco#998), including the hanging of the whole server due to a bus error in draco.

I was able to reproduce it with the generated gltf, but now wondering if the original fbx already had issues. I don't know any fbx validator out there. I'm attaching the originally uploaded fbx here, and also the resulting gltf. Could you perhaps check if it is valid to begin with?

To be clear our pipeline is as follows:

  • run fbx through fbx2gltf without draco
  • run resulting conversion through gltf-pipeline with draco

Fixing the resulting gltf's accessors, as described in the issues makes the problem go away.

Thanks!

broken fbx and resulting gltf.zip

@fire
Copy link
Member

fire commented Apr 14, 2023

I will look as I can, but will be slow, here are my proposed debug steps.

  1. take fbx
  2. run fbx through fbx2gltf without draco
  3. open in vscode's validator for gltf
  4. open in blender
  5. analyze

@Apidcloud
Copy link
Author

Apidcloud commented Apr 14, 2023 via email

@Apidcloud
Copy link
Author

javagl from the linked issues imported the fbx to blender and exported it as obj, generating an obj with:

...
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 399274241452221131862835200.000000 -0.000000
vt 0.000000 0.000000
vt -398781418237547907481862144.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 75599770116763773093019648.000000 -0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
vt 456099141423378226263097344.000000 0.000000
vt 0.000000 0.000000
vt 0.000000 0.000000
...

Might suggest the FBX already contains garbage data and it's not a problem with the fbx conversion.

@fire
Copy link
Member

fire commented Apr 23, 2023

What do you expect to be the proper result?

@Apidcloud
Copy link
Author

Apidcloud commented Apr 24, 2023 via email

@fire
Copy link
Member

fire commented May 7, 2023

Closing due to "don’t know what’s the best way to handle it for now, other than using gltf validator"

@fire fire closed this as completed May 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants