Skip to content

https://github.com/LuisMRaimundo/SoundSpectrAnalyse/compare/main...phase12/harmonic-peak-validation-edge-contracts?expand=1#62

Merged
LuisMRaimundo merged 1 commit into
mainfrom
phase12/harmonic-peak-validation-edge-contracts
Jun 13, 2026
Merged

https://github.com/LuisMRaimundo/SoundSpectrAnalyse/compare/main...phase12/harmonic-peak-validation-edge-contracts?expand=1#62
LuisMRaimundo merged 1 commit into
mainfrom
phase12/harmonic-peak-validation-edge-contracts

Conversation

@LuisMRaimundo

Copy link
Copy Markdown
Owner

Pin CFAR boundary branches, strict-vs-diagnostic classification fallbacks, malformed refinement inputs, and non-mutation guards for Adds a Phase 12 harmonic_peak_validation edge-contract regression layer.

New file:
tests/phase_12/test_harmonic_peak_validation_edge_contract_additional.py

Adds 23 helper-level regression tests covering:

  • CFAR edge/short-array behaviour
  • strict harmonic candidate classification
  • CFAR vs SNR-only validation paths
  • local peak validation
  • harmonic inclusion audit exclusion reasons
  • interpolation/refinement fallback schema
  • deterministic/non-mutating helper behaviour

Validation:

  • 23 passed for the new file
  • 1235 passed, 3 skipped for the full suite
  • harmonic_peak_validation.py coverage: 86.0% → 89.9%

Production code untouched.harmonic_peak_validation.py.

Pin CFAR boundary branches, strict-vs-diagnostic classification fallbacks, malformed refinement inputs, and non-mutation guards for harmonic_peak_validation.py.

Co-authored-by: Cursor <cursoragent@cursor.com>
@LuisMRaimundo LuisMRaimundo merged commit a53ecab into main Jun 13, 2026
3 of 4 checks passed
@LuisMRaimundo LuisMRaimundo deleted the phase12/harmonic-peak-validation-edge-contracts branch June 13, 2026 14:17
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.

2 participants