MonoRec icon indicating copy to clipboard operation
MonoRec copied to clipboard

problem regarding depth_refinement_loss

Open bpwl0121 opened this issue 3 years ago • 1 comments

Hi Felix,

https://github.com/Brummi/MonoRec/blob/b2b3decc130d9ca333d1350096b9838a12f977a3/model/loss_functions/monorec_loss.py#L334 https://github.com/Brummi/MonoRec/blob/b2b3decc130d9ca333d1350096b9838a12f977a3/model/loss_functions/monorec_loss.py#L339 you use the .detach() method in the sdl at depth_refinement_loss, which cut the backpropagation, different from the depth_refinement_loss in the paper. Or did I misunderstand?

bpwl0121 avatar Mar 20 '22 19:03 bpwl0121

Hi bpwl0121,

thank you for looking into the code so thoroughly, I appreciate it a lot! I double checked and the code I used for the paper corresponds to the version in the paper.

But you seem to be correct here, I will look into it. I suspect I may have introduced a mistake when refactoring the code before publishing it.

Best, Felix

Brummi avatar Apr 06 '22 22:04 Brummi