JuliaGaussianProcesses / JuliaGaussianProcesses/KernelFunctions.jl
Proposal: Remove IndependentMOKernel and IntrinsicCoregionMOKernel
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 275
- Forks
- 41
- PR merge metrics
- No merged PRs in 30d
Description
Copied from Slack:
BTW I propose to remove the IndependentMOKernel ane the IntrinsicCoregionMOKernel since they are just KernelTensorProducts with a WhiteKernel and a yet to define MatrixKernel (maybe a bad name but generally useful IMO). Of course, we can still keep a function that constructs them in a convenient way, as we do for other single-output kernels. And one could also specialize kernelmatrix for them if needed.
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 definitions and uses of IndependentMOKernel and IntrinsicCoregionMOKernel, then compare them with KernelTensorProducts, WhiteKernel, and the proposed MatrixKernel. Check the existing kernelmatrix behavior and constructor patterns; the work is done when the removal or replacement strategy, convenient constructors, and any needed kernelmatrix specialization are resolved.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- machine-learning
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100