search

stschubert / VPR_Tutorial

GNU General Public License v3.0
160 stars 19 forks source link

Plans to add Sum-of-Absolute Differences (SAD) method? #10

Closed Somayeh-h closed 11 months ago

Somayeh-h commented 11 months ago

Hi @stschubert,

Is it possible if you could please also add the Sum-of-Absolute Differences [1] VPR method?

[1] Milford, M.J. and Wyeth, G.F., 2012, May. SeqSLAM: Visual route-based navigation for sunny summer days and stormy winter nights. In 2012 IEEE international conference on robotics and automation (pp. 1643-1649). IEEE.

Thanks, Somayeh

stschubert commented 11 months ago

Hi @Somayeh-h,

Nice to hear from you. Sure, I added an implementation of the SAD "descriptor" (grayscale, downsampled, patch-normalized image) and the corresponding distance metric (the actual sum of absolute differences).

I'm looking forward to your feedback after testing.

Best, Stefan

Somayeh-h commented 11 months ago

Hi @stschubert,

Thank you so much for your quick integration of SAD to the repo. Yes, I can confirm it works as expected.

Thanks, Somayeh