Skip to content

Various improvements for using ViSP for iOS #1882

Merged
fspindle merged 10 commits intolagadic:masterfrom
fspindle:feat_ios
Jan 20, 2026
Merged

Various improvements for using ViSP for iOS #1882
fspindle merged 10 commits intolagadic:masterfrom
fspindle:feat_ios

Conversation

@fspindle
Copy link
Contributor

…7 in headers

- Using VISP_CXX_STANDARD_17 in headers brings issues when cross-compiling ViSP
- Fixes No template named 'optional' in namespace 'std' reported issue lagadic#1876
Such sequences will not work in the future
…loses lagadic#1878

- Instead of creating a static link to opencv2.framework in `xcrun --sdk iphoneos --show-sdk-platform-path...'
  we added --cmake_framework_path option to platforms/ios/build_framework.py that can point to
  the path that contains opencv2.framework
- Synchronize platforms/ios/cmake/Toolchains/ files with their correspondants in OpenCV
@codecov
Copy link

codecov bot commented Jan 19, 2026

Codecov Report

❌ Patch coverage is 0% with 58 lines in your changes missing coverage. Please review.
✅ Project coverage is 48.33%. Comparing base (bf9cb5c) to head (6ec6d15).
⚠️ Report is 53 commits behind head on master.

Files with missing lines Patch % Lines
modules/tracker/me/include/visp3/me/vpMeEllipse.h 0.00% 34 Missing ⚠️
...roc/include/visp3/imgproc/vpCircleHoughTransform.h 0.00% 20 Missing ⚠️
modules/core/src/image/vpImageCircle.cpp 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1882      +/-   ##
==========================================
+ Coverage   48.14%   48.33%   +0.18%     
==========================================
  Files         532      532              
  Lines       69012    69220     +208     
  Branches    32241    32371     +130     
==========================================
+ Hits        33229    33460     +231     
+ Misses      31721    31675      -46     
- Partials     4062     4085      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fspindle fspindle merged commit f0fdc62 into lagadic:master Jan 20, 2026
87 checks passed
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.

Unable to build ViSP framework from source for iOS with OpenCV support Unable to build ViSP framework from source for iOS

1 participant