STOR-i / GaussianProcesses.jl

A Julia package for Gaussian Processes
https://stor-i.github.io/GaussianProcesses.jl/latest/
Other
308 stars 53 forks source link

Tests failing #107

Closed tlienart closed 5 years ago

tlienart commented 5 years ago
julia> versioninfo()
Julia Version 1.2.0-DEV.169
Commit 7ed9e4fb35 (2019-01-14 22:06 UTC)
Platform Info:
  OS: macOS (x86_64-apple-darwin14.5.0)
  CPU: Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.1 (ORCJIT, skylake)
Environment:
  JULIA_NUM_THREADS = 2

I added GaussianProcesses via ] add GaussianProcesses#master

Truncated stacktrace:

Test Summary: | Pass  Total
Means         |   36     36
  1.975721 seconds (5.24 M allocations: 262.767 MiB, 3.70% gc time)
    Testing SEIso...
    Testing Mat12Iso...
    Testing Mat32Iso...
    Testing Mat52Iso...
    Testing RQIso...
    Testing Periodic...
    Testing LinIso...
    Testing Poly...
    Testing Noise...
    Testing Const...
    Testing SEArd...
    Testing Mat12Ard...
    Testing Mat32Ard...
    Testing Mat52Ard...
    Testing RQArd...
    Testing LinArd...
    Testing SumKernel...
    Testing SumKernel...
    Testing ProdKernel...
    Testing ProdKernel...
    Testing FixedKernel...
    Testing FixedKernel...
    Testing SumKernel...
    Testing maskedSEIso...
    Testing masked Masked...
Variance: Error During Test at /Users/tlienart/.julia/packages/GaussianProcesses/f03ek/test/kernels.jl:34
  Test threw exception
  Expression: spec[i, j] ≈ cov(kern, Xi, Xj)
  ArgumentError: unable to check bounds for indices of type StaticArrays.StaticIndexing{StaticArrays.SArray{Tuple{1},Int64,1,1}}
  Stacktrace:
   [1] checkindex(::Type{Bool}, ::Base.OneTo{Int64}, ::StaticArrays.StaticIndexing{StaticArrays.SArray{Tuple{1},Int64,1,1}}) at ./abstractarray.jl:506
   [2] checkbounds at ./abstractarray.jl:436 [inlined]
   [3] checkbounds at ./abstractarray.jl:451 [inlined]
   [4] view at ./subarray.jl:155 [inlined]
   [5] cov(::GaussianProcesses.Masked{GaussianProcesses.SEIso{Float64},1}, ::SubArray{Float64,1,Array{Float64,2},Tuple{Base.Slice{Base.OneTo{Int64}},Int64},true}, ::SubArray{Float64,1,Array{Float64,2},Tuple{Base.Slice{Base.OneTo{Int64}},Int64},true}) at /Users/tlienart/.julia/packages/GaussianProcesses/f03ek/src/kernels/masked_kernel.jl:70
   [6] macro expansion at /Users/tlienart/.julia/packages/GaussianProcesses/f03ek/test/kernels.jl:34 [inlined]
   [7] macro expansion at /Users/osx/buildbot/slave/package_osx64/build/usr/share/julia/stdlib/v1.2/Test/src/Test.jl:1084 [inlined]
   [8] testkernel(::GaussianProcesses.Masked{GaussianProcesses.SEIso{Float64},1}) at /Users/tlienart/.julia/packages/GaussianProcesses/f03ek/test/kernels.jl:32
Covariance: Error During Test at /Users/tlienart/.julia/packages/GaussianProcesses/f03ek/test/kernels.jl:47

(and it goes on). FWIW on Julia 1.0.3 I get the same issue.

maximerischard commented 5 years ago

Thank you for reporting this. It looks like there's been some changes in StaticArrays that we need to adjust to. I've just pushed a commit that caps StaticArrays at v0.9.2, which should fix your issue for now.