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

Feature/map #52

Merged
merged 45 commits into from
Mar 16, 2016
Merged

Feature/map #52

merged 45 commits into from
Mar 16, 2016

Conversation

dustinvtran
Copy link
Member

Summary:

This refactors the Inference class as discussed in #38. This will make it easier down the line for

It also adds MAP, where the user need only specify the model, data, and the transformation from unconstrained to constrained latent variables.

Examples using a Gaussian and the Beta-Bernoulli model are added.

Intended Effect:

N/A

How to Verify:

Run all examples to see that they still work.

Side Effects:

None.

Documentation:

Written in example script.

Reviewer Suggestions:

@mariru, @akucukelbir

@dustinvtran dustinvtran mentioned this pull request Mar 15, 2016
@akucukelbir
Copy link
Contributor

it looks good dustin.

but did i already lose the battle re: MAP being a type of VI? could we discuss briefly today before merging at least?

dustinvtran added a commit that referenced this pull request Mar 16, 2016
@dustinvtran dustinvtran merged commit 570bc5a into master Mar 16, 2016
@dustinvtran dustinvtran deleted the feature/map branch March 16, 2016 20:12
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.

None yet

3 participants