-
Notifications
You must be signed in to change notification settings - Fork 128
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Negative constant kernel #125
Comments
arodland@dae7293 seems to work fine although I'm not sure if it's the best approach. |
Did you have any specific comment? I have exactly the same problem... :) essentially how to model processes with a (partially, at least) negative ACF. Stefano |
@stefanocovino my branch is working 100% fine for me. Just hoping for an opinion from @dfm . |
My first, naive, feeling is only that mathematically it could work, however by itself a negative kernel is probably not a kernel, at least according to Eq. 4.2 in Rasmussen and Williams. O course if you are just summing it to some other kernel what counts is the total sum that might well satisfy Eq. 4.2, possibly limiting the value of the constant. My 2 cents. Bye, |
@stefanocovino yes, you're absolutely right, a negative constant is certainly not a kernel by that definition, but I don't care too much for terminological purity, and positive-semi-definiteness does get enforced on the final result when the solver tries to |
Considering that george supports the CosineKernel it probably can cope with negative covariances. My model instead uses an ExpSine2Kernel offset by a negative constant. But since ConstantKernel is parameterized by the log of the constant, it's impossible to specify a negative constant. I tried using
-1 * ConstantKernel(abs_value)
but that doesn't work either, probably since the*
overload just constructs aConstantKernel
anyway.The text was updated successfully, but these errors were encountered: