Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
bananasss00 opened this issue Aug 16, 2024 · 6 comments

Comments

@bananasss00
Copy link

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
Copy link

Danamir commented Aug 16, 2024

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

@BlackVortexAI
Copy link

Same her with version bfc214

@Trung0246
Copy link
Owner

Trung0246 commented Aug 16, 2024

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

Reference: comfyanonymous/ComfyUI#2666

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

@Trung0246
Copy link
Owner

Trung0246 commented Aug 16, 2024

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
Copy link

Working THX

@Danamir
Copy link

Danamir commented Aug 16, 2024

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 ! 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants