Merge pull request #126 from JuliaGaussianProcesses/fix_maha

Replace constructor of Maha with keyword one

Author: theogf

src/basekernels/maha.jl

@@ -10,7 +10,7 @@ where the matrix P is the metric.
 """
 struct MahalanobisKernel{T<:Real, A<:AbstractMatrix{T}} <: SimpleKernel
     P::A
-    function MahalanobisKernel(P::AbstractMatrix{T}) where {T<:Real}
+    function MahalanobisKernel(; P::AbstractMatrix{T}) where {T<:Real}
         LinearAlgebra.checksquare(P)
         new{T,typeof(P)}(P)
     end

test/basekernels/maha.jl

@@ -5,12 +5,12 @@
     v2 = rand(rng, 3)
 
     P = rand(rng, 3, 3)
-    k = MahalanobisKernel(P)
+    k = MahalanobisKernel(P=P)
 
     @test kappa(k, x) == exp(-x)
     @test k(v1, v2) ≈ exp(-sqmahalanobis(v1, v2, P))
     @test kappa(ExponentialKernel(), x) == kappa(k, x)
     @test repr(k) == "Mahalanobis Kernel (size(P) = $(size(P)))"
-    # test_ADs(P -> MahalanobisKernel(P), P)
+    # test_ADs(P -> MahalanobisKernel(P=P), P)
     @test_broken "Nothing passes (problem with Mahalanobis distance in Distances)"
 end