Add debug info for autoreload.

Carreau · Carreau · commit d791458a5435 · 2026-02-02T10:18:33.000+01:00
See issue #15116
diff --git a/IPython/extensions/autoreload.py b/IPython/extensions/autoreload.py
@@ -327,6 +327,10 @@ def check(self, check_all=False, do_reload=True, execution_info=None):
                         del self.failed[py_filename]
                 except:
                     if not self.hide_errors:
+                        logger = logging.getLogger("autoreload")
+                        logger.exception(
+                            f"Failed to reload module '{modname}' from file '{py_filename}'"
+                        )
                         print(
                             "[autoreload of {} failed: {}]".format(
                                 modname, traceback.format_exc(10)
diff --git a/IPython/extensions/deduperreload/deduperreload.py b/IPython/extensions/deduperreload/deduperreload.py
@@ -3,6 +3,7 @@
 import builtins
 import contextlib
 import itertools
+import logging
 import os
 import pickle
 import platform
@@ -209,11 +210,19 @@ def update_sources(self) -> None:
             ):
                 self.source_by_modname[new_modname] = ""
                 continue
-            with open(fname, "r") as f:
-                try:
+            # Skip binary files (e.g., .so, .pyd, .pyo)
+            if not fname.endswith(".py"):
+                self.source_by_modname[new_modname] = ""
+                continue
+            try:
+                with open(fname, "r") as f:
                     self.source_by_modname[new_modname] = f.read()
-                except Exception:
-                    self.source_by_modname[new_modname] = ""
+            except Exception as e:
+                logger = logging.getLogger("autoreload")
+                logger.exception(
+                    f"Failed to read module file '{fname}' for module '{new_modname}': {type(e).__name__}"
+                )
+                self.source_by_modname[new_modname] = ""
 
     constexpr_detector = ConstexprDetector()
 
@@ -542,8 +551,15 @@ def maybe_reload_module(self, module: ModuleType) -> bool:
             return False
         if (fname := get_module_file_name(module)) is None:
             return False
-        with open(fname, "r") as f:
-            new_source_code = f.read()
+        try:
+            with open(fname, "r") as f:
+                new_source_code = f.read()
+        except Exception as e:
+            logger = logging.getLogger("autoreload")
+            logger.exception(
+                f"Failed to read module file '{fname}' for module '{modname}': {type(e).__name__}"
+            )
+            return False
         patched_flag = False
         if old_source_code := self.source_by_modname.get(modname):
             # get old/new module ast
