Adding Matern72Kernel object, tests and docs#580
Adding Matern72Kernel object, tests and docs#580willtebbutt merged 2 commits intoJuliaGaussianProcesses:masterfrom
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #580 +/- ##
===========================================
- Coverage 93.61% 14.58% -79.04%
===========================================
Files 52 52
Lines 1457 1433 -24
===========================================
- Hits 1364 209 -1155
- Misses 93 1224 +1131 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Not too sure why codecov complains, I am testing the lines it is flagging following the tests of |
willtebbutt
left a comment
There was a problem hiding this comment.
CI breakages here look like they're related to AD issues on 1.12, rather than to problems with your PR, because all of the other CI passes nicely.
Could you please bump the patch version? I'm happy to merge once that's done.
Should be good to go @willtebbutt |
|
Thanks for this. Merging + creating a release. |
823219b
into
JuliaGaussianProcesses:master
|
Out of curiosity in what context are you using a Matern 7/2 ? |
I am working with gradient-enhanced GPs, and I need to compute the Hessian of their posterior mean. The Matern 5/2 is not smooth enough in this case, as I require a third-order derivative :) @theogf |
Summary
Adding the
Matern72Kernelin the same spirit asMatern12Kernel,Matern32KernelandMatern52KernelProposed changes
Matern72Kernelimplementation insrc/basekernels/matern.jl.Matern72Kernelin the docs, and refering in theMaternKenelto this implementation.What alternatives have you considered?
Simply followed the previous implementations of
Matern12Kernel,Matern32KernelandMatern52Kernel. Didn't consider other alternatives.Breaking changes
None
Closes #579