Skip to content

fix: vse-cut-list quality pass - #245

Merged
TMHSDigital merged 1 commit into
mainfrom
fix/vse-cut-list-quality-pass
Sep 25, 2026
Merged

TMHSDigital merged 1 commit into
mainfrom
fix/vse-cut-list-quality-pass

Conversation

@TMHSDigital

Copy link
Copy Markdown
Owner

Quality pass on vse-cut-list, a Track B example: the contract is frozen, and only the staging of the editing bay changes. The binaries were .scratch/blender-4.5.11-windows-x64 (reports Blender 4.5.11 LTS), blender-5.1.2-windows-x64 (Blender 5.1.2) and blender-5.2.1-windows-x64 (Blender 5.2.1 LTS). The hero was rendered on 5.2.1 through scripts/render_hero.py. The inspection sheets (hero angle, clay three-quarter, wireframe, per-object close-ups, ground contact) come from the example's own Bay scene via its framing call and stayed in .scratch/.

Picks: every showcase piece and every check-only example has already had a pass. I inspected three unreviewed gallery examples and only one had a defect worth fixing:

  • vse-cut-list: fixed below.
  • png-exr-alpha: both displays stand on struts in a plinth, captions are legible, ground contact is sound. No change.
  • socket-attach-points: a hovering drone with its skid clear of the floor, every part attached, framing ok. No change.

vse-cut-list (Track B)

Defects (round 0):

  1. The monitor neck's top face (z 1.24) sat exactly on the bezel's bottom face, a butt joint on one plane with no bite. Half the 0.12 m neck hung out behind the 0.09 m bezel. Seen in the clay three-quarter and the wireframe.
  2. The stand base sat exactly on the desk top, another coplanar contact.
  3. The "PROGRAM" caption lay flat on the desk top, floating 6 mm above it, and read as a smear at the camera's grazing angle. Seen in the hero and close_5_BayCaption.

Presentation changed: the part heights are now derived from named constants in the order they stack. The base bites the desk top by 5 mm; the neck starts inside the base and ends 0.10 m inside the bezel; the neck's depth (0.07 m) sits within the bezel's depth, set 0.02 m back. The caption stands upright on the desk's front face, set 1 mm into it. It is the same text curve with the same body, so nothing the example asserts about text strips is involved.

Contract unchanged: the sequencer frame on the screen is still the genuine rendered output. Every check and exit code is untouched, and the check line printed before and after is identical.

4.5.11 5.1.2 5.2.1
before: default / --swap-inputs / --output 0 / 7 / 0 0 / 7 / 0 0 / 7 / 0
after: default / --swap-inputs / --output 0 / 7 / 0 0 / 7 / 0 0 / 7 / 0

Framing on the render path: fill y 0.850, minimum margin 0.067, ok.

Contact sheet

vse-cut-list: luma 0.2215, stage 0.195, warmth +0.018, saturation 0.10. All are in family and essentially unchanged from the old hero (0.2214), since the edit is small and local. Legible at thumbnail size, and the caption now reads.

Checks run locally: check_exit_code_readme.py and check_gallery_images.py pass. I read the generated alt text in docs/gallery/index.html and docs/gallery/vse-cut-list/index.html.

🤖 Generated with Claude Code

Render path only; the Bay scene is staging. The monitor neck butted its
top face onto the bezel's bottom on one plane with half its depth behind,
and the PROGRAM caption lay flat on the desk 6 mm up, unreadable from the
camera. The neck now rises 0.10 m into the bezel, the base sits 5 mm into
the desk, and the caption stands on the desk's front face. Sequencer frame,
checks and exit codes unchanged on all three binaries.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Signed-off-by: TMHSDigital <154358121+TMHSDigital@users.noreply.github.com>
@TMHSDigital TMHSDigital added the needs-5.1 Opt-in Blender 5.1 smoke on this PR. Default matrix stays 4.5 + 5.2. Auto-label will not apply this. label Sep 24, 2026
@github-actions github-actions Bot added examples Runnable smoke-gated examples under examples/ documentation Improvements or additions to documentation labels Sep 24, 2026
@TMHSDigital
TMHSDigital merged commit 0c818a9 into main Sep 25, 2026
14 checks passed
@TMHSDigital
TMHSDigital deleted the fix/vse-cut-list-quality-pass branch September 25, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation examples Runnable smoke-gated examples under examples/ needs-5.1 Opt-in Blender 5.1 smoke on this PR. Default matrix stays 4.5 + 5.2. Auto-label will not apply this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant