Skip to content

Preserve source video FPS in CLI track output#253

Merged
SkalskiP merged 1 commit intodevelopfrom
fix/output-video-fps
Feb 15, 2026
Merged

Preserve source video FPS in CLI track output#253
SkalskiP merged 1 commit intodevelopfrom
fix/output-video-fps

Conversation

@SkalskiP
Copy link
Collaborator

Summary

  • Output videos were always written at 30 FPS regardless of the source frame rate. Now _VideoOutput receives the actual source FPS from _classify_source and falls back to _DEFAULT_OUTPUT_FPS (30.0) for sources without a known frame rate (e.g. image directories).
  • Extracted _DEFAULT_OUTPUT_FPS constant to eliminate the magic number.
  • Cleaned up video.py: renamed h, w to height, width, added return type annotations to __enter__/__exit__, removed forbidden module-level docstring.

@SkalskiP SkalskiP merged commit 88f9dd7 into develop Feb 15, 2026
15 checks passed
@SkalskiP SkalskiP deleted the fix/output-video-fps branch February 15, 2026 15:14
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