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

Added core/utility.hpp include #1

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

steevo87
Copy link

Necessary to enable calls such as "cv::getTickCount()"

Necessary to enable calls such as "cv::getTickCount()"
@kwaegel
Copy link
Owner

kwaegel commented Apr 15, 2014

Is there any reason to add this header without any corresponding code that uses it?

@steevo87
Copy link
Author

I had to include the header for the original code to compile. Without it, I was getting these errors:

2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(206): error C2039: 'getTickCount' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(206): error C3861: 'getTickCount': identifier not found
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(222): error C2039: 'getTickCount' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(222): error C2039: 'getTickFrequency' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(222): error C3861: 'getTickCount': identifier not found
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(222): error C3861: 'getTickFrequency': identifier not found
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(227): error C2039: 'getTickCount' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(227): error C3861: 'getTickCount': identifier not found
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(231): error C2039: 'getTickCount' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(231): error C2039: 'getTickFrequency' : is not a member of 'cv'
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(231): error C3861: 'getTickCount': identifier not found
2>C:\Users\ur10\Documents\GitHub\kinfu_remake\kfusion\src\core.cpp(231): error C3861: 'getTickFrequency': identifier not found

@kwaegel
Copy link
Owner

kwaegel commented Apr 15, 2014

I just double checked, and <opencv2/core/core.hpp> is already included in kfusion/types.hpp. The include chain (core.cpp<kinfu.hpp<types.hpp) should already have included that for you, so I suspect something else is wrong with your build environment.

@steevo87
Copy link
Author

Oh OK, so I'm guessing your <opencv2/core/core.hpp> includes a line like this:

#include <opencv2/core/utility.hpp>

Mine doesn't. I'm using the trunk version of OpenCV (last edited: 2014-04-11 19:18:04). Is there a chance that they removed that line of code in between the version of OpenCV you are using and the one I am? In my version of OpenCV I can only find a reference to utility.hpp in core_c.h

@kwaegel
Copy link
Owner

kwaegel commented Apr 15, 2014

That's probably it. I think trunk is in the middle of a huge refactoring for OpenCV 3.0. Try using the OpenCV 2.4.9 tag that was recently added.

I last built this with OpenCV commit opencv/opencv@a8cb5c3 from the 2.4 branch.

Should adapt better to different types of OpenCV installations (e.g.
from pre-compiled binaries or local builds). Should default to correct
OpenNI 2 address. Changes untested on UR10-PC.
Better at finding and handling different versions of OpenCV, and now
copies the necessary DLLs/drivers to run the demo.
REQUIRED listed twice in CMake line..
Improved OpenCV location hints
Includes an extra file if OpenCV version is greater than 2.
To handle trunk version better.
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

Successfully merging this pull request may close these issues.

2 participants