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

fix: 125 control net options should have the same naming as on the webapp #152

Conversation

TheLLspectre
Copy link
Contributor

Now according to modalities available on the webapp and SDXL / SD 1.5 tags, we have same options available on Unity plugin.
Adding an array of translation of those modalities to manage api request and displaying interface in Unity.
Also renaming some variables.

image

image

TheLLspectre and others added 4 commits March 6, 2024 15:46
According to tag some modalities option are now available.
And has a corresponding translation to call correct modality inside api.
@qvaleroo
Copy link
Collaborator

Opening prompt window without modifying the model does not prefill Modalities value:
image

@Morgan-6Freedom
Copy link
Contributor

@TheLLspectre still waiting for your fixes on this

@TheLLspectre
Copy link
Contributor Author

TheLLspectre commented Mar 15, 2024

How to reproduce this ?? I don't have this behaviour :/ @qvaleroo

Opening prompt window without modifying the model does not prefill Modalities value: image

@qvaleroo
Copy link
Collaborator

I do not reproduce, after starting from scratch. Let's merge. I let you handle conflicts

@TheLLspectre
Copy link
Contributor Author

TheLLspectre commented Mar 16, 2024

I do not reproduce, after starting from scratch. Let's merge. I let you handle conflicts

The only way I had to reproduce it, it's on a first install and launch, when I don't have any model selected and here I don't had modalities but it's normal

image

@TheLLspectre
Copy link
Contributor Author

@qvaleroo @Morgan-6Freedom

I managed conflict with a merge dev into branch !

@qvaleroo qvaleroo merged commit 63d1744 into develop Mar 18, 2024
1 check passed
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.

3 participants