Will Tebbutt

Results 139 issues of Will Tebbutt

There are currently instances when we want to use arrays of zeros in which size information is retained - this is not something supported by the `Zero` differential, but which...

performance
design

Lots has changed since the docs were first written. #152 addresses a number of things, but there are a few more things that we might want to consider: - changing...

good first issue

Firstly, thanks for making this fantastic package! It's great to have the various quadrature / cubature packages accessible via a common interface. The docs are currently a little harder to...

Are the maintainers of this package open to `ChainRules`-centric PRs? ie. are you happy to host "AD recipes" in this package? The consensus is that the "correct" place for rules...

While the example in the README is important in that it exposes all of the useful bits of AdvancedHMC, it would be nice if it were possible to define a...

enhancement
user-friendly

We have decent enough interface tests in this package, but we don't have good choices for test fakes. See [this blog](https://invenia.github.io/blog/2020/11/06/interfacetesting/) for a description of what they are / why...

https://github.com/JuliaGaussianProcesses/KernelFunctions.jl/pull/425 makes it possible to AD through the `MaternKernel` by dropping the derivative w.r.t. `\nu`. We tried ensuring that it returns a `NotImplemented`, but Zygote doesn't appear to handle it...

We currently return a view to the underlying data when `getindex` is called on a `ColVecs`. See https://github.com/JuliaGaussianProcesses/KernelFunctions.jl/blob/2d172125e72ca932133266c3b23d0865bec539df/src/utils.jl#L76 @theogf was the last person to touch this bit of code, but...

# Proposal 1 ```julia # Euclidean domain with D dimensions. struct Euclidean D::Int end dim(domain::Euclidean) = domain.D """ DomainKernel{Tkernel

This is more related to the last MR, but should we be providing any explanation as to why `Y` being a `ColVecs` or `RowVecs` affects the output? It looks like...