-
Notifications
You must be signed in to change notification settings - Fork 6k
Added brownian_noise to DMP 2++ SDE Scheduler and fixed use_exponential_sigmas behaviour #9955
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
base: main
Are you sure you want to change the base?
Conversation
@Crized-bit thanks for your PR!
Could you maybe supplement a minimal reproducible snippet to understand this better? |
Here it is! With this code i had issues that i've fixed
|
was it fixed in #9954? |
Seems so! So, just added noise |
What does this mean? |
This means that the half of my PR was fixed by another PR (#9954) , but the part where i've added brownian_noise (as it's done in k-diffusers and AUTO-1111) is unique and in need of review. |
@yiyixuxu could give a look? |
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
What does this PR do?
Added the option to supply custom Brownian noise generator as was made in AUTO1111 webui.
Also fixed issue with use_exponential_sigmas crashes (nonetype and order issue)
Before submitting
documentation guidelines, and
here are tips on formatting docstrings.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@yiyixuxu