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

Question about q_posterior. #6

Closed
guyuchao opened this issue Jul 1, 2022 · 2 comments
Closed

Question about q_posterior. #6

guyuchao opened this issue Jul 1, 2022 · 2 comments

Comments

@guyuchao
Copy link

guyuchao commented Jul 1, 2022

Although you have noticed us that we can use x_t instead of x_tmin1, I have trouble understanding why it can hold.
# unnormed_logprobs = log_EV_qxtmin_x0 + log q_pred_one_timestep(x_t, t)
# Note: _NOT_ x_tmin1, which is how the formula is typically used!!!

Instead, I want to to follow the original formula that first sample x_tmin1 based on x0, then use this to compute q(xt|x_{t-1}). Is the following code correct? And have you tried experiment with the original formula?
log_x_tmin = self.log_sample_categorical(log_EV_qxtmin_x0)
unnormed_logprobs = log_EV_qxtmin1_x0 + self.q_pred_one_timestep(log_x_tmin, t)

@ehoogeboom
Copy link
Owner

The comment in the code there is a bit confusing, I'll remove that.

Your last suggestion though is to sample x_tmin1, that is doing something different. In the original formula's we don't want to sample, we want to compute the posterior closed-form.

@guyuchao
Copy link
Author

guyuchao commented Jul 5, 2022

Great thanks.

@guyuchao guyuchao closed this as completed Jul 5, 2022
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