Merge pull request #595 from norlab-ulaval/release

Release 1.4.4
norlab-ulaval · Dec 20, 2024 · 3eb4483 · 3eb4483
2 parents 08729a2 + ed13e2c
commit 3eb4483
Show file tree

Hide file tree

Showing 65 changed files with 1,423 additions and 845 deletions.
diff --git a/.bumbversion.cfg b/.bumbversion.cfg
@@ -0,0 +1,35 @@
+[bumpversion]
+current_version = 1.4.4
+commit = False
+tag = False
+
+[bumpversion:file:package.xml]
+search = <version>{current_version}</version>
+replace = <version>{new_version}</version>
+
+[bumpversion:file:CHANGELOG.rst]
+search = Unreleased
+replace = {new_version} ({now:%Y-%m-%d})
+
+[bumpversion:file:doc/LinkingProjects.md]
+search = find_package(libpointmatcher {current_version} REQUIRED)
+replace = find_package(libpointmatcher {new_version} REQUIRED)
+
+[bumpversion:file:build_system/.env.build_matrix.dependencies]
+search = NBS_MATRIX_REPOSITORY_VERSIONS=( '{current_version}' 'latest'  )
+replace = NBS_MATRIX_REPOSITORY_VERSIONS=( '{new_version}' 'latest'  )
+
+[bumpversion:file:build_system/.env.build_matrix.libpointmatcher.release]
+search = NBS_MATRIX_REPOSITORY_VERSIONS=( '{current_version}' 'latest'  )
+replace = NBS_MATRIX_REPOSITORY_VERSIONS=( '{new_version}' 'latest'  )
+
+[bumpversion:file:pointmatcher/PointMatcher.h]
+search = #define POINTMATCHER_VERSION "{current_version}"
+replace = #define POINTMATCHER_VERSION "{new_version}"
+
+[bumpversion:file:./pointmatcher/PointMatcher.h]
+parse = (?P<major>\d+)\.(?P<minor>\d+)\.(?P<patch>\d+)
+serialize = 
+	{major}0{minor:02d}0{patch:02d}
+search = #define POINTMATCHER_VERSION_INT {current_version}
+replace = #define POINTMATCHER_VERSION_INT {new_version}
diff --git a/.github/workflows/build-python.yaml b/.github/workflows/build-python.yaml
@@ -13,38 +13,35 @@ env:
   BOOST_MAJOR_VERSION: '1'
   BOOST_MINOR_VERSION: '80'
   BOOST_PATCH_VERSION: '0'
-  LIBNABO_VERSION: '1.0.7'
-  PYBIND11_VERSION: 'v2.10.2'
+  PYBIND11_VERSION: '2.13.6'
+  LIBNABO_VERSION: '1.1.2'
   BOOST_INSTALL_PATH: ${{ github.workspace }}/.tmp/boost/install
   BOOST_SRC_DIR: ${{ github.workspace }}/.tmp/boost
   LIBNABO_INSTALL_PATH: ${{ github.workspace }}/.tmp/libnabo-build/install
   LIBNABO_SRC_DIR: ${{ github.workspace }}/.tmp/libnabo
-  PYBIND11_INSTALL_PATH: ${{ github.workspace }}/.tmp/pybind11-build/install
-  PYBIND11_SRC_DIR: ${{ github.workspace }}/.tmp/pybind11
   PYTHON_WHEEL_DIR: ${{ github.workspace }}/.tmp/python-wheel
   VCPKG_DEFAULT_BINARY_CACHE: '${{ github.workspace }}/.tmp/bin-cache'
-  CMAKE_BUILD_PARALLEL_LEVEL: 2
   BUILD_DIR: ${{ github.workspace }}/build
 
 jobs:
   compile-python-package:
     strategy:
       fail-fast: false
       matrix:
-        python_version: ['3.8', '3.9', '3.10', '3.11']
-        os: 
+        python_version: ['3.8', '3.9', '3.10', '3.11', '3.12']
+        os:
           # - windows-2019 cannot compile on Windows
           # (!) TODO: The workflow contains code for Windows building, but we cannot compile it with MS MPI. Changes in the source code are needed.
 
-          - ubuntu-20.04
-      
+          - ubuntu-22.04 # test on Ubuntu 22.04 since other versions should behave similarly
+
     permissions:
       contents: write
-    
+
     runs-on: ${{ matrix.os }}
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - name: Make temp dir
         run: mkdir ${{ github.workspace }}/.tmp
@@ -54,23 +51,23 @@ jobs:
         run: |
           echo "BOOST_DIR=boost_${{ env.BOOST_MAJOR_VERSION }}_${{ env.BOOST_MINOR_VERSION }}_${{ env.BOOST_PATCH_VERSION }}" >> $GITHUB_ENV
           echo "BOOST_VERSION=${{ env.BOOST_MAJOR_VERSION }}.${{ env.BOOST_MINOR_VERSION }}.${{ env.BOOST_PATCH_VERSION }}" >> $GITHUB_ENV
- 
+
       - name: Init env variable on Linux
         if: ${{ runner.os == 'Linux' }}
-        run: | 
+        run: |
           echo "BOOST_ARCHIVE_NAME=${{ env.BOOST_DIR }}.7z" >> $GITHUB_ENV
 
       - name: Init base env variable on Windows
         if: ${{ runner.os == 'Windows' }}
         run: |
           echo "BOOST_DIR=boost_${{ env.BOOST_MAJOR_VERSION }}_${{ env.BOOST_MINOR_VERSION }}_${{ env.BOOST_PATCH_VERSION }}" | Out-File -FilePath $env:GITHUB_ENV -Append
           echo "BOOST_VERSION=${{ env.BOOST_MAJOR_VERSION }}.${{ env.BOOST_MINOR_VERSION }}.${{ env.BOOST_PATCH_VERSION }}" | Out-File -FilePath $env:GITHUB_ENV -Append
- 
+
       - name: Init env variable on Windows
         if: ${{ runner.os == 'Windows' }}
-        run: | 
+        run: |
           echo "BOOST_ARCHIVE_NAME=${{ env.BOOST_DIR }}.zip" | Out-File -FilePath $env:GITHUB_ENV -Append
-      
+
       - name: Cache boost
         id: cache-boost
         if: ${{ runner.os  == 'Linux' }}
@@ -79,12 +76,12 @@ jobs:
           path: ${{ env.BOOST_INSTALL_PATH }}
           key: ${{ runner.os }}-boost-cache-${{ env.BOOST_VERSION }}-python-${{ matrix.python_version }}
 
-      - name: Cache libnabo
-        id: cache-libnabo
-        uses: actions/cache@v3
-        with:
-          path: ${{ env.LIBNABO_INSTALL_PATH }}
-          key: ${{ runner.os }}-libnabo-cache-${{ env.LIBNABO_VERSION }}
+      # - name: Cache libnabo
+      #   id: cache-libnabo
+      #   uses: actions/cache@v3
+      #   with:
+      #     path: ${{ env.LIBNABO_INSTALL_PATH }}
+      #     key: ${{ runner.os }}-libnabo-cache-${{ env.LIBNABO_VERSION }}
 
       - name: Cache binary vcpkg
         if: ${{ runner.os  == 'Windows' }}
@@ -94,21 +91,12 @@ jobs:
           path: ${{ env.VCPKG_DEFAULT_BINARY_CACHE }}
           key: ${{ runner.os }}-vcpkg-cache-${{ matrix.python_version }}
 
-      - name: Cache pybind11
-        id: cache-pybind11
-        uses: actions/cache@v3
-        with:
-          path: ${{ env.PYBIND11_INSTALL_PATH }}
-          key: ${{ runner.os }}-pybind11-cache--${{ env.PYBIND11_VERSION }}-python-${{ matrix.python_version }}
-
       - name: Make dirs on Linux
         if: ${{ runner.os  == 'Linux' }}
         run: |
           mkdir -p ${{ env.BOOST_SRC_DIR }}
           mkdir -p ${{ env.LIBNABO_SRC_DIR }}
-          mkdir -p ${{ env.PYBIND11_SRC_DIR }}
           mkdir -p ${{ env.BOOST_INSTALL_PATH }}
-          mkdir -p ${{ env.PYBIND11_INSTALL_PATH }}
           mkdir -p ${{ env.LIBNABO_INSTALL_PATH }}
           mkdir -p ${{ env.VCPKG_DEFAULT_BINARY_CACHE }}
 
@@ -117,9 +105,7 @@ jobs:
         run: |
           mkdir -Force ${{ env.BOOST_SRC_DIR }}
           mkdir -Force ${{ env.LIBNABO_SRC_DIR }}
-          mkdir -Force ${{ env.PYBIND11_SRC_DIR }}
           mkdir -Force ${{ env.BOOST_INSTALL_PATH }}
-          mkdir -Force ${{ env.PYBIND11_INSTALL_PATH }}
           mkdir -Force ${{ env.LIBNABO_INSTALL_PATH }}
           mkdir -Force ${{ env.VCPKG_DEFAULT_BINARY_CACHE }}
 
@@ -135,7 +121,10 @@ jobs:
             ninja-build \
             catch \
             libomp-dev \
-            wget
+            wget \
+            p7zip-full \
+            sudo \
+            cmake
 
       - name: Install MS MPI
         if: ${{ runner.os == 'Windows' }}
@@ -153,17 +142,16 @@ jobs:
 
       - name: Install required dependencies for Python
         run: |
-          python -m pip install -U 'pip>=23.0'
-          pip install 'numpy>=1.20' wheel 'setuptools>=61.0' 'build~=0.10'
-      
+          python -m pip install -U pip
+          pip install 'numpy>=1.20' 'wheel' 'setuptools>=61.0' 'build~=0.10' 'cmake' 'pybind11==${{ env.PYBIND11_VERSION }}'
+
       - name: Install dependencies on Windows
         if: ${{ runner.os == 'Windows' }}
         run: |
           vcpkg install eigen3:x64-windows-static `
                 libopenmpt:x64-windows `
                 boost-mpi:x64-windows-static `
                 boost-thread:x64-windows-static `
-                boost-filesystem:x64-windows-static `
                 boost-system:x64-windows-static `
                 boost-program-options:x64-windows-static `
                 boost-date-time:x64-windows-static `
@@ -185,7 +173,7 @@ jobs:
         if: ${{ steps.cache-boost.outputs.cache-hit != 'true' && runner.os == 'Linux' }}
         working-directory: ${{ env.BOOST_SRC_DIR }}/${{ env.BOOST_DIR }}
         run: |
-          ./bootstrap.sh --with-libraries=thread,filesystem,system,program_options,date_time,chrono --with-icu --prefix=${{ env.BOOST_INSTALL_PATH }}
+          ./bootstrap.sh --with-libraries=thread,system,program_options,date_time,chrono --with-icu --prefix=${{ env.BOOST_INSTALL_PATH }}
           ./b2 cxxflags=-fPIC cflags=-fPIC link=static install
 
       - name: Download source code libnabo ${{ env.LIBNABO_VERSION }}
@@ -216,61 +204,24 @@ jobs:
           cd libnabo
           cmake -GNinja \
             -DBOOST_ROOT=${{ env.BOOST_INSTALL_PATH }} \
-            -DCMAKE_INSTALL_PREFIX=${{ env.LIBNABO_INSTALL_PATH }} \
             -DCMAKE_BUILD_TYPE=RelWithDebInfo \
             -DUSE_OPEN_MP:BOOL=ON \
             -S . -B ./build
-          cmake --build ./build --target install
+          sudo cmake --build ./build --target install --parallel 4
           cd -
           rm -r ./libnabo/build
 
-      - name: Download source code pybind11 ${{ env.PYBIND11_VERSION }}
-        if: steps.cache-pybind11.outputs.cache-hit != 'true'
-        working-directory: ${{ env.PYBIND11_SRC_DIR }}
-        run: |
-          git clone -b ${{ env.PYBIND11_VERSION }} --single-branch https://github.com/pybind/pybind11.git
-
-      
-      - name: Install pybind11 ${{ env.PYBIND11_VERSION }} on Linux
-        working-directory:  ${{ env.PYBIND11_SRC_DIR }}
-        if: ${{ steps.cache-pybind11.outputs.cache-hit != 'true' && runner.os == 'Linux' }}
-        run: |
-          cd pybind11
-          cmake -GNinja \
-            -DCMAKE_INSTALL_PREFIX=${{ env.PYBIND11_INSTALL_PATH }} \
-            -DCMAKE_BUILD_TYPE=RelWithDebInfo \
-            -DPYBIND11_TEST:BOOL=OFF \
-            -S . -B ./build
-          cmake --build ./build --target install
-          rm -r ./build
-      
-      - name: Install pybind11 ${{ env.PYBIND11_VERSION }} on Windows
-        working-directory:  ${{ env.PYBIND11_SRC_DIR }}
-        if: ${{ steps.cache-pybind11.outputs.cache-hit != 'true' && runner.os == 'Windows' }}
-        run: |
-          cd pybind11
-          cmake -A x64 `
-            -DCMAKE_TOOLCHAIN_FILE="$env:VCPKG_INSTALLATION_ROOT/scripts/buildsystems/vcpkg.cmake" `
-            -DCMAKE_INSTALL_PREFIX=${{ env.PYBIND11_INSTALL_PATH }} `
-            -DCMAKE_BUILD_TYPE=RelWithDebInfo `
-            -DPYBIND11_TEST:BOOL=OFF `
-            -S . -B ./build
-          cmake --build ./build --target install
-          rm -r ./build
-
       - name: Compile libpointmatcher on Linux
         if: runner.os == 'Linux'
         run: |
           cmake \
             -DBOOST_ROOT:PATH=${{ env.BOOST_INSTALL_PATH }} \
-            -DLIBNABO_INSTALL_DIR=${{ env.LIBNABO_INSTALL_PATH }} \
-            -Dpybind11_DIR=${{ env.PYBIND11_INSTALL_PATH }}/share/cmake/pybind11 \
             -DBUILD_PYTHON_MODULE:BOOL=ON \
             -DUSE_OPEN_MP:BOOL=ON \
             -DBoost_USE_STATIC_LIBS:BOOL=ON \
             -DCMAKE_BUILD_TYPE=RelWithDebInfo \
             -S . -B ${{ env.BUILD_DIR }}
-          sudo cmake --build ${{ env.BUILD_DIR }} --target install
+          sudo cmake --build ${{ env.BUILD_DIR }} --target install --parallel 4
 
       - name: Compile libpointmatcher on Windows
         if: runner.os == 'Windows'
@@ -279,17 +230,16 @@ jobs:
         run: |
           cmake -A x64 `
             -DCMAKE_TOOLCHAIN_FILE:FILEPATH="$env:VCPKG_INSTALLATION_ROOT/scripts/buildsystems/vcpkg.cmake" `
-            -DLIBNABO_INSTALL_DIR:PATH="${{ env.LIBNABO_INSTALL_PATH }}" `
-            -Dpybind11_DIR:PATH="${{ env.PYBIND11_INSTALL_PATH }}/share/cmake/pybind11" `
+            -DLIBNABO_INSTALL_DIR:PATH="${{ env.LIBNABO_INSTALL_PATH }}" ``
             -DBUILD_PYTHON_MODULE:BOOL=ON `
             -DBoost_USE_STATIC_LIBS:BOOL=ON `
             -DPYTHON_INSTALL_TARGET:PATH="./python/${{ env.PYTHON_PACKAGE_NAME }}" `
             -DVCPKG_TARGET_TRIPLET="x64-windows-static" `
             -DUSE_OPEN_MP:BOOL=ON `
             -DCMAKE_BUILD_TYPE="RelWithDebInfo" `
             -S . -B ${{ env.BUILD_DIR }}
-          cmake --build ${{ env.BUILD_DIR }} --target install
-      
+          cmake --build ${{ env.BUILD_DIR }} --target install --parallel 4
+
       - name: Build python wheel
         working-directory: ./python
         run: |
@@ -299,7 +249,7 @@ jobs:
         working-directory: ./python
         run: |
           pip install ${{ env.PYTHON_WHEEL_DIR }}/*.whl
-      
+
       - name: Test import
         working-directory: ${{ runner.temp }}
         run: python -c "from pypointmatcher import *"

diff --git a/.gitignore b/.gitignore
@@ -346,4 +346,37 @@ pyrightconfig.json
 .history
 .ionide
 
+### macOS ###
+# General
+.DS_Store
+.AppleDouble
+.LSOverride
+
+# Icon must end with two \r
+Icon
+
+
+# Thumbnails
+._*
+
+# Files that might appear in the root of a volume
+.DocumentRevisions-V100
+.fseventsd
+.Spotlight-V100
+.TemporaryItems
+.Trashes
+.VolumeIcon.icns
+.com.apple.timemachine.donotpresent
+
+# Directories potentially created on remote AFP share
+.AppleDB
+.AppleDesktop
+Network Trash Folder
+Temporary Items
+.apdisk
+
+### macOS Patch ###
+# iCloud generated files
+*.icloud
+
 # End of https://www.toptal.com/developers/gitignore/api/visualstudiocode,clion,c++,python
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -1,6 +1,22 @@
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 Changelog for package libpointmatcher
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
+1.4.4 (2024-12-16)
+------------------
+**New features:**
+* Added Angle Limit Datapointsfilter, which filters points if they lie inside or outside of a given spherical wedge
+* Added support for binary PLY files IO
+
+**Major changes:**
+* Min. Cmake version pumped to 3.15
+* Removed dependency on boost filesystem
+* Python bindings compile with recent versions of pybind11
+
+**Other changes:**
+* Updated Python GitHub action
+* Updated gitignore
+* Updated docs
+
 
 1.4.3 (2024-06-28)
 ------------------