cf_tracking icon indicating copy to clipboard operation
cf_tracking copied to clipboard

Line 538-539 is useless in dsst_tracker.hpp

Open Superlee506 opened this issue 6 years ago • 1 comments

In dss_tracker.hpp, Line 538-539 seems useless. Do I miss anything?

boundingBox &= Rect(0, 0, static_cast<T>(image.cols), static_cast<T>(image.rows));
boundingBox = tempBoundingBox;

Superlee506 avatar Jun 19 '18 04:06 Superlee506

539 is useful as boundingBox is an out argument (aka. it's a return value, which is not set before) 538 does currently nothing. I am not sure if it's obsolete code or a bug (my first guess would be a bug, but that requires some analysis). https://github.com/klahaag/cf_tracking/blob/d31a248505fa7fc0c6d321d1f57caae43ac30568/src/cf_libs/kcf/kcf_tracker.hpp#L649 is also affected

myro avatar Jun 19 '18 17:06 myro