Skip to content
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

problem regarding depth_refinement_loss #27

Open
bpwl0121 opened this issue Mar 20, 2022 · 1 comment
Open

problem regarding depth_refinement_loss #27

bpwl0121 opened this issue Mar 20, 2022 · 1 comment

Comments

@bpwl0121
Copy link

Hi Felix,

mono_sdl = mask_mean(mono_sparsedl.detach(), mono_sparsedl_mask)

stereo_sdl = mask_mean(mono_stereodl.detach(), mono_stereodl_mask)

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?

@Brummi
Copy link
Owner

Brummi commented Apr 6, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants