Skip to content

Fix mannequin foot twisting 180 during jump loop animation - #3459

Merged
xen2 merged 1 commit into
stride3d:masterfrom
ds5678:mannequin_jump_loop_fix
Sep 21, 2026
Merged

xen2 merged 1 commit into
stride3d:masterfrom
ds5678:mannequin_jump_loop_fix

Conversation

@ds5678

@ds5678 ds5678 commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Issue

The foot unnaturally twists 180 degrees during the loop animation for jumping.

Stride.GameStudio_GhKljztzpw.mp4

Resolves #182

PR Details

I opened the animation file in Blender and edited that rotation.

Stride.GameStudio_eE7S1kIYh0.mp4

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@xen2

xen2 commented Sep 21, 2026

Copy link
Copy Markdown
Member

Curious, was it really a problem with the source FBX data, or something on our side with the quaternion/rotation inerpolation in the animation engine?

@ds5678

ds5678 commented Sep 21, 2026

Copy link
Copy Markdown
Contributor Author

Curious, was it really a problem with the source FBX data, or something on our side with the quaternion/rotation inerpolation in the animation engine?

The ankle did the same movement in Blender, so I doubt it's any fault of the engine.

@ds5678

ds5678 commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor Author

Also this is the only mannequin animation that has any obvious issue. I think it was just an accidental mistake by the animator.

I checked all the animations while investigating this issue a few days ago. The only other animation that looked remotely wrong to me was one of the sword swings.

Edit: I just double checked the sword animations at a really slow speed. The one I think looks weird is Sword_L, but there's no unnatural rotation. The way that the character handles the sword just looks incorrect to my eyes. No evidence of bugs in the engine.

@w0wca7a

w0wca7a commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Related #182

@MsEpsilon MsEpsilon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks great

@xen2
xen2 merged commit 4b49890 into stride3d:master Sep 21, 2026
26 checks passed
@ds5678
ds5678 deleted the mannequin_jump_loop_fix branch September 21, 2026 17:48
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.

Character ankle behaves strangely when jumping from high places

4 participants