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

T60268: GLTF export fails with simple bone animation #209

Closed
julienduroure opened this issue Jan 7, 2019 · 6 comments · Fixed by #217
Closed

T60268: GLTF export fails with simple bone animation #209

julienduroure opened this issue Jan 7, 2019 · 6 comments · Fixed by #217
Labels
bug Something isn't working exporter This involves or affects the export process
Milestone

Comments

@julienduroure
Copy link
Collaborator

From blender tracker : https://developer.blender.org/T60268

Python error & example blend file are provided in blender issue

@julienduroure julienduroure added bug Something isn't working exporter This involves or affects the export process labels Jan 7, 2019
@UX3D-nopper UX3D-nopper added this to the MS VI milestone Jan 8, 2019
@ghost
Copy link

ghost commented Jan 8, 2019

Please attach the used images — without them I can't even get that far.

@julienduroure
Copy link
Collaborator Author

@UX3D-schmithuesen Please comment original issue on blender tracker, it will take too much time for me to replicate messages here & on blender tracker

But ... This error is linked to animation, you should be able to reproduce without the texture, by removing material

@ghost
Copy link

ghost commented Jan 8, 2019

Both very valid points! I will create an account for the blender tracker, but first I will investigate this thing without materials.

@ghost ghost closed this as completed in #217 Jan 8, 2019
@julienduroure
Copy link
Collaborator Author

@UX3D-schmithuesen
I don't know how we have to handle it:

  • Not exporting entire animation
  • Ignore this curve, and export other curves that can be exported on same animation

Any thoughts @donmccurdy ?

@donmccurdy
Copy link
Contributor

I think I'd lean toward ignoring this channel but exporting the rest of the animation. If there is some way to report a user-visible warning (per #89) without aborting the export that may be ideal.

@ghost
Copy link

ghost commented Jan 10, 2019

Right now, we display a warning on the console (I don't know if that counts as user-visible) and skip that entire animation, but still export the rest of the asset.

Ignore this curve, and export other curves that can be exported on same animation

It sounds very good at first glance and should not even be that hard to implement, but we don't know if the animation will still be useful in the end. I like skipping the entire animation, because even without seeing the warning, the user will notice right away that the animation was not (correctly) exported while just missing piece of an animation might go unnoticed at first.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working exporter This involves or affects the export process
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants