Skip to content

Incorporate AePPL into GaussianRandomWalk #5762

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
larryshamalama opened this issue May 14, 2022 · 1 comment
Closed

Incorporate AePPL into GaussianRandomWalk #5762

larryshamalama opened this issue May 14, 2022 · 1 comment

Comments

@larryshamalama
Copy link
Member

larryshamalama commented May 14, 2022

A recent PR #5298 added GaussianRandomWalk distribution to v4. It would be possible to use AePPL to obtain the Aesara graph corresponding to desired rv_op and retrieve the correct log-probability graph. The use of AePPL here would be desirable as we will no longer need to implement a corresponding RandomVariable class nor implement the logp method. Moreover, it would generalize the random walk to make use of non-Gaussian distributions.

cc @ricardoV94

@ricardoV94
Copy link
Member

Closed via #6072

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants