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

Implement recognition of affine transforms #72

Closed
8 tasks done
fritzo opened this issue Mar 10, 2019 · 4 comments · Fixed by #286
Closed
8 tasks done

Implement recognition of affine transforms #72

fritzo opened this issue Mar 10, 2019 · 4 comments · Fixed by #286
Assignees
Labels
enhancement New feature or request

Comments

@fritzo
Copy link
Member

fritzo commented Mar 10, 2019

To perform full exact inference of Gaussian distributions, we will need some ability to recognize affine transforms. This might either rely on autograd, or could simply pattern match +, *, and einsum patterns.

Tasks

@fehiepsi
Copy link
Member

Thanks for keeping tracks of those tasks! It is convenient for me to understand what is going on.

@fritzo
Copy link
Member Author

fritzo commented Oct 16, 2019

@fehiepsi Do you want to try working on the task

Rewrite Gaussian(x=affine) as a Gaussian

following the ideas in #245 ? If not, I'm happy to do this part to unblock our first release. If so, it might be a nice intro to Funsor and give us insight into how to incorporate structured Gamma priors in Funsor #190, generalizing your Pyro PR pyro-ppl/pyro#2025 .

@fehiepsi
Copy link
Member

Hi @fritzo, could you do it instead? It is better for me to follow-up (though I want to try), given my lacking knowledge of Funsor. I am happy to review and learn from you.

@fritzo
Copy link
Member Author

fritzo commented Oct 17, 2019

@fehiepsi sounds good, I'll add you as reviewer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants