[SD] Fix custom model path for WebUI #1260
Conversation
| match model: | ||
| case "models": | ||
| return Path(prefix, "models") | ||
| return Path(Path.cwd(), "models") |
There was a problem hiding this comment.
This is reverting @dan-garvey patch. Does this work with ckpt_dir provided in the command line ?
There was a problem hiding this comment.
So, if --ckpt_loc is being provided to the CLI's exe it works perfectly without touching this part of the code. I tested the CLI for txt2img and it works after this patch too.
This revision concerns WebUI where "prefix" will erroneously become only --ckpt_loc (irrespective of selecting LoRA/Custom Vae) and it'll concatenate models/vae etc to the path.
Let me know if we'd also want to let users access --ckpt_loc when invoking WebUI as well - my understanding is that for WebUI we'd let users keep the checkpoints under models directory, like earlier, and let them pick from the dropdown list.
-- This commit fixes custom model path for WebUI. Signed-off-by: Abhishek Varma <abhishek@nod-labs.com>
Abhishek-Varma
force-pushed
the
ckpt_dir
branch
from
4a75518 to
b9c5102
Compare
-- This commit fixes custom model path for WebUI.
Signed-off-by: Abhishek Varma abhishek@nod-labs.com