Trung0246 / ComfyUI-0246

Random nodes for ComfyUI
MIT License
113 stars 11 forks source link

Comfyui latest update: AttributeError: 'NoneType' object has no attribute 'get_input_info' #38

Closed bananasss00 closed 3 months ago

bananasss00 commented 3 months ago
Traceback (most recent call last):
  File "V:\comfyu_py311\ComfyUI\nodes.py", line 1982, in load_custom_node
    module_spec.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "V:\comfyu_py311\ComfyUI\custom_nodes\ComfyUI-0246\__init__.py", line 1, in <module>
    from .nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
  File "V:\comfyu_py311\ComfyUI\custom_nodes\ComfyUI-0246\nodes.py", line 1071, in <module>
    lib0246.hijack(comfy_graph, "get_input_info", get_input_info_param_handle, get_input_info_res_handle)
  File "V:\comfyu_py311\ComfyUI\custom_nodes\ComfyUI-0246\utils.py", line 373, in hijack
    old_func = getattr(scope, name)
               ^^^^^^^^^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'get_input_info'
Danamir commented 3 months ago

Can confirm, I got the same bug with ComfyUI: 2546[bfc214](2024-08-15).

BlackVortexAI commented 3 months ago

Same her with version bfc214

Trung0246 commented 3 months ago

Hm looks like the topological execution changed something. I'll take a look tomorrow.

Reference: https://github.com/comfyanonymous/ComfyUI/pull/2666

Edit: from quick glance looks like the module path changed, hopefully should be simple fix.

Trung0246 commented 3 months ago

Initial quick fix has been done. Did not test it yet for actual feature breakage since it's 1:00am here, but it should be safe as long as there's no major change in the pull request. Plus I was on vacation for last 2 weeks so I did not heard about the news it being merged.

BlackVortexAI commented 3 months ago

Working THX

Danamir commented 3 months ago

Plus I was on vacation for last 2 weeks so I did not heard about the news it being merged.

No worries, thanks for the quick fix ! 😅