Skip to content

Add glTF STEP interpolation support#15525

Merged
sfan5 merged 1 commit into
luanti-org:masterfrom
appgurueu:feat/gltf-const-interp
Dec 24, 2024
Merged

Add glTF STEP interpolation support#15525
sfan5 merged 1 commit into
luanti-org:masterfrom
appgurueu:feat/gltf-const-interp

Conversation

@appgurueu

@appgurueu appgurueu commented Dec 7, 2024

Copy link
Copy Markdown
Contributor

Adds support for glTF STEP interpolation (no interpolation). It turns out many models require this (often unnecessarily for channels with singular values because Blender apparently forcibly creates STEP channels for unanimated properties by default). Related to a point on #15410.

How to test

Spawn the simple skin step test entity, compare it against the linearly interpolated one. Observe no interpolation.

@appgurueu appgurueu added @ Script API Feature ✨ PRs that add or enhance a feature Waiting (on dependency) Waiting on another PR or external circumstances (not for rebases/changes requested) labels Dec 7, 2024
@appgurueu
appgurueu force-pushed the feat/gltf-const-interp branch from 2efbc64 to 8d65720 Compare December 12, 2024 15:28
@appgurueu appgurueu removed the Waiting (on dependency) Waiting on another PR or external circumstances (not for rebases/changes requested) label Dec 12, 2024

@sfan5 sfan5 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

read the spec and our code, should work

@sfan5
sfan5 merged commit b087e25 into luanti-org:master Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants