HKUST-Aerial-Robotics / VINS-Mobile

Monocular Visual-Inertial State Estimator on Mobile Phones
GNU General Public License v3.0
1.27k stars 524 forks source link

New build problems - used to work! #113

Closed trnglt closed 6 years ago

trnglt commented 6 years ago

I had this all working two days ago, and really appreciate your great work. Today, alas it is not working. I've gone through everything - the same code and opencv library that used to compile doesn't. The only change I see is that Xcode command line tools were just updated. Has anyone else seen this?

The offending errors are a conflict between Apple 'NO' macro definition and the opencv definition. In the opencv headers, there's a user warning defined:

if defined(NO)

warning Detected Apple 'NO' macro definition, it can cause build conflicts. Please, include this header before any Apple headers.

endif

I've spent a few hours trying to rearrange header order, and can get it to compile, but then the code crashes on initialization:

OpenCV Error: Assertion failed (_mask.empty() || (_mask.type() == CV_8UC1 && _mask.sameSize(_image))) in goodFeaturesToTrack, file /Users/perry/opencv3/opencv/modules/imgproc/src/featureselect.cpp, line 271 libc++abi.dylib: terminating with uncaught exception of type cv::Exception: /Users/perry/opencv3/opencv/modules/imgproc/src/featureselect.cpp:271: error: (-215) _mask.empty() || (_mask.type() == CV_8UC1 && _mask.sameSize(_image)) in function goodFeaturesToTrack

I can keep plugging away but am curious whether I'm the only one dealing with this.

Thanks again for the great work!

trnglt commented 6 years ago

Oh never mind - I had installed opencv on my mac to run some camera calibrations and xcode was finding that install instead of the one you provided!