Repository navigation
Conversation
…ims.json #156: the chair's moves (sweeps, overhead, low sweep, guard and its push) and the rover, crab and drone timed their frames as n / 12. Each model now takes its board's anims.json (combat/clips.ts: frameTime, frameCount, clipTime, frameAt); the chair and the enemy row pass the loaded tables, and without one the authored 12 fps stays. The unused lifePose / paintLife (generic white tint) are gone. Refs #156
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
arudaev
changed the title
arudaev
changed the title
github-actions
Bot
added
area:player
The arena-platform board named tmp/girder-ref.png as an input; tmp/ is not in Git, so check_docs failed in CI. The crop of the maze girder is committed next to the arena plate. Refs #72
github-actions
Bot
added
area:art
Member
Author
|
Replaced by #224: the same commits replayed on the current base of the stack (this branch conflicted after the platform fixes were carried up). No force-push. |
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
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.
Stack: 8 of 8 (playtest): base #218. Merge bottom-up, only on Alexander's word.
What and why
#156, criterion "one timing source": every clip's speed comes from its
anims.json, with non / 12left in gameplay code. Before this PR, the chair's moves and the three enemies timed their frames by hand:combat/clips.ts:frameTime,frameCount,clipTimeandframeAtover a board's animation table. Without a table (tests, a board not loaded yet), the authored 12 fps and frame counts stay, so behaviour is unchanged today.Sweeps,Overhead,LowSweep,Guard,GuardPush): each takes its board's table (chair-side,chair-overhead,chair-block).ChairCombatand the player pass the loaded ones. Retiming a clip is now an edit to itsanims.json(or the board'sfpsin boards.json), not to five files.enemy-rover,enemy-crabandenemy-dronepassed fromrovers.ts.lifePoseandpaintLife, unused, which carried their own* 12and the genericsetTintFillflash.Left, on purpose:
hero-ladder.anims.json;jennyRam.ts), a scripted timeline. That one is next for Rework movement and combat feel against the art #156.Issue
Refs #156 (the timing criterion; the stride, transitions, data-not-code and played-and-recorded criteria stay open)
Verification
npm test: 336 passed (every existing model test unchanged, run on the defaults);tscclean.clipsset) animate on their own clocks (drive-02→drive-04,walk-02→walk-04,hover), with no errors.