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

Add animated glTF support #14685

Draft
wants to merge 62 commits into
base: master
Choose a base branch
from

Conversation

appgurueu
Copy link
Contributor

@appgurueu appgurueu commented May 21, 2024

This is the follow-up to #14557.

How to test

Test entities and unit tests are included.

@appgurueu appgurueu added @ Script API Roadmap The change matches an item on the current roadmap. Feature ✨ PRs that add or enhance a feature @ Client rendering Waiting (on dependency) Waiting on another PR or external circumstances (not for rebases/changes requested) labels May 21, 2024
@appgurueu appgurueu force-pushed the feat/gltf-loader++ branch 2 times, most recently from ac7bb44 to 0526309 Compare May 22, 2024 14:38
@appgurueu appgurueu force-pushed the feat/gltf-loader++ branch 2 times, most recently from 6b576c0 to 3ebf27c Compare June 12, 2024 18:07
@appgurueu appgurueu marked this pull request as ready for review June 12, 2024 18:52
@appgurueu
Copy link
Contributor Author

appgurueu commented Jun 12, 2024

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++.

@rubenwardy rubenwardy marked this pull request as draft June 13, 2024 16:02
@appgurueu appgurueu added this to the 5.10.0 milestone Jun 15, 2024
@appgurueu
Copy link
Contributor Author

appgurueu commented Jun 15, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@ Client rendering Feature ✨ PRs that add or enhance a feature Roadmap The change matches an item on the current roadmap. @ Script API Waiting (on dependency) Waiting on another PR or external circumstances (not for rebases/changes requested)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants