void DescriptorMatcher::radiusMatch() // description unclear (Bug #2967)


Added by Alexander Kleinsorge almost 12 years ago. Updated over 11 years ago.


Status:Done Start date:2013-04-16
Priority:Normal Due date:
Assignee:Alexander Kleinsorge % Done:

0%

Category:documentation
Target version:2.4.6
Affected version:2.4.5 (latest release) Operating System:
Difficulty: HW Platform:
Pull request:https://github.com/Itseez/opencv/pull/903

Description

ORIGINAL 2.4.5 DOCU
void DescriptorMatcher::radiusMatch()
...
maxDistance – Threshold for the distance between matched descriptors.

For each query descriptor, the methods find such training descriptors that the distance between the query descriptor and the training descriptor is equal or smaller than maxDistance. Found matches are returned in the distance increasing order.
----------------------------------------
'distance' has 2 meanings. If you do optical flow using descriptors, your first guess is coordinate_distance, even knowing that there is a descriptor_distance related to the function.
I know 2 people also woundering about it and loosing 1 hour - same happed to me.
So I would write:
maxDistance – Threshold for the distance between matched descriptors. This does not mean the distance between coordinates which is measured in Pixels!

thanks.


Associated revisions

Revision faac7f18
Added by Vadim Pisarevsky over 10 years ago

Merge pull request #2967 from LeonidBeynenson:bugfix_wave_correction_wrong_result

History

Updated by Anna Kogan almost 12 years ago

Hello Alexander,
Thank you for reporting the issue. If you could fix the issue on your side, a pull request in our GitHub repo would be highly appreciated!

  • Assignee changed from Vadim Pisarevsky to Alexander Kleinsorge

Updated by Alexander Kleinsorge almost 12 years ago

https://github.com/Itseez/opencv/pull/901
(3 trivial bugs together, only small trivial changes)

Updated by Vladislav Vinogradov over 11 years ago

Fix was merged into 2.4 branch.

Alexander, thanks for your participation.

  • Estimated time deleted (0.25)
  • Status changed from Open to Done
  • Pull request set to https://github.com/Itseez/opencv/pull/903

Also available in: Atom PDF