Skip to content

Refuse non-finite silence settings before ffmpeg runs - #204

Merged
jmjava merged 1 commit into
mainfrom
cursor/silence-settings-finite
Nov 1, 2026
Merged

jmjava merged 1 commit into
mainfrom
cursor/silence-settings-finite

Conversation

@jmjava

@jmjava jmjava commented Nov 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • A non-finite silence noise floor or a non-positive minimum silence used to be passed to ffmpeg.
  • Those settings are now refused before silencedetect runs, and a unit test fails if that check is removed.

Test plan

  • tests/test_align.py refuses infinite minimum silence and NaN noise without calling ffmpeg
  • complexity and hotspot gates pass against origin/main

Made with Cursor

An infinite minimum silence or a non-finite noise floor used to be passed to silencedetect.
@jmjava
jmjava merged commit 378a43c into main Nov 1, 2026
8 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