From cf2d7ed806fa7ea59e3a7646364a611920a2693d Mon Sep 17 00:00:00 2001
From: KoalaTrapLord <265133236+KoalaTrapLord@users.noreply.github.com>
Date: Mon, 30 Mar 2026 09:51:22 -0400
Subject: [PATCH] fix(eval): use reload() with explicit checkpoint path instead
 of load()

load() hardcodes best_policy.pt but PPO trainer saves as best_ppo.pt, causing the model to never load during eval. reload() accepts an explicit path.
---
 ml/eval/forge_evaluator.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/ml/eval/forge_evaluator.py b/ml/eval/forge_evaluator.py
index 94566c2..12c7891 100644
--- a/ml/eval/forge_evaluator.py
+++ b/ml/eval/forge_evaluator.py
@@ -219,9 +219,8 @@ class ForgeEvaluator:
                 embeddings_dir=self.cfg.embeddings_dir,
                 device=self.cfg.device,
             )
-            # Override the default checkpoint filename with the exact path
-            svc.checkpoint_dir = str(Path(self.cfg.checkpoint_path).parent)
-            ok = svc.load()
+                        # Use reload() with explicit checkpoint path (load() hardcodes best_policy.pt)
+ok = svc.reload(self.cfg.checkpoint_path)
             if not ok:
                 logger.warning("[Eval] Policy service load failed — entropy/latency metrics unavailable")
             return svc