distinguish 'face too small' from 'no face at all' in the log

They need completely different fixes - snapshot resolution and camera distance
versus viewing angle and light - but produced an identical log line. Now reports
the largest face found, the limit it missed, and the snapshot dimensions.

Author: skycryer

CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to FaceID. The Home Assistant app shows this file in the
 update dialog; standalone users can watch GitHub releases.
 
+## 0.6.12 — 2026-07-26
+
+- **"No face found" now says which kind.** The log distinguishes *a face was detected but
+  it is too small* (reporting its pixel width, the `min_face_px` limit and the snapshot
+  dimensions) from *no face at all* — two very different problems. Too small points at
+  Frigate's `snapshots.height` or camera distance; none at all points at viewing angle or
+  light. Previously both produced the same line, leaving nowhere to start.
+
 ## 0.6.11 — 2026-07-26
 
 - **New LOG tab.** The service log is now visible in the web UI — the last 500 lines,

app/mqtt_listener.py

@@ -161,12 +161,23 @@ def _process(self, eid: str):
         if img is None:
             log.info("Event %s (%s): kein Snapshot von Frigate", eid, st["camera"])
             return
-        face = FaceEngine.best_face(self.engine.faces(img), min_px=self.min_face_px)
+        found = self.engine.faces(img)
+        face = FaceEngine.best_face(found, min_px=self.min_face_px)
         if face is None:
-            # Der haeufigste Normalfall (Ruecken zur Kamera, zu weit weg) — trotzdem
-            # protokollieren, sonst sieht ein stiller Log wie ein Defekt aus.
-            log.info("Event %s (%s): Versuch %d, kein Gesicht >= %dpx im Snapshot",
-                     eid, st["camera"], st["attempts"], self.min_face_px)
+            # Haeufigster Normalfall (Ruecken zur Kamera, zu weit weg) — trotzdem
+            # protokollieren, sonst sieht ein stiller Log wie ein Defekt aus. Die
+            # Unterscheidung "zu klein" vs. "gar keins" entscheidet, wo man sucht:
+            # zu klein deutet auf Frigates snapshots.height oder Kameraabstand,
+            # gar keins eher auf Blickwinkel oder Licht.
+            h, w = img.shape[:2]
+            if found:
+                big = max(int(f.bbox[2] - f.bbox[0]) for f in found)
+                log.info("Event %s (%s): Versuch %d, groesstes Gesicht %dpx < min_face_px "
+                         "%d (Snapshot %dx%d)", eid, st["camera"], st["attempts"], big,
+                         self.min_face_px, w, h)
+            else:
+                log.info("Event %s (%s): Versuch %d, kein Gesicht im Snapshot %dx%d",
+                         eid, st["camera"], st["attempts"], w, h)
             return
         emb = face.normed_embedding
         slug, name, score = self.gallery.match(emb)

faceid-addon/CHANGELOG.md

@@ -3,6 +3,14 @@
 All notable changes to FaceID. The Home Assistant app shows this file in the
 update dialog; standalone users can watch GitHub releases.
 
+## 0.6.12 — 2026-07-26
+
+- **"No face found" now says which kind.** The log distinguishes *a face was detected but
+  it is too small* (reporting its pixel width, the `min_face_px` limit and the snapshot
+  dimensions) from *no face at all* — two very different problems. Too small points at
+  Frigate's `snapshots.height` or camera distance; none at all points at viewing angle or
+  light. Previously both produced the same line, leaving nowhere to start.
+
 ## 0.6.11 — 2026-07-26
 
 - **New LOG tab.** The service log is now visible in the web UI — the last 500 lines,

faceid-addon/app/mqtt_listener.py

@@ -161,12 +161,23 @@ def _process(self, eid: str):
         if img is None:
             log.info("Event %s (%s): kein Snapshot von Frigate", eid, st["camera"])
             return
-        face = FaceEngine.best_face(self.engine.faces(img), min_px=self.min_face_px)
+        found = self.engine.faces(img)
+        face = FaceEngine.best_face(found, min_px=self.min_face_px)
         if face is None:
-            # Der haeufigste Normalfall (Ruecken zur Kamera, zu weit weg) — trotzdem
-            # protokollieren, sonst sieht ein stiller Log wie ein Defekt aus.
-            log.info("Event %s (%s): Versuch %d, kein Gesicht >= %dpx im Snapshot",
-                     eid, st["camera"], st["attempts"], self.min_face_px)
+            # Haeufigster Normalfall (Ruecken zur Kamera, zu weit weg) — trotzdem
+            # protokollieren, sonst sieht ein stiller Log wie ein Defekt aus. Die
+            # Unterscheidung "zu klein" vs. "gar keins" entscheidet, wo man sucht:
+            # zu klein deutet auf Frigates snapshots.height oder Kameraabstand,
+            # gar keins eher auf Blickwinkel oder Licht.
+            h, w = img.shape[:2]
+            if found:
+                big = max(int(f.bbox[2] - f.bbox[0]) for f in found)
+                log.info("Event %s (%s): Versuch %d, groesstes Gesicht %dpx < min_face_px "
+                         "%d (Snapshot %dx%d)", eid, st["camera"], st["attempts"], big,
+                         self.min_face_px, w, h)
+            else:
+                log.info("Event %s (%s): Versuch %d, kein Gesicht im Snapshot %dx%d",
+                         eid, st["camera"], st["attempts"], w, h)
             return
         emb = face.normed_embedding
         slug, name, score = self.gallery.match(emb)

faceid-addon/config.yaml

@@ -1,5 +1,5 @@
 name: FaceID
-version: "0.6.11"
+version: "0.6.12"
 slug: faceid
 description: Face recognition for Frigate — trainable gallery, clustered unknown review, HA sensors
 url: https://github.com/SkyTechNerds/faceid