chore: remove hard rollout bound since we have lru

raklaptudirm · Jun 22, 2024 · 02d7c6b · 02d7c6b
1 parent 0329a65
commit 02d7c6b
Showing 1 changed file with 5 additions and 13 deletions.
diff --git a/src/mcts/mod.rs b/src/mcts/mod.rs
@@ -4,7 +4,6 @@ pub mod value;
 mod params;
 mod tree;
 
-use std::mem;
 use std::time;
 
 pub use self::params::*;
@@ -87,19 +86,12 @@ impl Searcher {
                 self.uci_report();
             }
 
-            if self.rollouts & 127 == 0 {
-                if self.start.elapsed().as_millis() >= movetime
+            if self.rollouts & 127 == 0
+                && (self.start.elapsed().as_millis() >= movetime
                     || self.avgdepth >= maxdepth
-                    || self.rollouts >= maxnodes
-                {
-                    break;
-                }
-
-                // Hard memory limit to prevent overuse.
-                // TODO: Fix this by removing old nodes and stuff.
-                if self.rollouts > 2_000_000_000 / mem::size_of::<Node>() {
-                    break;
-                }
+                    || self.rollouts >= maxnodes)
+            {
+                break;
             }
         }