-
Notifications
You must be signed in to change notification settings - Fork 305
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
Image Preview #416
base: master
Are you sure you want to change the base?
Image Preview #416
Conversation
@amblamps images are saved in this time, but impact is minimal. I will try to test different approaches to minimize times, but most of the time is |
After testing I can confirm that with right callback function, with will not stop execution - everything else is 0 second, except |
For big batches, if something went wrong - we are losing all generated images. I want to get images as they a sampled. This way I can save content to disk and/or show preview as it get done.
Its also a step to #354, we can just call a
sd_result_cb
with additional parameters. This will take time to modify each sample method insample_k_diffusion()
so i do plan to make it as separate PR (if i can get it working).What do you think about this changes? This PR is not breaking, just replacing functionality if callback is set.