Commit ·
5134951
1
Parent(s): 4b9e6f7
Add step-by-step numbered logging for HF debugging
Browse files- core/module_registry.py +10 -3
core/module_registry.py
CHANGED
|
@@ -76,19 +76,24 @@ class ModuleRegistry:
|
|
| 76 |
|
| 77 |
def load_module(self, module_name: str, modules_package: str = "modules") -> Optional[ModuleInfo]:
|
| 78 |
"""Load a single module and return its info"""
|
|
|
|
| 79 |
try:
|
| 80 |
full_module_name = f"{modules_package}.{module_name}"
|
|
|
|
| 81 |
module = importlib.import_module(full_module_name)
|
|
|
|
| 82 |
|
| 83 |
# Check required attributes
|
| 84 |
if not hasattr(module, 'register'):
|
| 85 |
-
logger.warning(f"Module {module_name} has no register function, skipping")
|
| 86 |
return None
|
|
|
|
| 87 |
|
| 88 |
# Get module metadata
|
| 89 |
name = getattr(module, 'MODULE_NAME', module_name)
|
| 90 |
prefix = getattr(module, 'MODULE_PREFIX', f"/api/{module_name}")
|
| 91 |
description = getattr(module, 'MODULE_DESCRIPTION', "")
|
|
|
|
| 92 |
|
| 93 |
info = ModuleInfo(
|
| 94 |
name=name,
|
|
@@ -98,11 +103,13 @@ class ModuleRegistry:
|
|
| 98 |
)
|
| 99 |
|
| 100 |
self._modules[name] = info
|
| 101 |
-
logger.info(f"Loaded module: {name} (prefix: {prefix})")
|
| 102 |
return info
|
| 103 |
|
| 104 |
except Exception as e:
|
| 105 |
-
logger.error(f"Failed to load module {module_name}: {e}")
|
|
|
|
|
|
|
| 106 |
return None
|
| 107 |
|
| 108 |
def register_all(self, app: FastAPI, config: Any) -> int:
|
|
|
|
| 76 |
|
| 77 |
def load_module(self, module_name: str, modules_package: str = "modules") -> Optional[ModuleInfo]:
|
| 78 |
"""Load a single module and return its info"""
|
| 79 |
+
logger.info(f"[STEP 4] Loading module: {module_name}")
|
| 80 |
try:
|
| 81 |
full_module_name = f"{modules_package}.{module_name}"
|
| 82 |
+
logger.info(f"[STEP 4.1] Importing: {full_module_name}")
|
| 83 |
module = importlib.import_module(full_module_name)
|
| 84 |
+
logger.info(f"[STEP 4.2] Import successful: {full_module_name}")
|
| 85 |
|
| 86 |
# Check required attributes
|
| 87 |
if not hasattr(module, 'register'):
|
| 88 |
+
logger.warning(f"[STEP 4.3] Module {module_name} has no register function, skipping")
|
| 89 |
return None
|
| 90 |
+
logger.info(f"[STEP 4.3] register() function found")
|
| 91 |
|
| 92 |
# Get module metadata
|
| 93 |
name = getattr(module, 'MODULE_NAME', module_name)
|
| 94 |
prefix = getattr(module, 'MODULE_PREFIX', f"/api/{module_name}")
|
| 95 |
description = getattr(module, 'MODULE_DESCRIPTION', "")
|
| 96 |
+
logger.info(f"[STEP 4.4] Module metadata: name={name}, prefix={prefix}")
|
| 97 |
|
| 98 |
info = ModuleInfo(
|
| 99 |
name=name,
|
|
|
|
| 103 |
)
|
| 104 |
|
| 105 |
self._modules[name] = info
|
| 106 |
+
logger.info(f"[STEP 4.5] Loaded module: {name} (prefix: {prefix})")
|
| 107 |
return info
|
| 108 |
|
| 109 |
except Exception as e:
|
| 110 |
+
logger.error(f"[STEP 4.ERROR] Failed to load module {module_name}: {e}")
|
| 111 |
+
import traceback
|
| 112 |
+
logger.error(f"[STEP 4.TRACEBACK]\n{traceback.format_exc()}")
|
| 113 |
return None
|
| 114 |
|
| 115 |
def register_all(self, app: FastAPI, config: Any) -> int:
|