Skip to content

BugFix the Necromancer Animation classes, now is usable - #196

Merged
Ocelot5836 merged 6 commits into
FoundryMC:next/1.21from
ElitTerrator:1.21
Sep 8, 2026
Merged

BugFix the Necromancer Animation classes, now is usable#196
Ocelot5836 merged 6 commits into
FoundryMC:next/1.21from
ElitTerrator:1.21

Conversation

@ElitTerrator

Copy link
Copy Markdown
Contributor

BugFixes:

  • KeyframeTimeline

    • ArrayIndexOutOfBoundsException becuase Math.max(nextIndex, keyframes.length - 1); and Math.max(nextNextIndex, keyframes.length - 1);
    • on findKeyframeIndex, there is no check for time == t1
  • KeyframedAnimation

    • bone.size.set() when setting the position
    • tempRotationA.identity() will cause to not rotate when mixFactor = 1'
    • changes on the API, to prevent unchecked-cast warning on this.addAnimation((Animation<ModelEntity, ModelSkeleton>) builder1.build(), priotity);

@Ocelot5836
Ocelot5836 changed the base branch from 1.21 to next/1.21 September 8, 2026 04:09
@Ocelot5836
Ocelot5836 merged commit 9fe94f2 into FoundryMC:next/1.21 Sep 8, 2026
Ocelot5836 added a commit that referenced this pull request Sep 8, 2026
* Update Interpolation.java

* Update Interpolation.java

* Update KeyframeTimeline.java

* Update KeyframedAnimation.java

* Fix import and backwards compatibility

---------

Co-authored-by: Ocelot <FinntheRaider@gmail.com>
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