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

Skinning broken for glTF 1.0 models with ModelExperimental #10621

Open
lilleyse opened this issue Aug 2, 2022 · 2 comments
Open

Skinning broken for glTF 1.0 models with ModelExperimental #10621

lilleyse opened this issue Aug 2, 2022 · 2 comments

Comments

@lilleyse
Copy link
Contributor

lilleyse commented Aug 2, 2022

Several glTF 1.0 models aren't skinning or animating correctly in ModelExperimental. Interestingly the 2.0 versions of those models don't have any issues. It's possible the models were sanitized or fixed in some way before they were converted to glTF 2.0.

Ignore the missing materials - that will be addressed separately in #10613.

From https://github.com/KhronosGroup/glTF-Sample-Models/tree/master/1.0:

  • BrainStem
  • CesiumMan
  • Monster
  • RiggedFigure
  • WalkingLady

ezgif com-gif-maker (1)

@javagl
Copy link
Contributor

javagl commented Aug 3, 2022

Don't focus on the WalkingLady model. There have been several issues with that one (and... when you find yourself in a rabbit hole, you might as well learn something about rabbits - including what might be my first bugfix in CesiumJS 😁 ). I think it is now considered to be "invalid", and has been removed from the sample models repo. (The other models should work, though, but they don't appear that broken...)

@lilleyse
Copy link
Contributor Author

lilleyse commented Aug 3, 2022

Haha, I remember the WalkingLady model being one of the more problematic ones when adding support for glTF 2.0. 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants