Further simplify move_importance()
authorMarco Costalba <mcostalba@gmail.com>
Wed, 1 Jan 2014 12:35:11 +0000 (13:35 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Wed, 1 Jan 2014 12:35:11 +0000 (13:35 +0100)
Function move_importance() is already always
positive, so we don't need to add a constant
term to ensure it.

Becuase move_importance() is used to calculate
ratios of a linear combination (as explained in
previous patch), result is not affected. I have
also verified it directly.

No functional change.

src/timeman.cpp

index fc2a327df253528a77ce9f80f1b286728124f93d..013d5f462f75e2e010a14fedd5aff7ae4c8e4b22 100644 (file)
@@ -44,7 +44,7 @@ namespace {
 
   double move_importance(int ply) {
 
-    return 1 / pow((1 + exp((ply - xshift) / xscale)), skewfactor) + 1e-3; // Ensure non-zero
+    return 1 / pow((1 + exp((ply - xshift) / xscale)), skewfactor);
   }