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

Multistate conditional sampling is inaccurate #97

Open
alxempirical opened this issue May 2, 2016 · 3 comments
Open

Multistate conditional sampling is inaccurate #97

alxempirical opened this issue May 2, 2016 · 3 comments

Comments

@alxempirical
Copy link
Contributor

Conditional draws from each model are given equal weight. This is inaccurate. Models should be drawn from the multinomial given by the total probability of the conditions.

@fsaad
Copy link
Collaborator

fsaad commented May 2, 2016

The same should be true for weighting the estimates of logpdf by probability of the conditions under each State.

For reference, gpmcc.Engine implements the weighting as post-processing steps on the entire set of samples and logpdfs returned from each State (which could be improved for simulate in order to remove the dependence between the resampled draws).

https://github.com/probcomp/gpmcc/blob/master/src/engine.py#L243-L256

@gregory-marton
Copy link
Contributor

I'd love to understand this better (problem and solution). May I ask for a
fifteen minute tutorial at some point?

On Mon, May 2, 2016 at 12:55 PM, F Saad notifications@github.com wrote:

The same should be true for weighting the estimates of logpdf by
probability of the conditions under each State.

For reference, gpmcc.Engine implements the weighting as post-processing
steps on the entire set of samples and logpdfs returned from each State
(which could be improved for simulate in order to remove the dependence
between the resampled draws).

https://github.com/probcomp/gpmcc/blob/master/src/engine.py#L243-L256


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#97 (comment)

Gregory Marton

gremio@mit.edu
617-858-0775
http://csail.mit.edu/~gremio/cal

@alxempirical
Copy link
Contributor Author

Sure, I'd be happy to. Just let me know when would be good.

On Mon, May 2, 2016 at 11:24 PM, Gregory Marton notifications@github.com
wrote:

I'd love to understand this better (problem and solution). May I ask for a
fifteen minute tutorial at some point?

On Mon, May 2, 2016 at 12:55 PM, F Saad notifications@github.com wrote:

The same should be true for weighting the estimates of logpdf by
probability of the conditions under each State.

For reference, gpmcc.Engine implements the weighting as post-processing
steps on the entire set of samples and logpdfs returned from each State
(which could be improved for simulate in order to remove the dependence
between the resampled draws).

https://github.com/probcomp/gpmcc/blob/master/src/engine.py#L243-L256


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#97 (comment)

Gregory Marton

gremio@mit.edu
617-858-0775
http://csail.mit.edu/~gremio/cal


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#97 (comment)

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

3 participants