fixed: potential error by extra_model_paths.yaml

https://github.com/ltdrdata/ComfyUI-Impact-Subpack/issues/27

__init__.py

@@ -8,7 +8,7 @@
 import importlib
 import logging
 
-version_code = [1, 2]
+version_code = [1, 2, 8]
 version_str = f"V{version_code[0]}.{version_code[1]}" + (f'.{version_code[2]}' if len(version_code) > 2 else '')
 logging.info(f"### Loading: ComfyUI-Impact-Subpack ({version_str})")
 

modules/subcore.py

@@ -37,6 +37,18 @@ def create_segmasks(results):
     return results
 
 
+# Limit the commands that can be executed through `getattr` to `ultralytics.nn.modules.head.Detect.forward`.
+def restricted_getattr(obj, name, *args):
+    if name != "forward":
+        logging.error(f"Access to potentially dangerous attribute '{obj.__module__}.{obj.__name__}.{name}' is blocked.\nIf you believe the use of this code is genuinely safe, please report it.\nhttps://github.com/ltdrdata/ComfyUI-Impact-Subpack/issues")
+        raise RuntimeError(f"Access to potentially dangerous attribute '{obj.__module__}.{obj.__name__}.{name}' is blocked.")
+        
+    return getattr(obj, name, *args)
+
+restricted_getattr.__module__ = 'builtins'
+restricted_getattr.__name__ = 'getattr'
+
+
 try:
     from ultralytics import YOLO
     from ultralytics.nn.tasks import DetectionModel
@@ -49,8 +61,13 @@ try:
     import ultralytics.utils.loss as loss_modules
     import dill._dill
     from numpy.core.multiarray import scalar
-    from numpy import dtype
-    from numpy.dtypes import Float64DType
+    try:
+        from numpy import dtype
+        from numpy.dtypes import Float64DType
+    except:
+        logging.error("[Impact Subpack] installed 'numpy' is outdated. Please update 'numpy' to 1.26.4")
+        raise Exception("[Impact Subpack] installed 'numpy' is outdated. Please update 'numpy' to 1.26.4")
+
 
     torch_whitelist = []
 
@@ -108,7 +125,7 @@ try:
 
         torch_whitelist += [DetectionModel, aliasYOLOv10DetectionModel, SegmentationModel, IterableSimpleNamespace,
                             aliasIterableSimpleNamespace, TaskAlignedAssigner, aliasTaskAlignedAssigner, aliasv10DetectLoss,
-                            getattr, dill._dill._load_type, scalar, dtype, Float64DType]
+                            restricted_getattr, dill._dill._load_type, scalar, dtype, Float64DType]
 
     build_torch_whitelist()
 
@@ -120,18 +137,43 @@ except Exception as e:
 # HOTFIX: https://github.com/ltdrdata/ComfyUI-Impact-Pack/issues/754
 # importing YOLO breaking original torch.load capabilities
 def torch_wrapper(*args, **kwargs):
-    return orig_torch_load(*args, **kwargs)
+    # NOTE: A trick to support code based on `'weights_only' in torch.load.__code__.co_varnames`.
+    if 'weights_only' in kwargs:
+        weights_only = kwargs.pop('weights_only')
+    else:
+        weights_only = None
+
+    if hasattr(torch.serialization, 'safe_globals'):
+        if weights_only is not None:
+            kwargs['weights_only'] = weights_only
+
+        return orig_torch_load(*args, **kwargs)  # NOTE: This code simply delegates the call to torch.load, and any errors that occur here are not the responsibility of Subpack.
+    else:
+        if weights_only is not None:
+            kwargs['weights_only'] = weights_only
+        else:
+            logging.warning("[Impact Subpack] Your torch version is outdated, and security features cannot be applied properly.")
+            kwargs['weights_only'] = False
+
+        return orig_torch_load(*args, **kwargs)
 
 torch.load = torch_wrapper
 
 
 def load_yolo(model_path: str):
     # https://github.com/comfyanonymous/ComfyUI/issues/5516#issuecomment-2466152838
-    with torch.serialization.safe_globals(torch_whitelist):
+    if hasattr(torch.serialization, 'safe_globals'):
+        with torch.serialization.safe_globals(torch_whitelist):
+            try:
+                return YOLO(model_path)
+            except ModuleNotFoundError:
+                # https://github.com/ultralytics/ultralytics/issues/3856
+                YOLO("yolov8n.pt")
+                return YOLO(model_path)
+    else:
         try:
             return YOLO(model_path)
         except ModuleNotFoundError:
-            # https://github.com/ultralytics/ultralytics/issues/3856
             YOLO("yolov8n.pt")
             return YOLO(model_path)
 

modules/subpack_nodes.py

@@ -30,6 +30,21 @@ class UltralyticsDetectorProvider:
 
     def doit(self, model_name):
         model_path = folder_paths.get_full_path("ultralytics", model_name)
+
+        if model_path is None:
+            print(f"[Impact Subpack] model file '{model_name}' is not found in one of the following directories:")
+
+            cands = []
+            cands.extend(folder_paths.get_folder_paths("ultralytics"))
+            if model_name.startswith('bbox/'):
+                cands.extend(folder_paths.get_folder_paths("ultralytics_bbox"))
+            elif model_name.startswith('segm/'):
+                cands.extend(folder_paths.get_folder_paths("ultralytics_segm"))
+
+            print(f'\t{"\n\t".join(cands)}\n')
+
+            raise ValueError(f"[Impact Subpack] model file '{model_name}' is not found.")
+
         model = subcore.load_yolo(model_path)
 
         if model_name.startswith("bbox"):

pyproject.toml

@@ -1,7 +1,7 @@
 [project]
 name = "comfyui-impact-subpack"
 description = "This node pack provides nodes that complement the Impact Pack, such as the UltralyticsDetectorProvider."
-version = "1.2"
+version = "1.2.8"
 license = { file = "LICENSE.txt" }
 dependencies = ["matplotlib", "ultralytics==8.3.40"]
 

requirements.txt

@@ -1,4 +1,5 @@
 matplotlib
 ultralytics==8.3.40
-numpy<2
-opencv-python-headless
+numpy==1.26.4
+opencv-python-headless
+dill