-
Notifications
You must be signed in to change notification settings - Fork 6
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
Initial proof-of-concept Expectation Propagation #64
base: master
Are you sure you want to change the base?
Conversation
…eGPs.jl into st/laplace_and_ep
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…rocesses/ApproximateGPs.jl into st/ExpectationPropagation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Why are the tests failing when all I did was merge in master (with #76)? 😢 |
Caused by JuliaDiff/ChainRules.jl#559 it seems (similar to issues in Distributions, DistributionsAD, NNlib etc.). Should be fixed by the new ChainRules release (bugfix was JuliaDiff/ChainRules.jl#562). |
Co-authored-by: David Widmann <[email protected]>
Thanks @devmotion ! it seems to be passing again :) |
@theogf what do you think of this as a first stab ? |
…eGPs.jl into st/ExpectationPropagation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
This is a very basic version to demonstrate proof-of-concept. See #92 for missing features.