-
Notifications
You must be signed in to change notification settings - Fork 6k
Fix Callback Tensor Inputs of the SD Controlnet Pipelines are missing some elements. #10907
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 Callback Tensor Inputs of the SD Controlnet Pipelines are missing some elements. #10907
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've get similar problem which I'm unable to get image from controlnet as callback in my logging for alignment purpose. This fix resolves it.
|
BTW SD3 and Flux have the same issue that this PR is trying to fix. |
thanks again, we're aware of the different name in the pipelines, that's why the new ones have a more uniform name instead of a changing one between the controlnet and non controlnet pipelines. This is something we're going to fix eventually but it's a breaking change so we have to plan for it and probably we will wait for modular diffusers first. |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
failing test is not related to this PR. |
Thank you! @asomoza |
The properties _callback_tensor_inputs of
StableDiffusionControlNetImg2ImgPipeline
,StableDiffusionControlNetInpaintPipeline
are missing an important elementcontrolnet_image
, which makes it impossible to retrieve controlnet image from callback_kwargs of callback_on_step_end in these pipelines.StableDiffusionControlNetPipeline
is a bit of different, because it is missingimage
, which is another name of control image in this pipeline.This PR is to fix the bug above.