Describe the playblast in the composed prompt (#164) #175

Merged

crowmaki merged 2 commits from issue-164-previz-prompt into main

2026-10-11 05:26:07 +00:00

crowmaki commented

2026-10-11 05:25:16 +00:00

Owner

Closes #164

When a Take sends a playblast, retention_analysis now has an entry for it:

<Video 1>: reference - a flat-shaded previz of the shot, a motion and camera reference only: follow how the subjects move, where they go and when, and how the camera moves; ignore its colours, its shading and the look of the figures, which carry no identity. <Subject 1> is the blue figure in <Video 1>. <Subject 2> is the red figure in <Video 1>.

The colours come from one shared figure_colour(index) helper, which the scene spec also uses now. Both read the Setup's subject order, so the prompt's colours always match the Blender figures, even when the action text numbers the subjects in another order.
Only Setup subjects get a colour. Characters who are only mentioned or only heard have no figure.
An excluded playblast, or a Shot that isn't complex, leaves the prompt exactly as it was.

Deviation from the issue text

The issue's example says "Mara is the red figure". The composer's rule is that nothing is named, so the sentence says <Subject N> instead of the Character's name.

Known limit

A Setup with more than 8 subjects reuses palette colours, so two subjects can be "the red figure". The scene spec already had this limit. This PR doesn't change it.

Tests

Four new tests in crates/studio-core/tests/previz.rs: the motion-only sentence with its <Video 1> tag, colours matching the scene spec (also after the subjects are reordered), and no sentence when the playblast is excluded or the Shot isn't complex. The full workspace suite passes.

Closes #164 When a Take sends a playblast, `retention_analysis` now has an entry for it: > `<Video 1>: reference - a flat-shaded previz of the shot, a motion and camera reference only: follow how the subjects move, where they go and when, and how the camera moves; ignore its colours, its shading and the look of the figures, which carry no identity. <Subject 1> is the blue figure in <Video 1>. <Subject 2> is the red figure in <Video 1>.` - The colours come from one shared `figure_colour(index)` helper, which the scene spec also uses now. Both read the Setup's subject order, so the prompt's colours always match the Blender figures, even when the action text numbers the subjects in another order. - Only Setup subjects get a colour. Characters who are only mentioned or only heard have no figure. - An excluded playblast, or a Shot that isn't complex, leaves the prompt exactly as it was. ## Deviation from the issue text The issue's example says "Mara is the red figure". The composer's rule is that nothing is named, so the sentence says `<Subject N>` instead of the Character's name. ## Known limit A Setup with more than 8 subjects reuses palette colours, so two subjects can be "the red figure". The scene spec already had this limit. This PR doesn't change it. ## Tests Four new tests in `crates/studio-core/tests/previz.rs`: the motion-only sentence with its `<Video 1>` tag, colours matching the scene spec (also after the subjects are reordered), and no sentence when the playblast is excluded or the Shot isn't complex. The full workspace suite passes.

crowmaki added 2 commits

2026-10-11 05:25:17 +00:00

Describe the playblast in the composed prompt (#164 ) 0abf6dfc6e

When a Take sends a playblast, retention_analysis names it a motion and
camera reference only and maps each subject to the colour of its figure,
using the same palette assignment as the Shot's scene spec.

Address review of #164 03ae134f24

crowmaki merged commit 1d75a272b1 into main

2026-10-11 05:26:07 +00:00

crowmaki deleted branch issue-164-previz-prompt

2026-10-11 05:26:07 +00:00

crowmaki referenced this pull request from a commit

2026-10-11 05:26:08 +00:00

Merge pull request 'Describe the playblast in the composed prompt (#164)' (#175) from issue-164-previz-prompt into main

crowmaki referenced this pull request

2026-10-11 06:41:39 +00:00

More than 8 subjects reuse previz figure colours #182