Repository navigation
t27f: token_floor renders in 48 shards; frames cached by the seal (Closes #10383) - #10384
Merged
Merged
Conversation
…oses #10383) - token_floor main renders shard k of N (TF_SHARD, TF_SHARDS). The physics steps from frame 0, and only the splats and the raster before the range are skipped. The partition is tested to tile [0, FRAMES) for every N from 1 to 64; bad input renders the whole reel. - New specs/t27f/frame_cache.t27 derives the frames' cache key from the token_floor seal. Each sealed digest (the spec and its closure) is checked against sha256sum now, and the key adds the t27c binary, zig and the build flags. A stale seal, failed tests or a missing part refuse the key, which means a miss. It lowers under gen-c, so the lab runs it as a C filter. - Lab: 48 shards in 3.43 s versus 54.0 s in one process. Both give sha256 7e0c6662 byte for byte, and the muxed reel is the v5 file (a18389ed). Tests: token_floor 24/24, frame_cache 8/8, 0 vacuous. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
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.
Closes #10383
The token_floor reel took 52 s to render on one of the lab's 48 cores, and every caption, voice or music edit paid that cost again.
What changed
specs/t27f/token_floor.t27, frame shards.mainrenders shard k of N, read fromTF_SHARDandTF_SHARDS. Shard k covers the frames[shard_a(k, N), shard_b(k, N)).[0, FRAMES)in order, with no gap or overlap, for every N from 1 to 64;getenvis reached only frommain, so the test runner never needs libc. The binary is built with-lc.specs/t27f/frame_cache.t27, the cache key of the rendered frames..trinity/seals/t27f_token_floor.jsonseal;sha256sumlines for the specs;zig versionand the build flags./root/tf/cache/<sha256 of the record>.rgb.t27c seal --verifychecks only the spec, not its closure.token_floor_mux.t27is not in the closure, so caption and mux edits keep the key.main, the same idiom as the lefthook and claw-sarif jobs.Measured on the Railway lab
The lab has 48 cores,
-O ReleaseSafe -lcand zig 0.16.0. Other jobs kept its load average around 50 during these runs./root/tf/cache)-lcbuild gives the same frames as the earlier build without it.refused: the spec changed since the sealand exited 1.Per-spec test reports on the lab:
Not in this PR
/root/tf/k/reel.sh) is not committed; it stays on the lab.🤖 Generated with Claude Code