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

Custom Nodes documentation #42

Open
ecornbower opened this issue Jun 19, 2024 · 5 comments
Open

Custom Nodes documentation #42

ecornbower opened this issue Jun 19, 2024 · 5 comments
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@ecornbower
Copy link

The current documentation recommends forking the entire repo and modifying the Dockerfile in order to add custom nodes.

It might be better to recommend creating a new Dockerfile and use the pre-build image as a base. For example:

FROM timpietruskyblibla/runpod-worker-comfy:2.2.0 as base

WORKDIR /comfyui/custom_nodes

# WAS Node Suite
RUN git clone https://github.com/WASasquatch/was-node-suite-comfyui.git ./was
RUN pip install -r ./was/requirements.txt --no-cache-dir

# Go back to the root
WORKDIR /

# Start the container
CMD /start.sh

It's a lot simpler, quicker to build, and easier to update to the latest release.

@billyberkouwer
Copy link

I've been following this pattern for installing custom nodes and it does work, however I've found this logged when I run the image. Was still fails to load, although it is installed. Do you have any ideas for how to approach a fix for this, Any help would be much appreciated!

2024-08-01T22:48:54.605832620Z Import times for custom nodes:
2024-08-01T22:48:54.605854420Z    0.0 seconds: /comfyui/custom_nodes/websocket_image_save.py
2024-08-01T22:48:54.605856540Z    0.0 seconds: /comfyui/custom_nodes/comfyui-various
2024-08-01T22:48:54.605858270Z    0.0 seconds: /comfyui/custom_nodes/ComfyUI-Custom-Scripts
2024-08-01T22:48:54.605869639Z    0.0 seconds: /comfyui/custom_nodes/ComfyUI_essentials
2024-08-01T22:48:54.605871529Z    0.1 seconds: /comfyui/custom_nodes/ComfyUI_FizzNodes
2024-08-01T22:48:54.605883159Z    0.3 seconds: /comfyui/custom_nodes/ComfyUI_Comfyroll_CustomNodes
**2024-08-01T22:48:54.605885169Z    0.8 seconds (IMPORT FAILED): /comfyui/custom_nodes/was-node-suite-comfyui**

@ecornbower
Copy link
Author

@billyberkouwer Sounds like an issue with WAS since your other nodes are working. I would reach out to them instead. Maybe try loading an older version of WAS and see if they broke something.

@billyberkouwer
Copy link

Thank you!

@TimPietrusky
Copy link
Member

@ecornbower that sounds like a super good idea! Thank you very much.

Once #30 is done, we hope that custom nodes can also be added with ease.

@TimPietrusky TimPietrusky self-assigned this Aug 11, 2024
@TimPietrusky TimPietrusky added the documentation Improvements or additions to documentation label Aug 11, 2024
@LotzF
Copy link

LotzF commented Sep 5, 2024

@billyberkouwer You're docker container's most likely missing ffmpeg. Docker doesn't have it installed per default and it's not part of the python packages but rather an apt command. Go ahead and install it yourself by altering the apt-get install -y section of the Dockerfile.

Screenshot from 2024-09-05 12-21-21

WAS requires ffmpeg to work otherwise the import will fail during startup.

Cheers
Leo

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

No branches or pull requests

4 participants