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

Add Bean Machine VI to PPL Bench #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

horizon-blue
Copy link
Collaborator

Summary:
Adding VI support to PPL Bench so we can easily benchmark the performance of different configurations on the models there.

To draw the samples that PPL Bench expects at the end, here we simply takes the VI distributions at the end of an inference (conditioned on the learned parameters) and draw N samples from them, where N equals number of iterations.

Differential Revision: D40951663

Summary:
Adding VI support to PPL Bench so we can easily benchmark the performance of different configurations on the models there.

To draw the samples that PPL Bench expects at the end, here we simply takes the VI distributions at the end of an inference (conditioned on the learned parameters) and draw N samples from them, where N equals number of iterations.

Differential Revision: D40951663

fbshipit-source-id: 1a04fc658c027684811a58c7d5bdf158e231e6f6
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Nov 8, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D40951663

iterations: int,
num_warmup: int,
seed: int,
algorithm: str = "ADVI",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

enum?

.clone()
.detach()
)
return MonteCarloSamples([samples])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha, this is fine but maybe we leave a TODO comment to prevent confusion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants