-
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
fix: comfyui manager download path #618
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.
I hade a issue with installing custom nodes. This fixed it.
Wouldn't it be better to have a separate directory for checkpoints? |
And config. |
https://github.com/AbdBarho/stable-diffusion-webui-docker/blob/master/services/comfy/extra_model_paths.yaml |
Does that file even do anything? It gets copied (twice), but the entrypoint doesn't use it? |
Hey, sorry for being late on this, I did an update in #603, maybe this has an effect or fixes the issue. I am planning on simplifying the entire mount structure for all UIs since they are making it very hard to maintain, so maybe your problem could also be solved by this refactoring. https://github.com/AbdBarho/stable-diffusion-webui-docker/blob/master/services/comfy/extra_model_paths.yaml |
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Hey I was also looking for some documentation to understand how the models, config, extensions can be saved to be used by the 3 UIs as much as possible. Instead of having separate paths specified to each interface. |
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
Not stale. Still looking for documentation on sharing models and resources uniformly across the 3 GUIs. |
Hey @RyzeNGrind, I understand that you want to share models between the UIs as much as possible. Unfortunately, it is causing a noticeable maintenance overhead on my side to keep everything aligned, especially since I don't maintain any of the 3 UIs and stuff keeps changing underneath. I want to do some poll to understand what the users want / need to decide on the future structure. |
This PR is stale because it has been open 14 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
This PR was closed because it has been stalled for 7 days with no activity. |
Fix
comfyui manager
not downloading to export directory