JuliaGaussianProcesses / JuliaGaussianProcesses/KernelFunctions.jl
Adding `AbstractKernel`
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 275
- Forks
- 41
- PR merge metrics
- No merged PRs in 30d
Description
This might be a controversial point, but I have a few times been surprised by the fact that Kernel is an abstract type.
I would like to propose/ consider adding AbstractKernel, with the subtypes SimpleKernel, MOKernel and Kernel, where Kernel would further be the supertype of the various kernels that are not mentioned.
This would be in line with how kernelmatrix is defined, which is specialized for SimpleKernels and for all concrete instances of MOKernels, but falls back to kernel.(x, permutedims(y)) for Kernels, which should continue.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the existing Kernel hierarchy and the kernelmatrix definitions mentioned in the issue. Check how SimpleKernel, MOKernel, and other concrete kernels are currently represented and used. Done means reaching agreement on the proposed AbstractKernel hierarchy and updating the affected behavior consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- machine-learning
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100