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

How to configure "extra_model_paths.yaml" for xlabs models? #5248

Open
ClothingAI opened this issue Oct 14, 2024 · 8 comments
Open

How to configure "extra_model_paths.yaml" for xlabs models? #5248

ClothingAI opened this issue Oct 14, 2024 · 8 comments
Labels
User Support A user needs help with something, probably not a bug.

Comments

@ClothingAI
Copy link

ClothingAI commented Oct 14, 2024

Your question

Xlabs models (https://github.com/XLabs-AI/x-flux-comfyui) explian to us that models shall be put inside this path:
ComfyUI/models/xlabs/loras and ComfyUI/models/xlabs/controlnets

I would like to have the models in another path, could you tell me if it's possible to do it with the yaml file?
I tried to create a new entry like this:

....
vae: models/vae/
xlabs: ..._\models\xlabs

It did not work that great!
Ho to configure it what to write in the yaml?
image

Btw this is how it looks in base install:

image

Logs

No response

Other

No response

@ClothingAI ClothingAI added the User Support A user needs help with something, probably not a bug. label Oct 14, 2024
@Yaiol
Copy link

Yaiol commented Oct 15, 2024

you can use the standard windows mklink command which is much more versatile than the yaml file
move all xlabs files to [xlabs-path]
cd models
mklink /d xlabs [xlabs-path]
for instance mklink /d xlabs d:\models\xlabs

@ClothingAI
Copy link
Author

ClothingAI commented Oct 15, 2024

you can use the standard windows mklink command which is much more versatile than the yaml file move all xlabs files to [xlabs-path] cd models mklink /d xlabs [xlabs-path] for instance mklink /d xlabs d:\models\xlabs

my machine does not allow this.

@LukeG89
Copy link

LukeG89 commented Oct 15, 2024

I'm not an expert, but if you have already set the base path, you need to add a relative path to the xlab folder.

Have you tried this way?

    xlabs: models/xlabs

@ClothingAI
Copy link
Author

ClothingAI commented Oct 15, 2024

I just noticed that I tried this:
xlabs: models\xlabs
and it should have been \ instead of /
but doubt it would work, because there are some codes related to these in nodes.py or and config paths scripts etc

@LukeG89
Copy link

LukeG89 commented Oct 15, 2024

I just noticed that I tried this:
xlabs: models\xlabs
and it should have been \ instead of /
but doubt it would work, because there are some codes related to these in nodes.py or and config paths scripts etc

Yeah, it should work both ways.

If changing \ to / didn't work, I would also give a chance to this format as a last attempt:

    xlabs: |
         models/xlabs/flux
         models/xlabs/loras
         models/xlabs/controlnets
         models/xlabs/ipadapters

@ClothingAI
Copy link
Author

ClothingAI commented Oct 15, 2024

I think there should be some changes in folder_paths.py and nodes.py (from main comfy) to accompany the yaml changes. I tried with ChatGPT but could not find a good working solution.

@ltdrdata
Copy link
Collaborator

ltdrdata commented Oct 15, 2024

I think there should be some changes in folder_paths.py and nodes.py (from main comfy) to accompany the yaml changes. I tried with ChatGPT but could not find a good working solution.

Was it working fine before?
Is this symptom happening after updating ComfyUI?

@lldacing
Copy link

lldacing commented Oct 16, 2024

image

You can try xlabs_controlnets xlabs_loras xlabs_ipadapters

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
User Support A user needs help with something, probably not a bug.
Projects
None yet
Development

No branches or pull requests

5 participants