Upload folder using huggingface_hub

src/ring_size.py

@@ -32,6 +32,12 @@ DEFAULT_RING_MODEL = "gen"
 # Backwards-compatible alias
 RING_SIZE_CHART = RING_MODELS[DEFAULT_RING_MODEL]
 
+# Diameter bounds for the ring-size lookup. Below MIN or above MAX,
+# recommend_ring_size() returns None and aggregate_ring_sizes() tags the
+# per-finger row as failed with diameter_below_chart_min / diameter_above_chart_max.
+MIN_DIAMETER_MM = 12.0
+MAX_DIAMETER_MM = 26.0
+
 
 def _get_sorted_sizes(ring_model: str) -> List[Tuple[int, float]]:
     chart = RING_MODELS.get(ring_model, RING_MODELS[DEFAULT_RING_MODEL])
@@ -51,7 +57,7 @@ def recommend_ring_size(diameter_cm: float, ring_model: str = DEFAULT_RING_MODEL
     """
     diameter_mm = diameter_cm * 10.0
 
-    if diameter_mm < 14.0 or diameter_mm > 26.0:
+    if diameter_mm < MIN_DIAMETER_MM or diameter_mm > MAX_DIAMETER_MM:
         return None
 
     sorted_sizes = _get_sorted_sizes(ring_model)
@@ -105,15 +111,32 @@ def aggregate_ring_sizes(per_finger_results: Dict[str, Dict]) -> Dict:
     # Build per_finger summary
     per_finger: Dict[str, Dict] = {}
     for name, result in per_finger_results.items():
-        failed = result.get("fail_reason") is not None or result.get("ring_size") is None
         rs = result.get("ring_size")
+        upstream_reason = result.get("fail_reason")
+        failed = upstream_reason is not None or rs is None
+
+        # If the measurement succeeded but ring_size is None, the diameter
+        # fell outside the chart bounds — surface that as a fail_reason so
+        # the dashboard does not show "failed, reason unknown".
+        derived_reason = upstream_reason
+        if derived_reason is None and rs is None:
+            diameter_cm = result.get("finger_outer_diameter_cm")
+            if diameter_cm is None:
+                derived_reason = "ring_size_lookup_failed"
+            elif diameter_cm * 10.0 < MIN_DIAMETER_MM:
+                derived_reason = "diameter_below_chart_min"
+            elif diameter_cm * 10.0 > MAX_DIAMETER_MM:
+                derived_reason = "diameter_above_chart_max"
+            else:
+                derived_reason = "ring_size_lookup_failed"
+
         per_finger[name] = {
             "diameter_cm": result.get("finger_outer_diameter_cm"),
             "confidence": result.get("confidence", 0.0),
             "best_match": rs["best_match"] if rs else None,
             "range": [rs["range_min"], rs["range_max"]] if rs else None,
             "status": "failed" if failed else "ok",
-            "fail_reason": result.get("fail_reason"),
+            "fail_reason": derived_reason,
         }
 
     # Filter to succeeded fingers

web_demo/app.py

@@ -556,9 +556,19 @@ def _compute_stats(rows: List[Dict[str, Any]], days: int = 30) -> Dict[str, Any]
         if m:
             mode_counter[m] += 1
 
-        size = row.get("overall_best_size")
-        if size not in (None, ""):
-            size_counter[str(size)] += 1
+        # Ring size distribution: index-finger size only, and exclude demo
+        # runs (photo_url is null when the user clicks "try with sample" —
+        # those measurements aren't the KOL's true size).
+        if row.get("photo_url"):
+            index_size = None
+            if row.get("mode") == "multi":
+                pf = (row.get("per_finger") or {}).get("index") or {}
+                if pf.get("status") == "ok":
+                    index_size = pf.get("best_match")
+            elif row.get("finger_index") == "index":
+                index_size = row.get("overall_best_size")
+            if index_size not in (None, ""):
+                size_counter[str(index_size)] += 1
 
         conf = row.get("confidence")
         if isinstance(conf, (int, float)):

web_demo/supabase_client.py

@@ -122,15 +122,17 @@ def list_measurements(limit: int = 200, offset: int = 0) -> List[Dict[str, Any]]
 
 STATS_COLUMNS = (
     "id,created_at,kol_name,mode,ring_model,confidence,fail_reason,"
-    "overall_best_size,ring_fit,gt_index_size,gt_middle_size,gt_ring_size"
+    "overall_best_size,ring_fit,gt_index_size,gt_middle_size,gt_ring_size,"
+    "finger_index,photo_url,per_finger"
 )
 
 
 def list_measurements_for_stats(limit: int = 5000) -> List[Dict[str, Any]]:
     """Lightweight projection used by the admin stats endpoint.
 
-    Skips heavy columns (`result_json`, `per_finger`, photo URLs) so we can
-    pull thousands of rows for aggregation without paying the JSON-blob cost.
+    Skips `result_json` (the heaviest blob). `per_finger` is included so the
+    size-distribution chart can use the index-finger size and detect demo
+    runs (`photo_url` is null when the user clicks "try with sample").
     """
     client = _get_client()
     if client is None:

web_demo/templates/admin.html

@@ -211,7 +211,7 @@
           <div class="chart-wrap"><canvas id="chartModels"></canvas></div>
         </div>
         <div class="chart-card">
-          <h2>Ring Size Distribution</h2>
+          <h2>Ring Size Distribution (Index, real uploads)</h2>
           <div class="chart-wrap"><canvas id="chartSizes"></canvas></div>
         </div>
         <div class="chart-card">