Skip to content

Commit

Permalink
Enable periodic kernel in example
Browse files Browse the repository at this point in the history
Now that JuliaGaussianProcesses/KernelFunctions.jl#528 has been merged, we can try re-enabling the `PeriodicKernel`
  • Loading branch information
simsurace authored Feb 10, 2024
1 parent cf311bd commit d4ad9f2
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions examples/1-mauna-loa/script.jl
Original file line number Diff line number Diff line change
Expand Up @@ -100,9 +100,7 @@ plotdata()
# We define a couple of helper functions to simplify the kernel construction:

SE(θ) = θ.σ^2 * with_lengthscale(SqExponentialKernel(), θ.ℓ)
## PeriodicKernel is broken, see https://github.com/JuliaGaussianProcesses/KernelFunctions.jl/issues/389
##Per(θ) = with_lengthscale(PeriodicKernel(; r=[θ.ℓ/2]), θ.p) # NOTE- discrepancy with GaussianProcesses.jl
Per(θ) = with_lengthscale(SqExponentialKernel(), θ.ℓ) PeriodicTransform(1 / θ.p)
Per(θ) = with_lengthscale(PeriodicKernel(; r=./2]), θ.p) # NOTE- discrepancy with GaussianProcesses.jl
RQ(θ) = θ.σ^2 * with_lengthscale(RationalQuadraticKernel(; α=θ.α), θ.ℓ)
#md nothing #hide

Expand Down

0 comments on commit d4ad9f2

Please sign in to comment.