Apply suggestions from code review by @devmotion

Co-Authored-By: David Widmann <[email protected]>

Author: sharanry

src/kernels/gabor.jl

@@ -1,5 +1,3 @@
-import Base.getproperty
-
 """
     GaborKernel(; ell::Real=1.0, p::Real=1.0)
 
@@ -10,12 +8,10 @@ Gabor kernel with length scale ell and period p. Given by
 
 """
 struct GaborKernel{T<:Real, K<:Kernel} <: BaseKernel
-    κ::K
+    kernel::K
     function GaborKernel(;ell=nothing, p=nothing)
         k = _gabor(ell=ell, p=p)
-        if ell == nothing ell=1.0 end
-        if p == nothing p=1.0 end
-        new{Union{typeof(ell),typeof(p)}, typeof(k)}(k)
+        new{typeof(k)}(k)
     end
 end
 
@@ -36,14 +32,20 @@ end
 function Base.getproperty(k::GaborKernel, v::Symbol)
     if v == :κ 
         return getfield(k, v)
-    elseif v == :ell && typeof(k.κ.kernels[1]) <: SqExponentialKernel
-        return 1.0
-    elseif v == :ell && typeof(k.κ.kernels[1]) <: TransformedKernel
-        return 1 ./ k.κ.kernels[1].transform.s[1]
-    elseif v == :p && typeof(k.κ.kernels[2]) <: CosineKernel
-        return 1.0
-    elseif v == :p && typeof(k.κ.kernels[2]) <: TransformedKernel
-        return 1 ./ k.κ.kernels[2].transform.s[1]
+    elseif v == :ell
+        kernel1 = k.κ.kernels[1]
+    	if kernel1 isa TransformedKernel
+    	    return 1 ./ kernel1.transform.s[1]
+    	else
+    	    return 1.0
+    	end
+    elseif v == :p
+        kernel2 = k.κ.kernels[2]
+        if kernel2 isa TransformedKernel
+            return 1 ./ kernel2.transform.s[1]
+        else
+            return 1.0
+        end
     else
         error("Invalid Property")
     end