-
Notifications
You must be signed in to change notification settings - Fork 21
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
reactivate AD tests: FiniteGP #314
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ractGPs.jl into st/ad_tests2
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ractGPs.jl into st/ad_tests2
test/sparse_approximations.jl
Outdated
f = GP(SqExponentialKernel()) | ||
|
||
for Σy in (Σy_Diagonal, Σy_ScalMat) | ||
for Σy in (Σy_Diagonal, Σy_ScalMat) #, Σy_dense) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's not implemented for dense 🙈
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably for the best for now. If the observation covariance is dense, I would imagine that performance will generally be worse than exact inference.
Codecov Report
@@ Coverage Diff @@
## master #314 +/- ##
=======================================
Coverage 97.64% 97.64%
=======================================
Files 10 10
Lines 382 382
=======================================
Hits 373 373
Misses 9 9 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy with this, subject to the comments.
test/sparse_approximations.jl
Outdated
f = GP(SqExponentialKernel()) | ||
|
||
for Σy in (Σy_Diagonal, Σy_ScalMat) | ||
for Σy in (Σy_Diagonal, Σy_ScalMat) #, Σy_dense) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably for the best for now. If the observation covariance is dense, I would imagine that performance will generally be worse than exact inference.
Follow-up to #313