-
-
Notifications
You must be signed in to change notification settings - Fork 448
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
[Feature]: Custom KDiffusion Scheduler #1242
Comments
Thanks @KohakuBlueleaf - I'd love to have this! I see that A1111 opted for global settings - I actually have different opinion, much closer to your original proposal. The reason is that I believe that any setting that user may choose to modify frequently should be local and passed as parameter to actual generate function instead of generate function reading it from options. Why?
Btw, for that exact reason I've reimplemented |
@vladmandic ok What I'm thinking is something like remove all the "karras" sampler, and then use the dropdown for user to choose the sceduler. So k_diffusion sampler with scheduler karras (default sigma) will behave like old xxx karras sampler |
yes, i think that behavior like hires fix would be perfect - show options when checkbox is selected. |
@KohakuBlueleaf btw, since you're interested in schedulers, take a look at #1268. i think notes raised in paper are very valid and this is pretty high-value item (also enabled via checkbox to have backwards compatibility) - what do you think? |
@vladmandic first, there is 3 problem 2, you need to change the alpha schedule in models to make sure k_diffusion has correct conversion between sigma and timesteps 3, I already try 1,2 on normal sd model and it just broken all the things, basically max_sigma higher than 1000 will give you super bad result. Normally max_sigma = 100 is the upper bound. Conclusions: And if the model already contain fixed/different schedule, just use 0.0 for sigma min/max and it will read the value from models' params |
@KohakuBlueleaf just looping back on this one - i see that your pr is merged in a1111 dev branch. |
@vladmandic I'm quite busy these days (until 7/5 I guess) |
finally got around to this :) |
Feature description
Based on AUTOMATIC1111/stable-diffusion-webui#10621
and pr at AUTOMATIC1111/stable-diffusion-webui#10649
I think it is worth to add this feature into this repo too.
But the problem is, I don't know if @vladmandic you want the settings one or something more like my original pr AUTOMATIC1111/stable-diffusion-webui#10634
And if you want, I can add something like export the plot of the noise schedule. (I think this is doable?)
Version Platform Description
No response
The text was updated successfully, but these errors were encountered: