Applied suggestions

Author: theogf

src/utils.jl

@@ -9,10 +9,7 @@ macro check_args(K, param, cond, desc=string(cond))
     end
 end
 
-
-# abstract type VecOfVecs{T, TX <: AbstractMatrix{T}, S} <: AbstractVector{S} end
-
-function vec_of_vecs(X::AbstractMatrix, obsdim)
+function vec_of_vecs(X::AbstractMatrix; obsdim::Int = 2)
     @assert obsdim ∈ (1, 2) "obsdim should be 1 or 2"
     if obsdim == 1
         RowVecs(X)

test/utils.jl

@@ -1,10 +1,10 @@
 @testset "utils" begin
-    using KernelFunctions: VecOfVecs, ColVecs, RowVecs
+    using KernelFunctions: vec_of_vecs, ColVecs, RowVecs
     rng, N, D = MersenneTwister(123456), 10, 4
     x, X = randn(rng, N), randn(rng, D, N)
     @testset "VecOfVecs" begin
-        @test vec_of_vecs(X, 2) == ColVecs(X)
-        @test vec_of_vecs(X, 1) == RowVecs(X)
+        @test vec_of_vecs(X, obsdim = 2) == ColVecs(X)
+        @test vec_of_vecs(X, obsdim = 1) == RowVecs(X)
     end
     # Test Matrix data sets.
     @testset "ColVecs" begin