NNLM icon indicating copy to clipboard operation
NNLM copied to clipboard

Feature request: supporting MAE loss

Open XaXaXa12345 opened this issue 4 years ago • 2 comments

Dear NNLM Team, Your package is great. Actually, at the moment this is only package supporting missing values. It would be nice to have absolute loss |x| supported as well. Kind Regards. Andrey.

XaXaXa12345 avatar Apr 07 '20 07:04 XaXaXa12345

I'm not sure why you'd like to use absolute loss rather than mean squared loss. NMF is not convex in absolute error, because NNLS is not convex in absolute error, only in squared error (hint, we are solving least SQUARES equations). Thus, absolute error can be deceiving because the exact absolute error solution can be different than the convex NNLS solution solved by least squares.

zdebruine avatar Mar 06 '21 15:03 zdebruine

Squared error loss for matrix factorization isn't convex either, unless regularized, depending on regularization parameter. Take a hessian and see that it's not PSD in general case. Absolute loss is resistant to outliers, since it's median.

XaXaXa12345 avatar Mar 08 '21 07:03 XaXaXa12345