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

Invgamma-Normal Metropolis Hastings #475

Merged
merged 3 commits into from
Feb 23, 2017
Merged

Invgamma-Normal Metropolis Hastings #475

merged 3 commits into from
Feb 23, 2017

Conversation

bertini36
Copy link
Contributor

No description provided.

@dustinvtran
Copy link
Member

Cool! Funny that the acceptance rate is so low. That proposal is close to the posterior though. Is Inverse Gamma just that pathological?

Looks like the Travis build failed due to a PEP8 error. In keeping convention with the other example scripts, I also recommend ending decimals with 0s, e.g., 7.0 instead of 7. and removing significant digits, e.g., 0.7 instead of 0.70. As a tip, you can also use tf.zeros(1000) which is slightly cleaner than tf.zeros([1000]).

@dustinvtran
Copy link
Member

@dustinvtran
Copy link
Member

thanks again.

@dustinvtran dustinvtran merged commit 27ecf8b into blei-lab:master Feb 23, 2017
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

Successfully merging this pull request may close these issues.

2 participants