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: not allowed to use "AND" operator in sd-webui mode #95

Open
MaikoTan opened this issue Nov 1, 2022 · 4 comments
Open

Bug: not allowed to use "AND" operator in sd-webui mode #95

MaikoTan opened this issue Nov 1, 2022 · 4 comments
Labels
enhancement New feature or request prompt syntax stablediffusion related to Stable Diffusion backend

Comments

@MaikoTan
Copy link
Member

MaikoTan commented Nov 1, 2022

sd-webui supports "AND" (upper case) to activate Composable-Diffusion, but currently the bot would convert all input into lower cases, which causes incompatible from sd-webui.

A possible workaround is to ignore the only upper case word "AND" to make it work.

sd-webui支持 "AND"(大写)来激活Composable-Diffusion,但目前机器人会将所有的输入转换成小写,这导致与sd-webui不兼容。

一个可能的解决方法是忽略唯一的大写字母单词 "AND "来使其工作。

@shigma
Copy link
Member

shigma commented Nov 1, 2022

To be honest I don't actually think this is a well-designed syntax, at least for chatbots. The use of upper-cased words as special syntax does not draw attention to the difference and will definately lose its semantics in the process of copying and dissemination. Although I can support this feature, I have some concerns that people will misuse it.

Still, can someone explain the difference between AND, | and commas? (on the docs)

@shigma shigma added enhancement New feature or request prompt syntax labels Nov 1, 2022
@MaikoTan
Copy link
Member Author

MaikoTan commented Nov 1, 2022

Oh, you remind me that the | in NAI seems be equal with AND in sd-webui.

(And | in sd-webui was used for prompt matrix...)

噢,你提醒了我 | 在 NAI 里等价于 sd-webui 中的 AND

(然后 sd-webui 中的 | 用作提示符矩阵……)

@MaikoTan
Copy link
Member Author

MaikoTan commented Nov 1, 2022

references 参考

@MaikoTan
Copy link
Member Author

So a possible way is that we accept | in the prompt. If the backend is sd-webui, we just replace it as AND (note spaces) before sending the payload.
Also we need to update the document for explaining the differences between our plugin and sd-webui or others.

As for prompt matrix feature, as it only effects when you pass the payload with an explicit script name "prompt matrix". And currently we weren't even able to parse the prompt at all. I don't think we should implement this.

So what do you think? Is it worth doing this for now?

Best regard.
Maiko Sinkyaet Tan

@MaikoTan MaikoTan added the stablediffusion related to Stable Diffusion backend label Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request prompt syntax stablediffusion related to Stable Diffusion backend
Projects
None yet
Development

No branches or pull requests

2 participants