JuliaGaussianProcesses / JuliaGaussianProcesses/KernelFunctions.jl
PeriodicKernel parameters pehaps a little un-intuitive
Nobody has claimed this yet.
- Dominant language
- Julia
- Stars
- 275
- Forks
- 41
- PR merge metrics
- No merged PRs in 30d
Description
The parameter r in our PeriodicKernel currently governs the size of the circle onto which the inputs are mapped. Would it not be more intuitive to parametrise it in terms of the frequency or period?
This might be a good opportunity to consider whether we actually need any parameters in the PeriodicKernel, since we can get everything that we need via transform, although we maybe want to ensure that that's wrapped for users.
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 locating the PeriodicKernel implementation and its transform entry point. Review how r controls the input mapping and whether transform already exposes the needed frequency or period behavior. Done means agreeing on the parameterization and providing a clear user-facing interface, including any needed wrapping for transform.
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
- 25/100