Skip to content
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

Forward Problem #3

Open
FeiiYin opened this issue Jan 5, 2023 · 1 comment
Open

Forward Problem #3

FeiiYin opened this issue Jan 5, 2023 · 1 comment

Comments

@FeiiYin
Copy link

FeiiYin commented Jan 5, 2023

Hi, thanks for your re-implementation.
There exist one question why do you use the U-Net to estimate the x_0 in the forward process? Why not directly add the fixed or sampled gaussian noise onto the given input and only use the U-Net in the reverse process?

Thanks in advance!

@wenqsun
Copy link

wenqsun commented Aug 25, 2023

Hi, thanks for your re-implementation. There exist one question why do you use the U-Net to estimate the x_0 in the forward process? Why not directly add the fixed or sampled gaussian noise onto the given input and only use the U-Net in the reverse process?

Thanks in advance!

I think that he uses the ODE to formulate the adding noise process, and for the reverse ODE process, we need to compute the predicted x_0, for the reverse process, you can refer to the codes in improved_DDPM: https://github.com/openai/improved-diffusion/blob/783b6740edb79fdb7d063250db2c51cc9545dcd1/improved_diffusion/gaussian_diffusion.py#L524C5-L561C1

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

No branches or pull requests

2 participants