Incorrect formula in calcOpticalFlowFarneback documentatoin (OpenCV 2.x C++ API) (Bug #1172)

Added by Jacek Komorowski over 13 years ago. Updated over 13 years ago.

Status:Done Start date:
Priority:Normal Due date:
Assignee:- % Done:

0%
Category:documentation
Target version:-
Affected version: Operating System:
Difficulty: HW Platform:
Pull request:

Description

In calcOpticalFlowFarneback documentation (OpenCV 2.x C++ API) there's incorrect formula relating prevImg, nextImg and calculated optical flow.

Formula is: prevImg(x,y) = nextImg( flow(x,y)r0, flow(x,y)r1 )

But flow array seems to be a relative flow (we must add previous coordiante to get absolute position). Additionally first image coordinate is y not x.

Correct formula is:
prevImg(y,x) = nextImg( y + flow(y,x)r0, x + flow(y,x)r1 )

Associated revisions

Revision 6a77bfb5
Added by Andrey Pavlenko over 11 years ago

Merge pull request #1172 from StevenPuttemans:bugfix_3166_24

History

#1
Updated by Vadim Pisarevsky over 13 years ago

Thanks, the formulae has been corrected in 2.3 branch and in the trunk. Updated PDFs are included into 2.3 distribution.

  • Status changed from Open to Done
  • (deleted custom field) set to fixed

Also available in: Atom PDF