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

[Bug]: Extra networks not showing tree view if Commandline arg points to another path #121

Closed
4 of 6 tasks
Dawgmastah opened this issue Feb 8, 2024 · 9 comments
Closed
4 of 6 tasks
Labels
bug Confirmed report of something that isn't working

Comments

@Dawgmastah
Copy link

Checklist

  • The issue exists after disabling all extensions
  • The issue exists on a clean installation of webui
  • The issue is caused by an extension, but I believe it is caused by a bug in the webui
  • The issue exists in the current version of the webui
  • The issue has not been reported before recently
  • The issue has been reported before but has not been fixed yet

What happened?

As written in title, the extra networks does not show folders (I checke don the options already)
The only thing im thinking should be interfering is that im pointing ther Loras folder to my auto folder using commandline args.

This is the webui copypaste:
set VENV_DIR=
set A1111_HOME="X:\STABLEDIFFUSION\AUTOMATIC11111"
set COMMANDLINE_ARGS= --always-gpu --ckpt-dir "X:\AIMODELS" --lora-dir %A1111_HOME%/models/Lora --embeddings-dir %A1111_HOME%/models/embeddings --hypernetwork-dir %A1111_HOME%/models/hypernetworks --vae-dir "X:\STABLEDIFFUSION\AUTOMATIC11111\models\VAE" --gfpgan-dir "X:\STABLEDIFFUSION\AUTOMATIC11111\models\GFPGAN" --esrgan-models-path "X:\STABLEDIFFUSION\AUTOMATIC11111\models\ESRGAN" --bsrgan-models-path "X:\STABLEDIFFUSION\AUTOMATIC11111\models\BSRGAN"

Maybe Its a problem with the quotes? The loras are found and load.

Steps to reproduce the problem

Point to a folder in other location using commandline args, use the extra networks viewer

What should have happened?

Should show folders/tree view

What browsers do you use to access the UI ?

No response

Sysinfo

sysinfo-2024-02-07-15-44.json

Console logs

N/A

Additional information

No response

@ManOrMonster
Copy link

I'm also having this issue (Windows 11). I'm not using quotes in my paths.

set A1111_HOME=C:/stable-diffusion/data
set COMMANDLINE_ARGS=%COMMANDLINE_ARGS% ^
--ckpt-dir %A1111_HOME%/models/Stable-diffusion ^
--embeddings-dir %A1111_HOME%/embeddings ^
--lora-dir %A1111_HOME%/models/Lora ^
--esrgan-models-path %A1111_HOME%/models/ESRGAN ^
--vae-dir %A1111_HOME%/models/VAE

I noticed that the default model that's downloaded at first run shows in the tree view. So extra networks is still looking at the original folders, and any models in folders there show up. Just not any folders in our folder paths we have set in the arguments.

@Gothmoth88
Copy link

Gothmoth88 commented Feb 11, 2024

i don´t know why this is not put into the settings anyway, why people who are not coders have to use arguments. the settings have so many option.... yet none to set the folder path. maybe it is coder logic but to me as a laymen it makes no sense.

@Dawgmastah
Copy link
Author

Dawgmastah commented Feb 12, 2024

I just bit the bullet and solved this on my end using Symlinks.
Now I have no arguments on forge, only 1 on automatic pointing to the forge controlnet models folder, and both UIs see the checkpoints, loras and embeddings precisely where they should.

There just some issues with how Lycoris are implemented in the civitai helper, but I can live with that

@ManOrMonster
Copy link

I just bit the bullet and solved this on my end using Symlinks.

That's definitely a quickfix for this, but I've personally been trying to get away from symlinks. And since the application supports pointing to different folders, it should support that internally throughout the app.

@ManOrMonster
Copy link

I've almost resolved the issue with this config:

set A1111_HOME=C:\\stable-diffusion\\data
set COMMANDLINE_ARGS=%COMMANDLINE_ARGS% ^
--ckpt-dir %A1111_HOME%\\models\\Stable-diffusion ^
--embeddings-dir %A1111_HOME%\\embeddings ^
--lora-dir %A1111_HOME%\\models\\Lora ^
--esrgan-models-path %A1111_HOME%\\models\\ESRGAN ^
--vae-dir %A1111_HOME%\\models\\VAE

Note the double backslashes instead of forward slashes.

My folder trees now show up properly in extra networks except for Checkpoints. My checkpoints still showed up though, but no tree.

However, after deleting the default model that is downloaded into the webui/models/Stable-diffusion folder when first running Forge, it now only displays one checkpoint, which happens to be the only one in a non-hidden folder. All of my model subfolders start with ".", since they are categorized heavily, and this is not causing a problem with the LORA or textual inversion trees. The models still are listing properly in the "Stable Diffusion checkpoint" selector.

So it seems like there is some hard coding somewhere that always looks at the default checkpoint folder in addition to the custom one in the config, which is affecting the tree view in Checkpoints under extra networks.

@catboxanon catboxanon added bug Confirmed report of something that isn't working upstream labels Feb 15, 2024
@catboxanon
Copy link
Collaborator

Confirmed issue with latest upstream. Replacing forward slashes with double backslashes fixes it in the meantime. I'm unable to reproduce the issue being described above where only the checkpoints folder does not display a tree though.

@ManOrMonster
Copy link

ManOrMonster commented Feb 15, 2024

@catboxanon

Confirmed issue with latest upstream. Replacing forward slashes with double backslashes fixes it in the meantime. I'm unable to reproduce the issue being described above where only the checkpoints folder does not display a tree though.

What is your --ckpt-dir pointing to? Is it the same as mine ("Stable-diffusion") or do you have a different folder name?

@catboxanon
Copy link
Collaborator

catboxanon commented Feb 16, 2024

You're right, it looks like it's broken if the folder uses the same default name (Stable-diffusion). Regardless, that should be reported and resolved upstream, not here.

@catboxanon
Copy link
Collaborator

catboxanon commented Feb 17, 2024

I'm closing this as the original issue in OP has been resolved via AUTOMATIC1111/stable-diffusion-webui#14934, which is now merged into this repo. Please open a new issue upstream for the other issue if it still persists as I mentioned above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed report of something that isn't working
Projects
None yet
Development

No branches or pull requests

4 participants