hypernerf icon indicating copy to clipboard operation
hypernerf copied to clipboard

Problem with Geman-McClure loss implementation

Open JokerYan opened this issue 3 years ago • 2 comments

I have been testing the Geman-McClure loss implemented in the repo, more specifically the model_utils.general_loss_with_squared_residual() with alpha = -2.

However, the output, as shown below, is very different from my expectation. (alpha = -2, scale = 0.1) image Shouldn't it be like the quadratic shape when |x| < scale

JokerYan avatar Aug 12 '22 02:08 JokerYan

Did you square the residual before passing it to the function?

keunhong avatar Aug 12 '22 07:08 keunhong

No, I didn't. I think that is the problem. Thank you very much!

JokerYan avatar Aug 12 '22 10:08 JokerYan