floppyrj45/cosma-qc
2
0
Fork 0
Code Issues 2 Pull Requests 5 Actions Packages Projects Releases Wiki Activity

fix(04/04b): load QC_BOTTOM_OK_PCT from thresholds.yaml #9

Open
poulpe wants to merge 2 commits from fix/threshold-yaml-source into feature/auto-pipeline
pull from: fix/threshold-yaml-source
merge into: floppyrj45:feature/auto-pipeline
Conversation 0 Commits 2 Files Changed 4 +83 -2

2 Commits

Author SHA1 Message Date
Poulpe
2b0c4dc06b auto-iter 2026-05-12: log iter-3 + veille 2026-05-12 10:39:48 +00:00
Poulpe
610b3a218b fix(stages 04/04b): load QC_BOTTOM_OK_PCT from thresholds.yaml (fallback env/hardcoded) 
Iter-1 patch (thresholds.yaml bottom_visible_pct_min 50→30) had zero effect:
04_frame_extract.py and 04b_trim_water.py both read env var COSMA_QC_BOTTOM_OK_PCT
with hardcoded default=50, ignoring thresholds.yaml entirely.

Add _load_bottom_ok_pct() loader in both stages: reads thresholds.yaml first,
falls back to COSMA_QC_BOTTOM_OK_PCT env var, then hardcoded 50.

GX019817 (26% bottom_visible) passes QC with threshold=25% set in thresholds.yaml.
 2026-05-12 10:36:22 +00:00