search

facebookresearch / TemporallyConsistentDepth

Code for our CVPR 2023 paper on online, temporally consistent depth estimation.
Other
86 stars 3 forks source link

name 'corr_sampler' is not defined #3

Closed TouqeerAhmad closed 1 year ago

TouqeerAhmad commented 1 year ago

Hello,

I was trying to run the code using MPI dataset and facing this error: "name 'corr_sampler' is not defined". It is triggering due to RAFT Stereo, also an open issue on RAFT repo: https://github.com/princeton-vl/RAFT-Stereo/issues/80

May be instructions for this repo need to be updated as how to run out of box without facing this error? As currently it is not running due to one of the dependency.

Thanks!

nkhan2 commented 1 year ago

Thanks for pointing this out. I'll push a fix soon. In the meantime, you can modify the options on line 34 of raftstereo_wrapper.py to run RAFT Stereo without the CUDA sampler.

TouqeerAhmad commented 1 year ago

Thank you, that worked -- looking forward to the fix.