From 7b7287c9b916aec2dd33d1b66031895947a58f75 Mon Sep 17 00:00:00 2001
From: Rachel Li <lirachel@mit.edu>
Date: Thu, 8 Jan 2026 19:20:24 -0500
Subject: [PATCH] Fix telemetry ping regression for explicit statistics

Fixed Codex regression: keep snapshot_download pings for explicit statistics values; detection only runs when statistics is None. Also replaced bare except.
---
 unsloth/models/_utils.py | 62 ++++++++++++++++++++--------------------
 1 file changed, 31 insertions(+), 31 deletions(-)

diff --git a/unsloth/models/_utils.py b/unsloth/models/_utils.py
index 701e6f633c..0cb6c8975e 100644
--- a/unsloth/models/_utils.py
+++ b/unsloth/models/_utils.py
@@ -1170,40 +1170,40 @@ def _get_statistics(statistics = None, force_download = True):
                 except:
                     statistics = "other"
 
-        if statistics is not None:
-            import tempfile
-            from huggingface_hub import snapshot_download
-            from unsloth_zoo.rl_environments import execute_with_time_limit
+    if statistics is not None:
+        import tempfile
+        from huggingface_hub import snapshot_download
+        from unsloth_zoo.rl_environments import execute_with_time_limit
 
-            if has_internet():
+        if has_internet():
 
-                def stats_check():
-                    with tempfile.TemporaryDirectory(ignore_cleanup_errors = True) as f:
-                        snapshot_download(
-                            f"unslothai/{statistics}",
-                            force_download = True,
-                            cache_dir = f,
-                            local_dir = f,
-                        )
-
-                time_limited_stats_check = execute_with_time_limit(120)(stats_check)
-                try:
-                    time_limited_stats_check()
-                except TimeoutError:
-                    raise TimeoutError(
-                        "Unsloth: HuggingFace seems to be down after trying for 120 seconds :(\n"
-                        "Check https://status.huggingface.co/ for more details.\n"
-                        "As a temporary measure, use modelscope with the same model name ie:\n"
-                        "```\n"
-                        "pip install modelscope\n"
-                        "import os; os.environ['UNSLOTH_USE_MODELSCOPE'] = '1'\n"
-                        "from unsloth import FastLanguageModel\n"
-                        "model = FastLanguageModel.from_pretrained('unsloth/gpt-oss-20b')\n"
-                        "```"
+            def stats_check():
+                with tempfile.TemporaryDirectory(ignore_cleanup_errors = True) as f:
+                    snapshot_download(
+                        f"unslothai/{statistics}",
+                        force_download = True,
+                        cache_dir = f,
+                        local_dir = f,
                     )
-                except:
-                    # Try no time limit check
-                    stats_check()
+
+            time_limited_stats_check = execute_with_time_limit(120)(stats_check)
+            try:
+                time_limited_stats_check()
+            except TimeoutError:
+                raise TimeoutError(
+                    "Unsloth: HuggingFace seems to be down after trying for 120 seconds :(\n"
+                    "Check https://status.huggingface.co/ for more details.\n"
+                    "As a temporary measure, use modelscope with the same model name ie:\n"
+                    "```\n"
+                    "pip install modelscope\n"
+                    "import os; os.environ['UNSLOTH_USE_MODELSCOPE'] = '1'\n"
+                    "from unsloth import FastLanguageModel\n"
+                    "model = FastLanguageModel.from_pretrained('unsloth/gpt-oss-20b')\n"
+                    "```"
+                )
+            except Exception:
+                # Try no time limit check
+                stats_check()
 
 
 def get_statistics(local_files_only = False):