facebookresearch / vggsfm

VGGSfM: Visual Geometry Grounded Deep Structure From Motion
Other
819 stars 47 forks source link

run error #31

Closed luoshuiyue closed 4 weeks ago

luoshuiyue commented 1 month ago

why I get this error when run demo.py: 1721632014677

jytime commented 1 month ago

Hi,

This does not show where the error is. Can you type exprt HYDRA_FULL_ERROR=1 in you terminal and rerun the demo.py again?

luoshuiyue commented 1 month ago

I set HYDRA_FULL_ERROR=1,then run the command again: 1721720614883

jytime commented 1 month ago

Hi this seems your downloaded file is not complete.

Check this post for more details: https://stackoverflow.com/questions/71617570/pytorchstreamreader-failed-reading-zip-archive-failed-finding-central-directory

surabhi96 commented 3 weeks ago

Hi @jytime thanks for the amazing work! I was getting a similar error. This is my o/p after setting export HYDRA_FULL_ERROR=1. Also, seems like there was no convergence. I also get the following warning before the BA routine. Not sure if it's relevant but posting it as well. Thanks! image image

jytime commented 3 weeks ago

oh this seems all the frames are filtered out due to

https://github.com/facebookresearch/vggsfm/blob/46570ccfc4978e485f3a0a2e33483e243face9de/vggsfm/runners/runner.py#L496-L509

Can you try to set filter_invalid_frame=False and see the results?

surabhi96 commented 3 weeks ago

Thanks for your prompt reply! I don't get errors anymore but seems like there is still no convergence? For reference, I am feeding it 10 sequential images with a linearly moving camera with enough overlap between images i.e. the translation between two camera positions would be about 10 cm. image

jytime commented 3 weeks ago

Hey @surabhi96

No convergence is okay here because it shows the optimisation result for last BA. A final cost of 0.18 seems not bad here. You can visualize the results by gr_visualize=True.

If it still does not work, feel free to send the images to me.