Skip to content

improve --lora-model-dir documentation - #1223

Closed
l29ah wants to merge 1 commit into
leejet:masterfrom
l29ah:lora-model-dir-nonuse
Closed

improve --lora-model-dir documentation#1223
l29ah wants to merge 1 commit into
leejet:masterfrom
l29ah:lora-model-dir-nonuse

Conversation

@l29ah

@l29ah l29ah commented Jan 22, 2026

Copy link
Copy Markdown

No description provided.

@leejet

leejet commented Jan 22, 2026

Copy link
Copy Markdown
Owner

I don’t think this change adds much to understanding. For users who are not familiar with LoRA, they still need to consult the documentation https://github.com/leejet/stable-diffusion.cpp/blob/master/docs/lora.md to understand how it works; and for users who are already familiar with it, this change doesn’t really provide any additional value either.

@l29ah

l29ah commented Jan 22, 2026

Copy link
Copy Markdown
Author

The linked file doesn't contain the information this PR adds, instead If not specified, the default is the current working directory. directly contradicts the current behavior and misleads the user when LoRA doesn't work when the option is not present.

@leejet

leejet commented Jan 22, 2026

Copy link
Copy Markdown
Owner

“lora tags in prompt are ignored when unset” is not very accurate here.
When lora_model_dir is not set, LoRA tags are actually treated as normal prompt text. This behavior was likely caused by some previous changes and is now inconsistent with the documentation. #1224 should fix that issue and bring the behavior back in line with the docs.

@l29ah

l29ah commented Jan 22, 2026

Copy link
Copy Markdown
Author

Great, thanks!

@l29ah l29ah closed this Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants