rust-cv / cv

Rust CV mono-repo. Contains pure-Rust dependencies which attempt to encapsulate the capability of OpenCV, OpenMVG, and vSLAM frameworks in a cohesive set of APIs.
792 stars 62 forks source link

Akaze: Check on a large dataset that the boundary check is not necessary #73

Open stephanemagnenat opened 1 year ago

stephanemagnenat commented 1 year ago

This is a follow-up of #63, in which an out-of-bound check was added (86b70fec024201ec91ac8ddcb3e83231bbc90903), but looks now to be not necessary after the algorithm was fixed to match the original C++ code (822a3c13d5c72d6e3cca77bd176e1f6926121c0b).

@vadixidav this issue tracks the validation that the boundary check is not necessary any more, as you wanted to run Akaze a large dataset of yours to make sure of that.