Repository navigation
t27f: R4f -- mode 5 holds the address for REEL_CTA_MIN_MS, cut 1 capped at BEAT_CUT1_LAST_MS (Closes #10629) - #10632
Merged
Merged
Conversation
…s, crossfaded (Closes #10596) R4a of epic #10560. lesson_reel.t27 gains a beat clock: cut 0 at the goal phrase's start (held to a 1250 ms open floor), cut 1 at cta_from, each a crossfade of reel.t27's TRANSITION_FRAMES centred on the cut. Seven tests: frame constants, lesson 1's measured cuts, the short-title floor, a grid of 80 voices x every lesson x both languages (full clock, voice too short for an open scene, refused), weights summing to 1000 at every 10 ms, exactly two beat changes over 450 frames, and a fixed-thirds control that lands mid-phrase. The clock lives in lesson_reel.t27, not in the beat_clock.t27 module the issue names: importing phrase_start or cta_from into another module leaves them UNRESOLVED because token_floor_mux.t27 declares the same names, and the splicer drops ambiguous imports even on an explicit path (#10556). Lab: 13/13 tests pass, 0 vacuous; seal saved and verified. t27b still blocked on type [N]T, as the ledger row already says; max_not_pass unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…eel (Refs #10596) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ock's cuts, on whole frames (Closes #10603) lesson_reel.t27 gains: - beat_xfade_frame: the last base frame at or before c - BEAT_HALF_MS. - beat_solo_frames: a floor on how many frames each beat shows alone. - beat_graph: inputs 0..2 (the open, main and close scene clips), crossfaded with xfade at offset beat_frame_ms(f) in AV_TIME_BASE, then trimmed to 450 frames. - LR_MODE=4, which prints the graph and refuses a voice that does not fit, as mode 3 does. Tests: 17/17 pass, 0 vacuous. - New: every cut from 1250 to 13500 ms lands on its frame. - New: the offset for every frame 0..450 selects that frame. - New control: an offset rounded up misses 300 of 451 frames. - New: on R4a's voice grid, every beat keeps MIN_SCENE_FRAMES whole frames, except the counted early case. Measured with ffmpeg 5.1 on the Railway lab, on solid-colour inputs, 450 frames each: - 1300/4200/6800 ms: runs 50 / 332 / 38. - 2500/8000/9000 ms: runs 88 / 235 / 98. - 200/1500/13100 ms: runs 28 / 30 / 362. - 9000/9000/9000 ms: refused. Every run equals beat_solo_frames, with one extra frame when the offset falls on a whole ms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…r runs (Refs #10603) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… LR_MODE=5 (Closes #10609) lesson_reel.t27: - graph() takes a beats flag. With it, the picture is the beat chain over inputs 0..2 instead of input 0, and the voice and music inputs move past the three clips (3..5 and 6). - beat_graph() is now beat_chain() plus its [v] label, so mode 5 goes on filtering the same stream. - beat_video_inputs / beat_voice_input / beat_music_input number the inputs. - LR_MODE=5 prints both orientations and refuses a voice that does not fit. Tests: 19/19 pass, 0 vacuous. New: every input from the picture to the music is used exactly once in both layouts. New control: without beats the inputs are mode 3's. Regression on the Railway lab: 96 graphs (4 lessons, 2 languages, 4 voices; mode 3 against master and the beat-graph branch, mode 4 against the beat-graph branch) are byte-identical. Mode 5 rendered with ffmpeg 5.1 from solid-colour clips, sine phrases and music. Tall and wide each give 450 frames and 15.000 s of audio. Pixels outside the text run 50 red, 332 green and 38 blue frames, which equals beat_solo_frames for 1300/4200/6800 ms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…th cmp, and three lab traps (Refs #10609) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… clock (Refs #10609) specs/skills/t27-reel-beat.t27 is a spec-only skill card, in the same shape as the four t27-social-visual cards from #10570. It points at specs/t27f/lesson_reel.t27 (beat_* functions, LR_MODE=4 and LR_MODE=5) and copies none of it. It does not cite reel_factory.t27: lesson_reel does not read the scene plan yet, and the card says so. Railway lab: 1/1 test, 9 runtime asserts, 0 vacuous. Seal saved and verified. t27b's front end accepts it (x86 lab: JIT-only CRASH, as for t27-self-host), so the ledger row is pass. The ledger row sits between providers-steward and self-host. That keeps it one line away from the four rows #10613 adds after self-host, so the two PRs do not conflict. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ress on a beat crossfade (Closes #10614) LR_MODE=5 now takes every caption, tag and address window from the beat clock, on whole frames. beat_scene_lo/hi give the frames each scene owns outside its crossfades; beat_piece_lo/hi clip an overlay to them; beat_piece_kept drops a piece with no room (fewer than BEAT_OVERLAY_MIN_FRAMES and not the whole overlay) instead of squeezing it; and print_pieces prints exactly those frames as between(n,lo,hi) terms. Modes 3 and 4 print the same bytes as before. Railway lab: - t27c test-report: 25/25, 0 vacuous; seal saved and verified. - The grid test covers every lesson, both languages and every voice the clock accepts: 85437 asserts that no enabled frame is on a crossfade. A control shows mode 3's windows over mode 4's cuts put the goal's first caption on one. - Regression: 216 graphs (modes 3 and 4, 27 lessons, 2 languages, 2 voices) cmp equal to the #10610 binary. - ffmpeg, mode 5 rendered from solid-colour clips, both orientations: a 40-row band over the captions shows text on 405 frames and on 0 of the 30 crossfade frames; the tag band, 404 and 0. Costs, named in tests and the header: a short caption whose scene ends under it is dropped (every phrase keeps one); on a voice the reel speeds up the address is held for less than REEL_CTA_MIN_MS; the cards are still on their own frames, since token_floor_mux prints their enable. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… is dropped, and two lab traps (Refs #10614) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ndows # Conflicts: # .claude/skills/t27f/SKILL.md # .trinity/seals/t27f_lesson_reel.json # specs/t27f/lesson_reel.t27
…ed at BEAT_CUT1_LAST_MS (Closes #10629) On a voice that runs long the reel speeds it up so the address starts at REEL_MS - REEL_CTA_MIN_MS, but the beat clock centred crossfade 1 on that start, so the close beat owned fewer than 45 frames and mode 5 held the address for less than REEL_CTA_MIN_MS. beat_cut_ms now caps cut 1 at BEAT_CUT1_LAST_MS = 13249, the latest cut whose close beat still owns 45 frames (one ms later it owns 44). Tests: the bound itself with a +1 ms control; every lesson, both languages, every grid voice the clock accepts holds the address for the floor; the cost -- on a moved cut crossfade 1 starts under the task's last words, by at most 184 ms, and lesson 1's measured voice is one of them. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 18, 2026
Contributor
gHashTag
added a commit
that referenced
this pull request
Oct 18, 2026
Closes #10651) (#10675) Whole-frame colour count on every crossfade frame of every lesson, language, voice and orientation with PR #10647's binary: 10368 frames, 0 flagged. The #10632 control is flagged on 162 of 162 renders. Adds the method and four ffmpeg lab traps (-nostdin, pids.max thread caps, lavfi duration format, pkill leaving children). Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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
1 participant
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 #10629
Stacked on #10627 (R4d beat windows), which carries #10610 and #10605. This PR is based on master and therefore carries those PRs as well; merge #10627 first or let this one carry it.
What was wrong
When a voice runs long, the reel speeds it up so the address starts at
REEL_MS - REEL_CTA_MIN_MS(13500 ms). The beat clock then centred crossfade 1 on that start, so the close beat owned fewer than 45 frames. Mode 5 therefore showed the address for less thanREEL_CTA_MIN_MS. Lesson 1's measured voice was one of the affected voices.The change
beat_cut_msnow caps cut 1 atBEAT_CUT1_LAST_MS = 13249. That is the latest cut at which the close beat still owns 45 frames: crossfade frame 389, and the close beat owns frames 405..449. One millisecond later the crossfade lands on frame 390 and the close beat owns only 44 frames. One test asserts both sides of that edge.Evidence (Railway lab)
t27c test-report--verifyall hashes MATCH; spec sha256 a380a3fe... equals the committed file[*:0]const u8, env_num, @sin), nothing new[x0][b2]xfade offset13.233 -> 12.966 (the sped-up voices)n-376->n-405)The cost, stated plainly
On a voice the cap moves, crossfade 1 starts while the task's last words are still being spoken, by at most 184 ms. A test asserts that bound, and asserts that lesson 1's measured voice falls under it. The header's NOT CLAIMED section records this as well.
Not checked
🤖 Generated with Claude Code