Fix Windows action (#90)

* Add erase after remove in std::vector to check Windows CI. * Remove Debug build type in Windows CI.
lisitsyn · Apr 19, 2024 · 0ad0861 · 0ad0861
1 parent 7226b75
commit 0ad0861
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml
@@ -12,7 +12,7 @@ jobs:
     runs-on: windows-latest
     strategy:
       matrix:
-        build_type: [Debug, Release]
+        build_type: [Release]
 
     steps:
     - uses: actions/checkout@v3

diff --git a/include/tapkee/neighbors/neighbors.hpp b/include/tapkee/neighbors/neighbors.hpp
@@ -162,7 +162,8 @@ Neighbors find_neighbors_vptree_impl(const RandomAccessIterator& begin, const Ra
     for (RandomAccessIterator i = begin; i != end; ++i)
     {
         LocalNeighbors local_neighbors = tree.search(i, k + 1);
-        std::remove(local_neighbors.begin(), local_neighbors.end(), i - begin);
+        auto it = std::remove(local_neighbors.begin(), local_neighbors.end(), i - begin);
+        local_neighbors.erase(it, local_neighbors.end());
         neighbors.push_back(local_neighbors);
     }