Skip to content
This repository has been archived by the owner on Dec 18, 2023. It is now read-only.

Support parameters in World #1605

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

Conversation

feynmanliang
Copy link
Contributor

Summary:
To support the use of bm.params in the generative model, beanmachine's VI now:

  • Traces through World in addition to VariationalWorld so that params used in World are added to the optimizer
  • Utilizes a VariationalWorld for both the generative and guide distributions in gradient_estimators so that params can be used in the generative model

This is needed to support PPCA (N2358633)

Differential Revision: D38754091

Summary:
To support the use of `bm.param`s in the generative model, beanmachine's VI now:
 * Traces through `World` in addition to `VariationalWorld` so that `param`s used in `World` are added to the optimizer
 * Utilizes a `VariationalWorld` for both the generative and guide distributions in `gradient_estimators` so that `params` can be used in the generative model

This is needed to support PPCA (N2358633)

Differential Revision: D38754091

fbshipit-source-id: c110e22f9a4b7c9c86e37b9884d8e931466a0abf
@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 Aug 16, 2022
@facebook-github-bot
Copy link
Contributor

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.

2 participants