Update app.py

app.py

@@ -138,6 +138,15 @@ def _download_one(repo_id: str, tries: int = 3) -> bool:
     """
     cache_home = os.path.expanduser(os.getenv("HF_HOME", "/home/user/.cache/huggingface"))
     local_dir = os.path.join(cache_home, "models", repo_id.replace("/", "__"))
+    # Déjà présent localement ? On log et on saute proprement.
+    try:
+        if _is_repo_cached(repo_id):
+            _log_warmup(f"Déjà en cache: {repo_id} (skip)")
+            return True
+    except Exception:
+        # En cas d'erreur d'audit, on continue le process normal
+        pass
+
     for attempt in range(1, tries + 1):
         if warmup_stop.is_set():
             return False
@@ -185,6 +194,9 @@ def _warmup_thread(models: List[str]):
                 warmup_state["ok_count"] = ok_count
         else:
             _log_warmup(f"ÉCHEC: {repo}")
+        # Met à jour la progression globale après ce repo (ok ou échec)
+        with warmup_lock:
+            warmup_state["percent"] = int(((i + 1) / max(1, len(models))) * 100)
 
     with warmup_lock:
         warmup_state["percent"] = 100
@@ -487,6 +499,10 @@ async def warmup_start(payload: Optional[Dict[str, Any]] = Body(None)):
         models = [str(x).strip() for x in (payload.get("models") or []) if str(x).strip()]
     if not models:
         models = _default_model_list()
+    # Dé-doublonnage en conservant l'ordre
+    seen = set()
+    models = [m for m in models if not (m in seen or seen.add(m))]
+
     if not models:
         raise HTTPException(400, "Aucun modèle fourni (payload.models) et WARMUP_MODELS vide")
 
@@ -962,13 +978,15 @@ async function refreshWarmupUI(){
     if (warmupStatusEl) {
   const tot = (s.total ?? 0);
   if (running) {
-
-
-
-    const idx = (s.idx ?? 0) + 1;
-        warmupStatusEl.textContent = `⏳ ${pct}% — ${s.current||''} (${idx}/${tot}) [inst:${instanceId}]`;
   } else {
-        // Si le run vient de se terminer et que le backend expose ok_count, on priorise ce résumé
+    // Détection fin de run (par job_id) => toast récap une seule fois
+    if (s.done && s.job_id && window._lastNotifiedJobId !== s.job_id) {
+      const tot2 = (s.total ?? 0);
+      const ok2  = Number.isFinite(s.ok_count) ? s.ok_count : '—';
+      showToast(`Warm-up terminé — ${ok2}/${tot2} téléchargés (inst:${instanceId})`);
+      window._lastNotifiedJobId = s.job_id;
+    }
+
     if (s.done && Number.isFinite(s.ok_count)) {
       warmupStatusEl.textContent = `✅ Terminé — ${s.ok_count}/${tot} téléchargés (inst:${instanceId})`;
     } else {
@@ -989,6 +1007,7 @@ async function refreshWarmupUI(){
     if (warmupPopupStatus)  warmupPopupStatus.textContent = running ? 'Téléchargement en cours…' : 'Terminé';
 
     // PRÉAMBULE FIXE (conservé pendant tout le run)
+        // (Auto-scroll sur chaque rafraîchissement)
     const logsTxt = Array.isArray(s.logs) ? s.logs.join('\n') : '';
     const fixedPreface = (warmupPreface && warmupPreface.trim().length)
       ? warmupPreface
@@ -1006,8 +1025,10 @@ async function refreshWarmupUI(){
             asked.map(m => ' • ' + m).join('\n')
           );
         })();
-    if (warmupLogs) warmupLogs.textContent = fixedPreface + '\n\n' + logsTxt;
-
+    if (warmupLogs) {
+      warmupLogs.textContent = fixedPreface + '\n\n' + logsTxt;
+      warmupLogs.scrollTop = warmupLogs.scrollHeight;
+    }
 
     if (warmupStopBtn) warmupStopBtn.style.display = running ? 'inline-block' : 'none';