Skip to content

Small follow-ups: streams_error in the console, box editor inset, import checks snapshot crops, crop drift, T-bar edges - #117

Merged
LeeO86 merged 1 commit into
devfrom
feat/fb-small-fixes
Oct 23, 2026
Merged

LeeO86 merged 1 commit into
devfrom
feat/fb-small-fixes

Conversation

@LeeO86

@LeeO86 LeeO86 commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Adrian's round 2026-10-23, item 2.

  • Console shows why Program fell back to the mosaic: the PROGRAM title gets an amber note from the mosaic: <reason> when GET /preview → streams_error is set (the engine could not open the Program streams' NVENC sessions).
  • Box editor over an inset mosaic tile (~1 % off): preview.ts rasterView() gives the part of the raster a picture shows (all of it for a Program stream; 5 mosaic px less at each edge for a tile, e.g. 20 raster px of 1920 for a 480-px tile); BoxEditor takes it as view and maps boxes and drags to it.
  • POST /config/import checks snapshot crops: a snapshot's DME crop that leaves no picture is refused (409/422) like a keyer's (restore still repairs it).
  • Crop drift on repeated format changes: a crop not changed since an earlier format change is scaled from its value then (its base, kept for the session), e.g. 481 px at 1080p → 321 at 720p → 481 again (was 482); a crop changed in between is scaled from its new value.
  • T-bar edges and source cap: the deadband's edges count (<=/>=, with a 1e-9 tolerance: 1 − 0.98 is 0.020000000000000018 in floats), server and GUI; at most 64 hardware levers (source) per ME are kept for the pickup (the oldest picks up again).

Tests

  • pytest: test_back_and_forth_format_changes_do_not_drift, test_an_import_refuses_a_snapshot_crop_that_leaves_no_picture, test_the_snap_edges_are_inclusive, test_at_most_64_hardware_levers_per_me_are_kept.
  • check-preview.ts: rasterView for a stream, a tile and no map.
  • Docs: README (Pictures, T-bar sources), CHANGELOG.

Independent of the raw-tally PR (both touch the top of CHANGELOG; whichever merges second needs a one-line rebase).

🤖 Generated with Claude Code

https://claude.ai/code/session_01St7QwmonpoG9EEEqaLFXLi

…, import checks snapshot crops, crop drift, T-bar edges and source cap)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01St7QwmonpoG9EEEqaLFXLi
@LeeO86
LeeO86 merged commit 15d35e1 into dev Oct 23, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant