{"action":"closed","issue":{"url":"https://api.github.com/repos/robotology/iol/issues/21","repository_url":"https://api.github.com/repos/robotology/iol","labels_url":"https://api.github.com/repos/robotology/iol/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/robotology/iol/issues/21/comments","events_url":"https://api.github.com/repos/robotology/iol/issues/21/events","html_url":"https://github.com/robotology/iol/issues/21","id":144306590,"number":21,"title":"milClassifier not compatible with OpenCV3","user":{"login":"Tobias-Fischer","id":5497832,"avatar_url":"https://avatars.githubusercontent.com/u/5497832?v=3","gravatar_id":"","url":"https://api.github.com/users/Tobias-Fischer","html_url":"https://github.com/Tobias-Fischer","followers_url":"https://api.github.com/users/Tobias-Fischer/followers","following_url":"https://api.github.com/users/Tobias-Fischer/following{/other_user}","gists_url":"https://api.github.com/users/Tobias-Fischer/gists{/gist_id}","starred_url":"https://api.github.com/users/Tobias-Fischer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tobias-Fischer/subscriptions","organizations_url":"https://api.github.com/users/Tobias-Fischer/orgs","repos_url":"https://api.github.com/users/Tobias-Fischer/repos","events_url":"https://api.github.com/users/Tobias-Fischer/events{/privacy}","received_events_url":"https://api.github.com/users/Tobias-Fischer/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/robotology/iol/labels/bug","name":"bug","color":"fc2929"}],"state":"closed","locked":false,"assignee":{"login":"pattacini","id":3738070,"avatar_url":"https://avatars.githubusercontent.com/u/3738070?v=3","gravatar_id":"","url":"https://api.github.com/users/pattacini","html_url":"https://github.com/pattacini","followers_url":"https://api.github.com/users/pattacini/followers","following_url":"https://api.github.com/users/pattacini/following{/other_user}","gists_url":"https://api.github.com/users/pattacini/gists{/gist_id}","starred_url":"https://api.github.com/users/pattacini/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pattacini/subscriptions","organizations_url":"https://api.github.com/users/pattacini/orgs","repos_url":"https://api.github.com/users/pattacini/repos","events_url":"https://api.github.com/users/pattacini/events{/privacy}","received_events_url":"https://api.github.com/users/pattacini/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2016-03-29T15:48:34Z","updated_at":"2016-03-29T16:23:04Z","closed_at":"2016-03-29T16:23:04Z","body":"Hi,\r\nWe recently updated some of our machines to OpenCV3. Thanks to @pattacini most repositories are compatible with this version. However, `milClassifier` does not compile with the new version:\r\n```\r\n[ 92%] Building CXX object src/milClassifier/CMakeFiles/milClassifier.dir/main.cpp.o\r\n/home/icub/robot/iol/src/milClassifier/main.cpp: In constructor ‘MILPort::MILPort(yarp::os::ResourceFinder&)’:\r\n/home/icub/robot/iol/src/milClassifier/main.cpp:818:17: error: ‘create’ is not a member of ‘cv::FeatureDetector {aka cv::Feature2D}’\r\n        detector=cv::FeatureDetector::create(detector_type);\r\n                 ^\r\n/home/icub/robot/iol/src/milClassifier/main.cpp:819:18: error: ‘create’ is not a member of ‘cv::DescriptorExtractor {aka cv::Feature2D}’\r\n        extractor=cv::DescriptorExtractor::create(extractor_type);\r\n```\r\n\r\nThe corresponding lines in the source are these ones:\r\n```\r\n       string detector_type=bFeature.check(\"detector\",Value(feature_name.c_str())).asString().c_str();\r\n       string extractor_type=bFeature.check(\"extractor\",Value(feature_name.c_str())).asString().c_str();\r\n\r\n       detector=cv::FeatureDetector::create(detector_type);\r\n       extractor=cv::DescriptorExtractor::create(extractor_type);\r\n```\r\n\r\nUnfortunately, creating instances by string is not possible in OpenCV3 anymore (see http://docs.opencv.org/3.1.0/db/dfa/tutorial_transition_guide.html). I am not sure what the best way will be to get around this.\r\nDid you need to solve similar issues in the other repositories @pattacini?\r\n\r\nThanks,\r\nTobi\r\n\r\nPS: This is not urgent, as far as I know `milClassifier` is not needed in the recent pipelines anymore."}}