Skip to content

Conversation

@StAlKeR7779
Copy link
Contributor

@StAlKeR7779 StAlKeR7779 commented Jun 18, 2023

Added sde schedulers.
Problem - they add random on each step, to get consistent image we need to provide seed or generator.
I done it, but if you think that it better do in other way - feel free to change.

Also made ancestral schedulers reproducible, this done same way as for sde scheduler.

@psychedelicious
Copy link
Contributor

Am I understanding correctly - inside SDE schedulers, they use RNG to determine the... time steps?

@StAlKeR7779
Copy link
Contributor Author

StAlKeR7779 commented Jun 19, 2023

Am I understanding correctly - inside SDE schedulers, they use RNG to determine the... time steps?

No, they use noise to generate output
https://github.com/huggingface/diffusers/blob/main/src/diffusers/schedulers/scheduling_dpmsolver_multistep.py#L639
image
image

Copy link
Contributor

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested working on txt2img, img2img and inpaint (cherry picked onto #3550), sweet!

@blessedcoolant blessedcoolant merged commit bb2df88 into main Jun 19, 2023
@blessedcoolant blessedcoolant deleted the feat/sde_schedulers branch June 19, 2023 10:52
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.

4 participants