Merge pull request #218 from alicevision/dev/fixes

[fixes] Boost fixes for versions >=1.85

Author: fabiencastan

.github/workflows/continuous-integration.yml

@@ -2,22 +2,22 @@ name: Continuous Integration
 
 on:
   push:
-    branches:
-      - master
-      - develop
+#    branches:
+#      - master
+#      - develop
     # Skip jobs when only documentation files are changed
-    paths-ignore:
-      - '**.md'
-      - '**.rst'
-      - 'docs/**'
+#    paths-ignore:
+#      - '**.md'
+#      - '**.rst'
+#      - 'docs/**'
   pull_request:
     paths-ignore:
       - '**.md'
       - '**.rst'
       - 'docs/**'
 
 jobs:
-  build:
+  build-linux:
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -68,3 +68,83 @@ jobs:
            -DBUILD_SHARED_LIBS:BOOL=ON \
            -DCMAKE_PREFIX_PATH:PATH="$PWD/../../cctag_install;${DEPS_INSTALL_DIR}"
           make -j8
+
+  build_windows:
+    runs-on: windows-latest
+    strategy:
+      matrix:
+        config: [ Debug, Release ]
+    env:
+      buildDir: '${{ github.workspace }}\build\'
+      vcpkgDir: '${{ github.workspace }}\..\e\vcpkg'
+      # commit for version 2023.11.20
+      COMMIT_ID: a42af01b72c28a8e1d7b48107b33e4f286a55ef6
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v3
+      with:
+        submodules: true
+
+    - name: Install latest CMake.
+      uses: lukka/get-cmake@latest
+
+    # Restore from cache the previously built ports. If a "cache miss" occurs, then vcpkg is bootstrapped. Since a the vcpkg.json is being used later on to install the packages when run-cmake runs, no packages are installed at this time and the input 'setupOnly:true' is mandatory.
+    - name: vcpkg - Setup dependencies
+      uses: lukka/run-vcpkg@v7
+      with:
+        # Just install vcpkg for now, do not install any ports in this step yet.
+        setupOnly: false
+        # Location of the vcpkg submodule in the Git repository.
+        vcpkgDirectory: ${{ env.vcpkgDir }}
+        vcpkgGitCommitId: ${{ env.COMMIT_ID }}
+        vcpkgArguments: >
+          boost-accumulators 
+          boost-algorithm boost-container
+          boost-date-time
+          boost-exception
+          boost-filesystem
+          boost-foreach
+          boost-iterator
+          boost-lexical-cast
+          boost-math
+          boost-mpl
+          boost-multi-array
+          boost-ptr-container
+          boost-program-options
+          boost-serialization
+          boost-spirit
+          boost-static-assert
+          boost-stacktrace
+          boost-test
+          boost-thread
+          boost-throw-exception
+          boost-timer
+          boost-type-traits
+          boost-unordered
+          opencv
+          tbb
+          eigen3
+        vcpkgTriplet: x64-windows
+        # doNotCache: true
+        # This is used to unbreak cached artifacts if for some reason dependencies fail to build,
+        # the action does not notice it and saves broken artifacts.
+        appendedCacheKey: cache007
+
+    - name: vcpkg - Display installed packages
+      run:
+          ${{ env.vcpkgDir }}\vcpkg.exe list
+
+    - name: Build
+      uses: lukka/run-cmake@v3
+      with:
+        cmakeListsOrSettingsJson: CMakeListsTxtAdvanced
+        cmakeListsTxtPath: '${{ github.workspace }}/CMakeLists.txt'
+        buildDirectory: ${{ env.buildDir }}
+#        cmakeGenerator: VS16Win64
+        cmakeAppendedArgs: -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON -DBUILD_SHARED_LIBS:BOOL=ON -DVCPKG_TARGET_TRIPLET=x64-windows -A x64 -T host=x64 -DCCTAG_WITH_CUDA:BOOL=OFF
+        # This input tells run-cmake to consume the vcpkg.cmake toolchain file set by run-vcpkg.
+        cmakeBuildType: ${{ matrix.config }}
+        useVcpkgToolchainFile: true
+        buildWithCMake: true
+        buildWithCMakeArgs: -j1 --config ${{ matrix.config }}

doc/sphinx/source/install/install.rst

@@ -120,6 +120,7 @@ To install the dependencies:
           boost-container
           boost-date-time
           boost-exception
+          boost-foreach
           boost-filesystem
           boost-iterator
           boost-lexical-cast
@@ -142,6 +143,8 @@ To install the dependencies:
           tbb
           eigen3
 
+Note that for boost >= 1.85 you have to use :code:`math[legacy]`.
+
 You can add the flag :code:`--triplet` to specify the architecture and the version you want to build.
 For example:
 

src/applications/detection/main.cpp

@@ -21,7 +21,6 @@
 #include <boost/archive/xml_oarchive.hpp>
 #include <boost/exception/all.hpp>
 #include <boost/filesystem.hpp>
-#include <boost/filesystem/convenience.hpp>
 #include <boost/ptr_container/ptr_list.hpp>
 #include <boost/timer/timer.hpp>
 #include <opencv2/core/core.hpp>
@@ -501,7 +500,7 @@ int main(int argc, char** argv)
         tbb::parallel_for(0, 2, [&](size_t fileListIdx) {
             for(const auto& fileInFolder : files[fileListIdx])
             {
-                const std::string subExt(bfs::extension(fileInFolder.second));
+                const std::string subExt(fileInFolder.second.extension().string());
 
                 if((subExt == ".png") || (subExt == ".jpg") || (subExt == ".PNG") || (subExt == ".JPG"))
                 {

src/cctag/CCTag.cpp

@@ -17,7 +17,6 @@
 
 #include <opencv2/core/core_c.h>
 
-#include <boost/foreach.hpp>
 #include <boost/array.hpp>
 #include <boost/mpl/bool.hpp>
 

src/cctag/CCTagFlowComponent.cpp

@@ -8,7 +8,6 @@
 #include <cctag/CCTagFlowComponent.hpp>
 #include <cctag/utils/Defines.hpp>
 
-#include <boost/foreach.hpp>
 
 namespace cctag
 {

src/cctag/DataSerialization.hpp

@@ -15,7 +15,6 @@
 #include <boost/serialization/nvp.hpp>
 #include <boost/serialization/serialization.hpp>
 #include <boost/archive/text_oarchive.hpp>
-#include <boost/foreach.hpp>
 
 namespace cctag {
 

src/cctag/Detection.cpp

@@ -28,7 +28,6 @@
 #include "cctag/cuda/tag.h"
 #endif
 
-#include <boost/foreach.hpp>
 #include <boost/math/constants/constants.hpp>
 #include <boost/math/special_functions/round.hpp>
 #include <boost/mpl/bool.hpp>

src/cctag/EllipseGrowing.cpp

@@ -23,7 +23,6 @@
 #include <opencv2/core/types_c.h>
 
 #include <boost/math/special_functions/pow.hpp>
-#include <boost/foreach.hpp>
 #include <boost/math/constants/constants.hpp>
 #include <boost/mpl/bool.hpp>
 #include <boost/assert.hpp>

src/cctag/Fitting.cpp

@@ -7,7 +7,6 @@
  */
 #include <boost/algorithm/string/classification.hpp>
 #include <boost/algorithm/string/split.hpp>
-#include <boost/foreach.hpp>
 #include <boost/lexical_cast.hpp>
 #include <boost/math/constants/constants.hpp>
 #include <boost/math/special_functions/round.hpp>
@@ -226,8 +225,6 @@ template void fitEllipse(std::vector<cctag::Point2d<Eigen::Vector3f>>::const_ite
 } // geometry
 
 float innerProdMin(const std::vector<cctag::EdgePoint*>& filteredChildren, float thrCosDiffMax, Point2d<Vector3s> & p1, Point2d<Vector3s> & p2) {
-            using namespace boost::numeric;
-            //using namespace cctag::numerical;
 
             EdgePoint* pAngle1 = nullptr;
             EdgePoint* pAngle2 = nullptr;
@@ -335,7 +332,6 @@ void ellipseFitting( cctag::numerical::geometry::Ellipse& e, const std::vector<c
 }
 
 void circleFitting(cctag::numerical::geometry::Ellipse& e, const std::vector<cctag::EdgePoint*>& points) {
-  using namespace boost::numeric;
 
   std::size_t nPoints = points.size();
 

src/cctag/Fitting.hpp

@@ -11,7 +11,6 @@
 #include <cctag/EdgePoint.hpp>
 #include <cctag/geometry/Ellipse.hpp>
 #include <cctag/geometry/Point.hpp>
-#include <boost/foreach.hpp>
 
 #include <list>
 #include <string>