-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
DDPMScheduler claims to have set_alpha_to_one in combination with steps_offset #6926
Comments
Hi @bghira It wasn't implemented for DDPM so I think it's a mistake in docstring. Do you think we need to add this? |
i'm not sure that it needs it, there is a further discussion on this issue now in #6068 |
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. |
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. |
When inspecting the configuration for SD/SDXL we noticed that
set_alpha_to_one
is mentioned in the scheduler's init docstring but it is nowhere to be found. It seems that DDPM either lacks this functionality, or does not need it.The text was updated successfully, but these errors were encountered: