-
Notifications
You must be signed in to change notification settings - Fork 8
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
How to use existing custom nodes #3
Comments
let me know which custom nodes you want to add, and i'll ensure they all work |
Thanks, give it a try now. |
Thanks for the quick fix. ComfyUI-Advanced-ControlNet, ComfyUI-AnimateDiff-Evolved, ComfyUI-Inspire-Pack, ComfyUI-VideoHelperSuite, ComfyUI-Impact-Pack custom_nodes are failing to import. Comfyroll custom_nodes partial importing. Also it seems like custom_nodes import is happening twice.
|
Fixed some import errors. One error remaining with ComfyUI-AnimateDiff-Evolved custom node. I need to test if nodes are actually working
|
I'll resolve these tomorrow, thanks for testing everything for me. The underlying issue is that I moved some files around, and the custom nodes are looking for them in places they no longer exist. To mitigate this, the names of imported files are monkey-patched in for the purposes of importing custom nodes. Once all the names have been added - and all the ways and names that the packages use have been addressed - the errors should go away. The |
Few things to consider on how I resolved these issues:
Take a look at my fork if you need: https://github.com/iRohith/ComfyUI |
thanks for these changes and for investigating the issue so thoroughy, I'll merge them tomorrow. It was always my intention to support "vanilla" custom nodes, while I have been porting nodes to use |
i think im having a similar issue... at least part of it. Python executable: /opt/conda/bin/python3.10 Prestartup times for custom nodes: Total VRAM 22518 MB, total RAM 16008 MB Loading: ComfyUI-Manager (V2.8.3)ComfyUI Revision: UNKNOWN (The currently installed ComfyUI is not a Git repository)[WARN] ComfyUI-Manager: Your ComfyUI version is outdated. Please update to the latest version. ComfyUI-Manager: Copy .js from '/home/jnkade/custom_nodes/ComfyUI-Manager/js/comfyui-manager.js' to '/opt/conda/lib/python3.10/site-packages/comfy/cmd/web/extensions/comfyui-manager'Traceback (most recent call last): Cannot import /home/jnkade/custom_nodes/ComfyUI-Manager module for custom nodes: partially initialized module 'comfy.cmd.server' has no attribute 'PromptServer' (most likely due to a circular import) Import times for custom nodes: Loading: ComfyUI-Manager (V2.8.3)ComfyUI Revision: UNKNOWN (The currently installed ComfyUI is not a Git repository)[WARN] ComfyUI-Manager: Your ComfyUI version is outdated. Please update to the latest version. ComfyUI-Manager: Copy .js from '/home/jnkade/custom_nodes/ComfyUI-Manager/js/comfyui-manager.js' to '/opt/conda/lib/python3.10/site-packages/comfy/cmd/web/extensions/comfyui-manager'Traceback (most recent call last): Cannot import /home/jnkade/custom_nodes/ComfyUI-Manager module for custom nodes: type object 'PromptServer' has no attribute 'instance' Import times for custom nodes: Starting server To see the GUI go to: |
I was going to do this now, but I see that your repo has gone private. |
Sorry about that. Updated now. Fixed importing vanilla nodes except ComfyUI-Manager with your latest changes. Please handle web extensions. I am not sure how they actually work. When importing custom nodes some web files are being placed in current directory and some in installed comfyui web directory. I did not test if vanilla custom nodes actually work when running. Only successful import tested |
thanks, will do, I inferred the changes from what you shared. I'll keep cranking at it |
The web extensions are really annoying but nothing I can't figure out :) Your |
I have started the long journey with ComfyUI-Manager and there's a big slog ahead. I appreciate your patience. I need to research a better way to patch these packages, probably by using mocking. |
I think it would be easiest to reimplement comfyui-manager's backend functionality, rather than try to patch it with code. that means reusing its UI and its lists of nodes. what behavior in it do you actually care about? |
Excellent project.
I am trying to use existing custom nodes. I created directories using "comfyui --create-directories". But adding existing custom nodes to the custom_nodes directory not working.
The text was updated successfully, but these errors were encountered: