Skip to content

fix: respect animation priority and defer cinematic anims until model ready#2

Merged
OmeWillem merged 4 commits into
OmeWillem:mainfrom
Taiyou06:main
May 28, 2026
Merged

fix: respect animation priority and defer cinematic anims until model ready#2
OmeWillem merged 4 commits into
OmeWillem:mainfrom
Taiyou06:main

Conversation

@Taiyou06

Copy link
Copy Markdown
Contributor
  • Route play/stop animation calls through IStateMachineHandler priority slots with an auto-assigned hash-based slot per animation name, so distinct animations stack instead of replacing each other at ModelEngine's default slot
  • Add an explicit priority field on AnimationSettings and the stop entries to pin a specific slot when needed; non-state-machine handlers keep the original behavior
  • Apply the same routing to the cinematic segment's animation list so the inspector's Animations entries actually stack
  • Queue animation calls via ModelEngineEntity.whenModelReady and flush them after activeModel is initialized, fixing the first-open cutscene race where frame-0 animations were dropped before the deferred spawn runnable fired

@Taiyou06

Copy link
Copy Markdown
Contributor Author

bump @OmeWillem

@OmeWillem OmeWillem merged commit 79e2146 into OmeWillem:main May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants