Model animation system#2331
Merged
RainWarrior merged 1 commit intoMinecraftForge:masterfrom Jan 24, 2016
Merged
Conversation
Closed
Contributor
There was a problem hiding this comment.
This confused me when I saw it in context. I honestly thought it was a Null Parameter from it's name.
Can I suggest this be PassthroughParameter or something to that effect.
RainWarrior
added
the
RFC
RainWarrior
force-pushed
the
model-animation
branch
2 times, most recently
from
d9f4ece to
0481339
Compare
Contributor
|
In order to support advanced use cases, as we have in SteamNSteel, it would be idea if the animation system could support the following: Triggers Specifying Timeframes in terms of FPS Start an animation, pretending that the world time is offset to 0 Upon finishing a non-looping clip, auto-transition to another. |
RainWarrior
force-pushed
the
model-animation
branch
2 times, most recently
from
56f39e8 to
66b0ee3
Compare
Main things of interest:
* IAnimationStateMachine - state machine for animations; can load
from json.
* AnimationTESR - automatic TESR for animated models.
* AnimationModelBase - same for entities.
* ITimeValue - time-varying value, used to control animation
parameters from code.
* TESRs can now be batched - look at TESR.renderTileEntityFast +
TE.hasFastRenderer.
* RegionRenderCache is not accessible to TESRs and other client-side
logic - MinecraftForgeClient.getRegionRenderCache.
RainWarrior
force-pushed
the
model-animation
branch
from
66b0ee3 to
0710bdf
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Continuation of #2308. Creating a new PR due to 1.8.X moving to the master branch.
WIP, making a PR for much needed feedback.
neither API nor actual json formats are finalized.
Related: #2163