Skip to content

Conversation

@horizon-blue
Copy link
Contributor

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.

3 participants