-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add animated glTF support #14685
base: master
Are you sure you want to change the base?
Add animated glTF support #14685
Conversation
ac7bb44
to
0526309
Compare
Co-authored-by: sfan5 <[email protected]>
This also corrects the function name to be snake_case because it is not a method.
6b576c0
to
3ebf27c
Compare
3ebf27c
to
7dd13bc
Compare
Rebased. The static PR is ready. This is ready too. The diff after the static PR is merged amounts to about 400 loc, as can be seen here: JosiahWI/minetest@feat/gltf-loader...appgurueu:minetest:feat/gltf-loader++. |
84c5373
to
d9a37a2
Compare
To clarify the state of this PR: This PR is ready for review, as is the static PR. It depends on the static PR however, hence the "waiting on dependency" tag. It is probably a good idea to review the static PR first, until it is merged. |
This is the follow-up to #14557.
How to test
Test entities and unit tests are included.