Can't handle more than two keywords

[alfonslm]

The current interface can't handle more than two keywords and ie. SynthPunk V.3 (https://civitai.com/models/2856/synthpunk-search) uses 3 keywords. I would probably solve it by using checkboxes and ordering them from an input number next to them or by having a drag and drop list with keywords at the start, end and inactive.

[mix1009]

Extension only runs when you generate an image, or interact with the extension UI. I don't think it's possible to update the UI according to the current selected model. I could add 'keyword3', 'keyword4' ... etc. would it help?

[alfonslm]

Could you do something like foreach(keyword){print "keywordname" + selectionboxN} and add an update button that you press when you change diffusion model since using a hardcoded list will probably always have the wrong number of options in this case.

[mix1009]

I'll check if it's possible to change the dropdown list when a button is pressed. But if user doesn't press the button it will not update and will confuse users more IMO.

You can see the keyword list by clicking "Check" button in 'Add Custom Mappings' section. I think adding 'keyword3', 'keyword4', 'keyword5' is the second best way to solve the problem. If there is no keywordX, it will default to keyword1.

[alfonslm]

If there is a button that says "Update keywords for new model" at the top it is hopefully good enough for most

[mix1009]

Refresh button for Multiple keywords has been implemented :)

Button placement is a bit off. It's not easy to control using gradio components.

Thank you for the request.

[alfonslm]

So, this is only relevant since you got it working but what do you feel about adding a persistent list probably mostly for "Textual Inversion"

[mix1009]

Interesting! Should it be included in this extension? Or maybe a new extension?

[alfonslm]

Not sure if it would be beneficial to have a second extension with some code removed with more or less the same usage. You don't need to use get_old_model_hash or do all of the elif since only the multiple_keywords condition is relevant for the keywords, at least sort of. It might be easier to try and optimize the code for the keywords also since they don't need any extra steps.

I guess that default settings per model could be the next step since some got keywords that is optional eg Arts and Eros list (ELDEN RING STYLE, MODELSHOOT STYLE, DREAMLIKEART, POSTAPOCALYPSE, ANALOG STYLEK, NOLLINGCASE, SWPUNK, SYNTHWAVE, CYBORGDIFFUSION)

[mix1009]

If I add "textual inversion" dropdown separately to this extension, how should I order the keywords?

model_keyword ti_keyword or ti_keyword model_keyword

Should that be an option?

[alfonslm]

I guess ti_keyword makes more sense first if model_keyword will get a default setting. Otherwise it probably don't mater all that much

[alfonslm]

Right, model and textural inversion keyword should probably have a switch for what order it would be added. Model, Textural Inversion or Textural Inversion, Model

[mix1009]

Textual Inversion is now implemented!