Add Gaussian Processes - #334
Conversation
…aintyQuantification.jl into gaussian-processes
…yQuantification.jl into gaussian-processes
…aintyQuantification.jl into gaussian-processes
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Setting sigma^2=0.0 can lead to numerical instabilities. Thus, I changed the default value to 1e10 and added a warning the the input check when sigma^2 is not optimized and it is smaller than `eps()`.
|
I started some work on adaptive GPs based on the current state of the GP implementation. Do we want to add this in this PR or later? |
Let's do the base here and the refinement in a second step. |
Start adding adaptive GPs with several learning functions, test and docs; add random restarts to hyperparameter optimization; add Base.show methods for GaussianProcess for nicer printing; and some small fixes to GP examples and to tests for stability (e.g., not setting variance = 0 and removing ARDTransform form Himmelblau example)
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 25 out of 26 changed files in this pull request and generated 6 comments.
Suppressed comments (3)
src/models/gp/gaussianprocess.jl:126
- Docstring says
σ²defaults to0.0, but the constructor default isσ²::Float64 = 1.0e-10. Please keep the documented default consistent with the implementation.
- `σ²`: The noise variance. Defaults to 0.0.
src/models/gp/gaussianprocess.jl:208
- Docstring says
σ²defaults to0.0, but the constructor default isσ²::Float64 = 1.0e-10. This should be aligned for accurate docs.
- `σ²`: The noise variance. Defaults to 0.0.
src/models/gp/gaussianprocess.jl:280
- Docstring says
σ²defaults to0.0, but the constructor default isσ²::Float64 = 1.0e-10. Please update the docs or the default.
- `σ²`: The noise variance. Defaults to 0.0.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
The tests in Also, I'm unsure how the |
This PR replaces #245 with one originating from within the repository.
There is still some work needed to clean the code: