Merge pull request #8 from northwesternfintech/egelja/fix-tests

Fix FP comparison
northwesternfintech · Aug 1, 2023 · bc59348 · bc59348
2 parents 4d736d0 + 38bfff2
commit bc59348
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -67,6 +67,10 @@ jobs:
         sudo add-apt-repository ppa:ubuntu-toolchain-r/test
         sudo apt-get update
         sudo apt-get install gcc-13 g++-13
+        
+        sudo update-alternatives --install /usr/bin/gcc gcc /usr/bin/gcc-13 130
+        sudo update-alternatives --install /usr/bin/g++ g++ /usr/bin/g++-13 130
+        sudo update-alternatives --install /usr/bin/gcov gcov /usr/bin/gcov-13 130
 
     - name: Install dependencies
       run: |

diff --git a/src/storage/processing.cpp b/src/storage/processing.cpp
@@ -50,7 +50,7 @@ OrderbookProcessor::process_incoming_update_(const Update& newUpdate)
 
     auto updateOrderbook =
         [](std::unordered_map<double, double>& orderSide, double price, double volume) {
-            if (volume == 0.0) {
+            if (volume <= 0.00000000001) {
                 orderSide.erase(price);
             }
             else {