Skip to content

Make TV-L1 rescaling flexible and add median filtering [with GPU and test updates]#724

Merged
opencv-pushbot merged 3 commits intoopencv:masterfrom
vinograd47:tvl1-updates
Apr 8, 2013
Merged

Make TV-L1 rescaling flexible and add median filtering [with GPU and test updates]#724
opencv-pushbot merged 3 commits intoopencv:masterfrom
vinograd47:tvl1-updates

Conversation

@vinograd47
Copy link
Copy Markdown
Contributor

This is an extended version of #693

I used original commit and updated GPU implementation and test data.
Now test on buildbot must pass.

@vinograd47
Copy link
Copy Markdown
Contributor Author

@taka-no-me Andrey, Could you review this pull request?

If we merge this PR we can close #693.

@ghost ghost assigned taka-no-me Apr 1, 2013
@taka-no-me
Copy link
Copy Markdown

👍 However it will wait for the next merge of 2.4 into master. Because b34c2a9 is a duplicate of 924b0ef already merged to 2.4.

Stefan Walk and others added 3 commits April 8, 2013 15:18
Previously the pyramid was done with a rescaling factor of 2 (implied by the
use of pyrDown). This often leads to inferior results compared to a scale step
of e.g. 0.8 (a factor of 2 is obviously faster). This commit makes the scale
step configurable and uses a resonable default value.

The other change in this commit is that median filtering is added. This is not
described in this paper but it is done in the author's implementation. (See
e.g. "Secrets of optical flow estimation and their principles", Sun et al.,
CVPR 2010) This serves as periodic outlier removal during optimization, leading
to smoother flow fields while preserving motion edges. This includes splitting
the optimization loop into two loops.
@vinograd47
Copy link
Copy Markdown
Contributor Author

@taka-no-me I fixed merge conflicts and removed duplicate commit. Now this pull request can be merged.

taka-no-me pushed a commit that referenced this pull request Apr 8, 2013
@opencv-pushbot opencv-pushbot merged commit c89786b into opencv:master Apr 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants