From c22c92e9ba4c5d3cec4df5c28996fb2af409b5a2 Mon Sep 17 00:00:00 2001 From: Felix Benning Date: Mon, 5 Jun 2023 17:23:04 +0200 Subject: [PATCH] push broken state (upstream issue) https://github.com/JuliaGaussianProcesses/KernelFunctions.jl/issues/517 --- test/diffKernel.jl | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/diffKernel.jl b/test/diffKernel.jl index 647c25f..32e642d 100644 --- a/test/diffKernel.jl +++ b/test/diffKernel.jl @@ -15,8 +15,11 @@ ] k = EnableDiff(k1) for x in [0, 1, -1, 42] - # correlation with self should be positive - @test k((x, partial(1)), (x, partial(1))) > 0 skip = true + # correlation with self should be positive + ## This fails for Matern and RationalQuadraticKernel + # because its implementation branches on x == y resulting in a zero derivative + # (cf. https://github.com/JuliaGaussianProcesses/KernelFunctions.jl/issues/517) + @test k((x, partial(1)), (x, partial(1))) > 0 # the slope should be positively correlated with a point further down @test k(