Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Add tolerance for floating point precision comparisons.
authorAugustin Degomme <adegomme@gmail.com>
Thu, 21 Nov 2019 16:16:00 +0000 (17:16 +0100)
committerAugustin Degomme <adegomme@gmail.com>
Thu, 21 Nov 2019 16:16:00 +0000 (17:16 +0100)
Fails on 32bits if equality is used.

src/xbt/random_test.cpp

index d8cee22..c964cb8 100644 (file)
@@ -7,6 +7,7 @@
 #include "xbt/log.h"
 #include "xbt/random.hpp"
 #include <random>
+#include <cmath>
 
 TEST_CASE("xbt::random: Random Number Generation")
 {
@@ -14,10 +15,10 @@ TEST_CASE("xbt::random: Random Number Generation")
   {
     simgrid::xbt::random::set_mersenne_seed(12345);
 
-    REQUIRE(simgrid::xbt::random::exponential(25) == 0.00291934351538427348);
+    REQUIRE(fabs(simgrid::xbt::random::exponential(25) - 0.00291934351538427348) <= 1e-14);
     REQUIRE(simgrid::xbt::random::uniform_int(1, 6) == 4);
-    REQUIRE(simgrid::xbt::random::uniform_real(0, 1) == 0.31637556043369124970);
-    REQUIRE(simgrid::xbt::random::normal(0, 2) == 1.62746784745133976635);
+    REQUIRE(fabs(simgrid::xbt::random::uniform_real(0, 1) - 0.31637556043369124970) <= 1e-14);
+    REQUIRE(fabs(simgrid::xbt::random::normal(0, 2) - 1.62746784745133976635) <= 1e-14);
   }
 
   SECTION("Using XBT_RNG_std")
@@ -33,9 +34,9 @@ TEST_CASE("xbt::random: Random Number Generation")
     std::uniform_real_distribution<> distC(0, 1);
     std::normal_distribution<> distD(0, 2);
 
-    REQUIRE(simgrid::xbt::random::exponential(25) == distA(gen));
+    REQUIRE(fabs(simgrid::xbt::random::exponential(25) - distA(gen)) <= 1e-14);
     REQUIRE(simgrid::xbt::random::uniform_int(1, 6) == distB(gen));
-    REQUIRE(simgrid::xbt::random::uniform_real(0, 1) == distC(gen));
-    REQUIRE(simgrid::xbt::random::normal(0, 2) == distD(gen));
+    REQUIRE(fabs(simgrid::xbt::random::uniform_real(0, 1) - distC(gen)) <= 1e-14);
+    REQUIRE(fabs(simgrid::xbt::random::normal(0, 2) - distD(gen)) <= 1e-14);
   }
 }