Fixed issues to compile and run FaceDetection demo#14
Fixed issues to compile and run FaceDetection demo#14madhavajay wants to merge 2 commits intoLegoless:masterfrom
Conversation
madhavajay
commented
Mar 18, 2019
- Updated OpenCV submodule to 4.0.1
- Renamed methods detectMultiscaleWith to DetectMultiScaleOnImage
- Removed unused test targets
- Added reference to data folder in opencv for haar cascade files
- Updated Xcode Project to Swift Version 4.2
- Fixed some minor project Code Deprecation changes
- Updated OpenCV submodule to 4.0.1 - Renamed methods detectMultiscaleWith to DetectMultiScaleOnImage - Removed unused test targets - Added reference to data folder in opencv for haar cascade files - Updated Xcode Project to Swift Version 4.2 - Fixed some minor project Code Deprecation changes
|
Firstly, this is an awesome project. Thanks for all the hard work! I removed the tests because they were empty and I had trouble with OpenCV compiling for the simulator but I think it would be good to add them back and add some CI and tests. The Face Detector app wasnt compiling and now it compiles but the bounding boxes arent being drawn or something is wrong however the code is working and the Face Haar cascades work. I want to use this with Carthage and I looked at adding a compile step into the scripts for the LegoCV target like this: But its really slow and I was thinking whats wrong with using the binaries? Otherwise it would be nice to add some kind of build flag to allow building from binaries vs building from source however im not sure how that would work with Cocapods / Carthage as an extra step? OpenCV isn't going anywhere so I think this project should be more and more popular. Thoughts? |
- Removed some empty DevelopmentTeam / DEVELOPMENT_TEAM settings
|
Thanks for your help! It's true that OpenCV takes a while to build and at some point it should be included in the repo (git lfs would be required, as it's a huge file). Since I've already had the Swift 5 update in the work, not sure if this needs to be merged. I am keeping the test targets though, as I will be adding tests soon in the future. |