Skip to content

feat: détection multi-pics pour scan_templates (seuil cc, ondes S optionnelles)#10

Open
MariusYvard wants to merge 1 commit into
SeismicSource:mainfrom
MariusYvard:feature/swave-ncc
Open

feat: détection multi-pics pour scan_templates (seuil cc, ondes S optionnelles)#10
MariusYvard wants to merge 1 commit into
SeismicSource:mainfrom
MariusYvard:feature/swave-ncc

Conversation

@MariusYvard

@MariusYvard MariusYvard commented Jun 28, 2026

Copy link
Copy Markdown

Salut Claudio,

Le cœur du stage. scan_templates passe d'un pic unique par chunk (déclenché par le ratio cc/MAD) à une corrélation glissante correlate_template + find_peaks au seuil template_cc_min : plusieurs événements détectables par chunk. La MAD (min_cc_mad_ratio) est retirée, comme tu le souhaitais.

Inclus aussi : fusion des pics trop proches (t_min), interpolation sous-échantillon du pic, décimation decim_factor avec anti-repliement pour la vitesse, passe-bande zéro-phase. En option (off par défaut) : une seconde corrélation centrée sur les ondes S (NCCs) et un critère combiné entièrement paramétrable, avec une colonne ccs ajoutée à template_detections (migration en place).

Point de nommage à trancher : j'ai introduit template_cc_min car cc_min existe déjà (0.95, similarité des paires du clustering). Dis-moi si tu préfères réutiliser cc_min.

Tests unitaires (dont un test d'injection synthétique) et une note de réglage des performances ajoutés.

Marius

…d and optional S-wave criterion

Replace the single best peak and the MAD-ratio trigger (removed) with a sliding correlate_template + find_peaks at template_cc_min, so several events can be detected per chunk; add t_min spacing, sub-sample interpolation, decim_factor, zero-phase filtering, and an optional S-wave NCCs combined criterion with a ccs column.
@MariusYvard MariusYvard changed the title feat: detection multi-pics pour scan_templates (seuil cc, ondes S optionnelles) feat: détection multi-pics pour scan_templates (seuil cc, ondes S optionnelles) Jun 28, 2026
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