-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[ADD] ComfyUI-Manager #712
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works!
comfy-cpu-1 | 10.4 seconds: /stable-diffusion/custom_nodes/ComfyUI-Manager
comfy-cpu-1 |
comfy-cpu-1 | ****** User settings have been changed to be stored on the server instead of browser storage. ******
comfy-cpu-1 | ****** For multi-user setups add the --multi-user CLI argument to enable multiple user profiles. ******
comfy-cpu-1 | ### Loading: ComfyUI-Manager (V2.48.6)
comfy-cpu-1 | ### ComfyUI Revision: 2197 [276f8fce] | Released on '2024-05-20'
comfy-cpu-1 | [ComfyUI-Manager] default cache updated: https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/model-list.json
comfy-cpu-1 | [ComfyUI-Manager] default cache updated: https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/alter-list.json
comfy-cpu-1 | [ComfyUI-Manager] default cache updated: https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/github-stats.json
comfy-cpu-1 | [ComfyUI-Manager] default cache updated: https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/extension-node-map.json
comfy-cpu-1 | [ComfyUI-Manager] default cache updated: https://raw.githubusercontent.com/ltdrdata/ComfyUI-Manager/main/custom-node-list.json
comfy-cpu-1 | FETCH DATA from: /stable-diffusion/custom_nodes/ComfyUI-Manager/extension-node-map.json [DONE]
The only issue seems to be that ComfyUI manager ignores the Related: ltdrdata/ComfyUI-Manager/issues/420 |
I just installed and it works but is there a workaround for this? |
Hi, I have been trying to install custom nodes via GitHub and getting the error: Here is the docker file:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me. This would be great to merge, but I also understand that this does have potential side effects with an additional repo in the mix.
How are you accessing the interface? a lot of actions are restricted if you're not accessing it from localhost, this can be changed in the ComfyUI Manager settings, see https://github.com/ltdrdata/ComfyUI-Manager?tab=readme-ov-file#security-policy |
Closes issue #711
This PR aims to install the ComfyUI-Manager when building the Comfy docker image