Stability-AI / stability-sdk

SDK for interacting with stability.ai APIs (e.g. stable diffusion inference)
https://platform.stability.ai/
MIT License
2.42k stars 338 forks source link

T2IAdapterParameters not present in generaion_pb2 #250

Closed jfdelgad closed 1 year ago

jfdelgad commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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 commented 1 year ago

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

Rikelai commented 11 months ago

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 commented 11 months ago

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

Rikelai commented 11 months ago

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