Repository navigation
Remove the debug mode and simplify the detection path - #54
Merged
Merged
Conversation
Drop --debug/--debug-dir, the debug options on DetectorOptions and MatchOptions, the Python debug keyword arguments, and all debug-only output (stderr diagnostics, candidate WAV sections, peak dumps, safe_path_component). The flanking Pearson windows were computed only for debug output, so only the deciding center window is kept. Remove the completed v1 section from the roadmap and its references to the Python implementation. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Format the section timestamp only for the out-of-bounds skip message, store the short-clip flag per clip instead of recomputing it for every candidate, keep the partition similarities in a fixed array, and return the peaks from correlation_method directly now that it cannot fail. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…tion State that there is no debug mode yet instead of narrating its removal, and describe the single center Pearson window the detector uses. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Saves one pass over the section correlation per clip per chunk. Verification already normalizes each candidate slice on its own. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… window Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--debug/--debug-dir, the debug options onDetectorOptionsandMatchOptions, the Pythondebug/debug_dirkeyword arguments, and all debug-only output.Testing
cargo clippy --all-targets -- -D warnings, also with--features pythoncargo testandtests/test_python_bindings.py🤖 Generated with Claude Code