divamgupta / stable-diffusion-tensorflow

Stable Diffusion in TensorFlow / Keras
Other
1.57k stars 227 forks source link

[Inquiry] VAEEncoder bug in KerasCV #71

Closed LukeWood closed 1 year ago

LukeWood commented 1 year ago

Hello @divamgupta ! As you know, we ported the image encoder in https://github.com/keras-team/keras-cv from your library! We've supported inpainting via a method on StableDiffusion, and we're seeing a very strange issue! https://github.com/keras-team/keras-cv/issues/1172 is the github issue.

We've tried just passing images through the encoder/decoder and this padding shows up.

My question for you is: did you have this bug in your repo? Did you submit a fix at some point? Anything that you may know that may be helpful?

Thanks in advance for any guidance! We appreciate your efforts a ton

divamgupta commented 1 year ago

You would need to use zero padding of 1 0 1 0 rather than 1 1 1 1

On Sun, Apr 9, 2023 at 8:00 PM Luke Wood @.***> wrote:

Hello @divamgupta https://github.com/divamgupta ! As you know, we ported the image encoder in https://github.com/keras-team/keras-cv from your library! We've supported inpainting via a method on StableDiffusion, and we're seeing a very strange issue! keras-team/keras-cv#1172 https://github.com/keras-team/keras-cv/issues/1172 is the github issue.

We've tried just passing images through the encoder/decoder and this padding shows up.

My question for you is: did you have this bug in your repo? Did you submit a fix at some point? Anything that you may know that may be helpful?

Thanks in advance for any guidance! We appreciate your efforts a ton

— Reply to this email directly, view it on GitHub https://github.com/divamgupta/stable-diffusion-tensorflow/issues/71, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAONR5MDOOF4N4EYBIWZHI3XANEQDANCNFSM6AAAAAAWYM4YDM . You are receiving this because you were mentioned.Message ID: @.***>

LukeWood commented 1 year ago

Thank you sir!