gpytorch icon indicating copy to clipboard operation
gpytorch copied to clipboard

PRs for custom kernels?

Open jpchen opened this issue 2 years ago • 7 comments

We have a few additional custom kernels we've implemented for a project that I think would be better served upstream (eg white noise, changepoint, etc). Would you welcome PRs for these or would you prefer bespoke kernels to not clutter the kernels library?

jpchen avatar Sep 10 '21 06:09 jpchen

@jpchen I think generally the answer is yes, happy to consider if they are more generally useful. Can you point me to the implementations?

Re the white noise kernel, we used to have one but deprecated that in favor of having this be part of the likelihood - does this need to be a proper kernel for your use cases?

Balandat avatar Sep 11 '21 00:09 Balandat

They are internal atm, but I can open some PRs once I port tests over and defer to you on which are worth adding. We used a white noise kernel as a multiplicative component in a composite kernel.

jpchen avatar Sep 18 '21 01:09 jpchen

I can also look at the internal ones :)

Balandat avatar Sep 18 '21 15:09 Balandat

Will follow up over chat, probably won't get around to this in the next few weeks :)

jpchen avatar Sep 20 '21 08:09 jpchen

@jpchen Is a PR still in the works for the change point kernel? I have a project where this kernel could be of use and I will only develop myself if this PR won't happen in the foreseeable future. Thanks!

GStechschulte avatar May 08 '22 10:05 GStechschulte

Hi @GStechschulte, I have a version locally but it doesn't yet support batching (and thus probably won't pass the unit tests) and I havent had the bandwidth to do so. I'm happy to put up a PR you can add to or perhaps reference if that's helpful?

jpchen avatar May 09 '22 22:05 jpchen

Hi @GStechschulte, I have a version locally but it doesn't yet support batching (and thus probably won't pass the unit tests) and I havent had the bandwidth to do so. I'm happy to put up a PR you can add to or perhaps reference if that's helpful?

I appreciate the quick reply @jpchen. That would be great if you could put up a PR to which I can reference. Then, my plan would be to build off of your work and add batching. Thanks!

GStechschulte avatar May 10 '22 04:05 GStechschulte