Skip to content

Increase performance of bedrock animations & Loop type support#64

Merged
drtheodor merged 10 commits intomainfrom
perf/anim
Apr 8, 2026
Merged

Increase performance of bedrock animations & Loop type support#64
drtheodor merged 10 commits intomainfrom
perf/anim

Conversation

@duzos
Copy link
Copy Markdown
Member

@duzos duzos commented Jan 25, 2026

Previously we were iterating over all the bones to find the specified bone, every frame
Now we just cache it

closes #63

@duzos duzos self-assigned this Jan 25, 2026
@duzos duzos marked this pull request as ready for review January 25, 2026 20:46
@duzos duzos requested a review from drtheodor January 25, 2026 20:46
@duzos duzos added the enhancement New feature or request label Jan 25, 2026
@duzos duzos changed the title Increase performance of bedrock animations Increase performance of bedrock animations & Loop type support Jan 27, 2026
Comment thread src/main/java/dev/amble/lib/client/bedrock/BedrockAnimation.java
Comment thread src/main/java/dev/amble/lib/client/bedrock/BedrockAnimation.java Outdated
@duzos duzos requested a review from drtheodor February 3, 2026 00:16
Copy link
Copy Markdown
Member

@drtheodor drtheodor left a comment

Choose a reason for hiding this comment

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

lgtm (havent read it)

@drtheodor drtheodor merged commit 09a6d1d into main Apr 8, 2026
1 check passed
@drtheodor drtheodor deleted the perf/anim branch April 8, 2026 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"hold last frame" animation type always resets

2 participants