From 105099c819be2180665abdbee6349b9ce8d0eedd Mon Sep 17 00:00:00 2001 From: Holden Date: Sun, 14 Jun 2026 16:46:09 +0000 Subject: [PATCH] fix: mark assets rejected when faces API confidence is below threshold Previously, assets that passed embedding-phase selection but failed the faces API confidence check in execute_jobs were silently skipped with no tracker entry. They appeared as valid candidates on every future run, were re-selected, and re-skipped in an endless cycle. Now they are marked rejected so they are excluded from future runs. RETRY_REJECTED=true clears them if Immich later re-processes the image. --- winnow/executor.py | 1 + 1 file changed, 1 insertion(+) diff --git a/winnow/executor.py b/winnow/executor.py index ac33577..10f852a 100644 --- a/winnow/executor.py +++ b/winnow/executor.py @@ -231,6 +231,7 @@ def execute_jobs(jobs: list[dict]) -> None: f"[yellow]Skipped {asset['id']}" f" (detection confidence {conf:.2f} < {Config.MIN_CONFIDENCE})[/yellow]" ) + mark_rejected(asset["id"], person_name=name) progress.advance(job_task) progress.advance(overall_task) continue