You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
with: $k(x,y) = <\phi(x),\phi(y)>$, $k$ being a kernel and $\phi$ a transformation.
It seems to me that in your implementation, you use the form with $\phi$ (which corresponds to y_mapping) rather than using the form with a kernel.
Suggested solution
As you mentioned in the docs, MMD is a "kernel-based method", so I would suggest to use the kernel form and replace the y_mapping argument with a kernel argument. It would be more consistent with common MMD implementations I found ([1], [2], [3] or [4]).
I would be happy to make the changes if you agree!
The text was updated successfully, but these errors were encountered:
Problem
I would like to use Maximum Mean Discrepancy metric with a kernel (e.g. gaussian kernel).
According to the reference paper, we have (eq. 3):
with:$k(x,y) = <\phi(x),\phi(y)>$ , $k$ being a kernel and $\phi$ a transformation.
It seems to me that in your implementation, you use the form with$\phi$ (which corresponds to
y_mapping
) rather than using the form with a kernel.Suggested solution
As you mentioned in the docs, MMD is a "kernel-based method", so I would suggest to use the kernel form and replace the
y_mapping
argument with akernel
argument. It would be more consistent with common MMD implementations I found ([1], [2], [3] or [4]).I would be happy to make the changes if you agree!
The text was updated successfully, but these errors were encountered: