feature/face-stats-exclude-videos #64

cameron · 2026-04-30T21:17:15Z

cameron commented

2026-04-30 21:17:15 +00:00

No description provided.

cameron added 2 commits 2026-04-30 21:17:16 +00:00

faces: exclude videos from backlog drain and SCANNED denominator 0840d55c70

list_unscanned_candidates pulled every hashed image_exif row, including
videos. filter_excluded then dropped them client-side without writing a
marker, so the same set re-appeared every watcher tick — emitting the
"backlog drain — running detection on N candidate(s)" log forever and
producing no progress.

face_stats.total_photos counted the same video rows in the denominator,
so the SCANNED percentage was structurally capped below 100%.

Add an image-extension SQL predicate (case-insensitive, sourced from
file_types::IMAGE_EXTENSIONS) and apply it to both queries. Videos
never enter the candidate set, total_photos counts only what can
actually be scanned, and 100% becomes reachable.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

style: cargo fmt 67abd8d8ff

Pre-existing whitespace drift in test bodies, normalized by rustfmt.
No behavior change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

cameron merged commit d0833177c7 into master

2026-04-30 21:17:20 +00:00

cameron deleted branch feature/face-stats-exclude-videos

2026-04-30 21:17:21 +00:00

cameron referenced this issue from a commit

2026-04-30 21:17:22 +00:00

Merge pull request 'feature/face-stats-exclude-videos' (#64) from feature/face-stats-exclude-videos into master

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Apps/ImageApi#64