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

T2IAdapterParameters not present in generaion_pb2 #250

Closed
jfdelgad opened this issue Jul 16, 2023 · 9 comments
Closed

T2IAdapterParameters not present in generaion_pb2 #250

jfdelgad opened this issue Jul 16, 2023 · 9 comments

Comments

@jfdelgad
Copy link

The generaion_pb2 is present in the API interface repo, but the SDK repository points to an older version of the API interface of it where T2IAdapterParameters is not defined.

Does this mean that the SDK does not support the use of the T2IAdpaters? if so, is there a plan to support this?

@satsumas
Copy link
Contributor

Well spotted!

We will update the submodule pointer of the SDK to the head of Interfaces and enable support for T2I adapters very soon. The current SDK doesn't yet support them.

@jfdelgad
Copy link
Author

Well spotted!

We will update the submodule pointer of the SDK to the head of Interfaces and enable support for T2I adapters very soon. The current SDK doesn't yet support them.

Do you have any estimation for this, are these weeks, or months?

Also, if the SDK does not support this yet, is there a way to do this directly by calling the API, given that the interface already exists?

@satsumas
Copy link
Contributor

It should be in the order of days.

Using the adapters requires more code to be merged server-side, which will happen when the feature is released.

@jfdelgad
Copy link
Author

jfdelgad commented Jul 25, 2023

It should be in the order of days.

Using the adapters requires more code to be merged server-side, which will happen when the feature is released.

Is there any update about this, is T2IAdapter available already?

I can see that is now possible to pass the adapter in the generate. function. But this is not part of the latest release, or is it?

@jfdelgad
Copy link
Author

It should be in the order of days.

Using the adapters requires more code to be merged server-side, which will happen when the feature is released.

is there any update on this @satsumas ?

@jfdelgad
Copy link
Author

never mind, I can see that you have already released a new version with those changes. thank you!

@Rikelai
Copy link

Rikelai commented Oct 20, 2023

Hey @jfdelgad, I was looking for a way to add extras params to image generation api. In docs I read about this "extras" params where you can add struct object. I'm using rest api and trying to pass adapter. So i append to my formData: "extras" as a stringinfied object with
{
adapter_type: "DEPTH",
adapter_strength: 1,
adapter_init_type: "IMAGE"
}
But no effect on image generation.
Could you show me how to successfully pass data ?
Thank you !

@pharmapsychotic
Copy link
Member

@Rikelai sorry the feature wound up never released so it won’t work in the SDK or through REST.

@Rikelai
Copy link

Rikelai commented Oct 21, 2023

@pharmapsychotic thank you for your answer, so from my understanding what is ready client side doesn't match server side ?

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

No branches or pull requests

4 participants