-
Notifications
You must be signed in to change notification settings - Fork 22
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
VFE/DTC should be moved to ApproximateGPs.jl #318
Comments
I keep changing my mind on this. On the one hand, the name suggests that this is a good idea. On the other, VFE / DTC / FITC don't add any additional deps to AbstractGPs, and the approximate posteriors produced satisfy the AbstractGPs API. I don't think I would get in the way of this move, but I can also live with the status quo. |
Well in my naive user experience, I kept looking in ApproximateGPs for it before realizing it was in AbstractGPs after asking on Slack. |
Haha fair enough. If you didn't know it was there, we should probably just move it here. |
Just to add my 2 cents (far more naive than @theogf): After quite some digging to figure out where does Stheno find VFE, I was quite surprised to find it in AbstractGPs. |
No description provided.
The text was updated successfully, but these errors were encountered: