Contribute: Adding FHOGDescriptor [closed]

asked 2015-09-04 14:13:23 -0600

joaofaro gravatar image

Hi! I'm trying to add FHOG descriptor but the buildbot returned an error while building for Mac and IOS.

The report showed the error: error: no member named 'FHOGDescriptor' in namespace 'cv'

This is strange because FHOGDescriptor is defined inside the namespace. Moreover, the other builds did not return any other error.

I will appreciate if anyone can help me with that,

PR: http://pullrequest.opencv.org/#/summa...

Log file: http://pullrequest.opencv.org/buildbo...

edit retag flag offensive reopen merge delete

Closed for the following reason the question is answered, right answer was accepted by sturkmen
close date 2018-02-04 02:42:15.505674

Comments

  • do use cv::Mat consistently, they won't pull in anything dependant on IplImage, CvMat or in general - the outdated c-api (check headers as well, anything with _c.h is probably not welcome)

  • you declared cv::FHOGDescriptor::CvLSVMFeatureMapCaskade in "opencv2/objdetect.hpp" , so you will need to include that in fhogtools.hpp, too. (buildbot error, precompiled headers work mysteriously different on different os'es) (totally minor nitpicking - cascade)

  • i'm most probably wrong about this, but - there was a latentsvm implementation previousy (in opencv_contrib) , which just (a week ago) got removed in favour of a dpm version (can you shed some light on how those are related ?)

berak gravatar imageberak ( 2015-09-04 14:23:46 -0600 )edit

I'm not using latentsvm, I've just adapted some functions that were used in latentsvm. However, I tried to keep it the closest possible to the original.

I will add the objectdetect.hpp and see if it works.

Thank you!

joaofaro gravatar imagejoaofaro ( 2015-09-04 14:32:08 -0600 )edit

Once again. just saying you should probably pull all of this to the contrib repo

LorenaGdL gravatar imageLorenaGdL ( 2015-09-04 15:26:43 -0600 )edit
1

sorry for not being clear enough.

"opencv2/objdetect.hpp" , not "opencv2/objectdetect.hpp"

berak gravatar imageberak ( 2015-09-05 01:24:15 -0600 )edit

much nicer now. good luck !

berak gravatar imageberak ( 2015-09-06 02:23:38 -0600 )edit

Thanks! It was my fault, I should have verified the syntax.

joaofaro gravatar imagejoaofaro ( 2015-09-06 09:21:41 -0600 )edit
add a comment