Ask Your Question

BinaryDescriptorMatcher matching Mask not working as specified in the Documentation

asked 2017-04-07 08:27:45 -0500

leunam gravatar image

The Mask does not work as the documentation states: ( ) Optional mask (or masks) can be passed to specify which query and training descriptors can be matched. Namely, queryDescriptors[i] can be matched with trainDescriptors[j] only if<uchar>(i,j) is non-zero.

But the implementation only lets you choose which queryDescriptors shall be matched (against all trainDescriptors, no choice against which of them).


Am I mistaken or is the documentation inconsistent with the implementation. The documentations states what would actually be desirable here.

edit retag flag offensive close merge delete

1 answer

Sort by ยป oldest newest most voted

answered 2017-04-07 08:46:56 -0500

berak gravatar image

you're right, that's a typo, it probably should be:

queryDescriptors[i] can be matched with trainDescriptors[j] only if<uchar>(i,0) is non-zero.
edit flag offensive delete link more


Oh, that's sad - it changes the behaviour. You can not prefilter which descriptor should be matched against which, only exclude some totally, which could also be done by just leaving them out of the query vector. I didn't read the multi index hashing paper - maybe it has the disadvantage that you can only always match against all trainDescriptors?

This should be fixed in the Documentation, at multiple places... Who can fix the wrong Documentation?

leunam gravatar imageleunam ( 2017-04-07 12:42:10 -0500 )edit

again, you cannot filter on the train descriptors, only on the query ones. imho, you simply misunderstood that

berak gravatar imageberak ( 2017-04-10 12:45:46 -0500 )edit

I got that -> So I think the Documentation should be corrected. Who is able to do that?

leunam gravatar imageleunam ( 2017-04-13 08:39:06 -0500 )edit
berak gravatar imageberak ( 2017-04-13 08:40:51 -0500 )edit

Question Tools

1 follower


Asked: 2017-04-07 08:24:46 -0500

Seen: 120 times

Last updated: Apr 07 '17