flan/winnow
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix TypeError when worst mapped file is None in quality replacement

Browse Source 
When no mapped files exist for a person, worst is None and the old
f-string tried to subscript it before the conditional was evaluated.
Extracted worst_score_str as a local variable to avoid the crash.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
flan
2026-06-13 05:14:15 +00:00
co-authored by Claude Sonnet 4.6
parent 11e7d4aad7
commit 12bccaa631
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
winnow/executor.py
+2 -1
View File
@@ -338,9 +338,10 @@ def upload_to_frigate(jobs: list[dict]) -> None:
continue continue
worst = get_lowest_quality_mapped_file(name) worst = get_lowest_quality_mapped_file(name)
if worst is None or new_score <= worst[2]: if worst is None or new_score <= worst[2]:
worst_score_str = f"{worst[2]:.3f}" if worst is not None else "N/A"
progress.console.print( progress.console.print(
f" [dim]⏭ {fname}: score {new_score:.3f} ≤ worst mapped" f" [dim]⏭ {fname}: score {new_score:.3f} ≤ worst mapped"
f" {worst[2]:.3f if worst else 'N/A'}, skipping[/dim]" f" {worst_score_str}, skipping[/dim]"
) )
progress.advance(upload_task) progress.advance(upload_task)
continue continue