feature/video-frame-rate #98

cameron · 2026-05-18T00:09:30Z

cameron commented

2026-05-18 00:09:30 +00:00

No description provided.

cameron added 3 commits 2026-05-18 00:09:30 +00:00

video: probe frame rate via ffprobe and return on /video/generate 1b70a6f0b4

Adds frame_rate to GenerateVideoResponse so the mobile scrubber can step
at the source's real fps instead of a hardcoded 30. probe_video_stream_meta
gains a frame_rate field (avg_frame_rate preferred, r_frame_rate fallback,
nonsense values rejected) and is now pub so the handler can reuse it.
Cost is one ffprobe per /video/generate call; degrades silently to None
on probe failure.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

chore: add .gitattributes + unit tests for ffprobe rational parser bd61e10158

LF normalization across OSes; *.sql pinned to LF for stable diffs.

Tests cover the rational frame-rate parser (NTSC 29.97, integer fps,
slow-mo 240, ffprobe's 0/0 unknown sentinel, malformed and out-of-range
inputs). Extracted the closure into a free fn for the test seam.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

cargo fmt: drop trailing blank line in actors.rs acdffc1558

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

cameron merged commit d275150db6 into master

2026-05-18 00:09:36 +00:00

cameron deleted branch feature/video-frame-rate

2026-05-18 00:09:36 +00:00

cameron referenced this issue from a commit

2026-05-18 00:09:37 +00:00

Merge pull request 'feature/video-frame-rate' (#98) from feature/video-frame-rate into master

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Apps/ImageApi#98