Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Why was SURF Vocab and Database Removed? #48

Open
marcusabate opened this issue Jun 20, 2019 · 0 comments
Open

Why was SURF Vocab and Database Removed? #48

marcusabate opened this issue Jun 20, 2019 · 0 comments

Comments

@marcusabate
Copy link

I'm noticing that 2bbc1a0 replaced SURF with ORB. However, the FSurf64 class still remains. Is there a reason this out-of-the-box support for SURF descriptors was removed? If I add those typdefs back in I can make/make-install with no problem. But when I then build a project that includes the DBoW2 library, I end up with build errors:

undefined reference to `DBoW2::FSurf64::distance(std::vector<float, std::allocator<float> > const&, std::vector<float, std::allocator<float> > const&)'
undefined reference to `DBoW2::FSurf64::toString[abi:cxx11](std::vector<float, std::allocator<float> > const&)'
...

I've cut out a lot of the error messages above but that's the main idea.
It continues on with similar 'undefined reference errors' for all the methods defined in FSurf64.h. But all these methods are defined in that file.

Is this related to the removal of SURF descriptor support in that commit?

I'm running OpenCV 3.3.1 on Ubuntu 18.04, and I've built OpenCV with opencv_contrib modules.

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

No branches or pull requests

1 participant