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

Replace vl-hog with a modern C++ solution #6

Open
patrikhuber opened this issue Jun 9, 2015 · 2 comments
Open

Replace vl-hog with a modern C++ solution #6

patrikhuber opened this issue Jun 9, 2015 · 2 comments

Comments

@patrikhuber
Copy link
Owner

vl-hog has really good performance and it's hard to replace. But I don't like to include C headers and I would really like to replace it with a header-only, modern C++ solution. It's just hard to find one, and rewriting vl-hog would be a lot of work.

I managed to include vl-hog as "header-only" by #include'ing vl-hog's .c file, and I had to make some casts in its code explicit in the process to get it to compile. I hope what I did is not problematic.

@ghost
Copy link

ghost commented Feb 10, 2017

Did you try the opencv HOG implementation ? since you already use opencv...

@patrikhuber
Copy link
Owner Author

I've had quite bad performance with OpenCV's HOG implementation when I tried it back then. It was either accuracy, speed, or both.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant