From 5211113736ae3ca3277095d9e774865ed2a3efbf Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Fri, 26 Sep 2025 13:33:31 +0200 Subject: [PATCH] fix(reforce): try to make hyperopt outputs not resusable by FT MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit And hopefully still cleaned up Signed-off-by: Jérôme Benoit --- ReforceXY/user_data/freqaimodels/ReforceXY.py | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/ReforceXY/user_data/freqaimodels/ReforceXY.py b/ReforceXY/user_data/freqaimodels/ReforceXY.py index 1e1a471..92b1659 100644 --- a/ReforceXY/user_data/freqaimodels/ReforceXY.py +++ b/ReforceXY/user_data/freqaimodels/ReforceXY.py @@ -474,14 +474,19 @@ class ReforceXY(BaseReinforcementLearningModel): ) callbacks.append(self.eval_callback) else: - trial_data_path = f"{data_path}/hyperopt/trial_{trial.number}" + data_path: Path = Path(data_path) + trial_data_path = ( + data_path.parent + / f"{data_path.name}-hyperopt" + / f"trial-{trial.number}" + ) self.optuna_eval_callback = MaskableTrialEvalCallback( eval_env, trial, eval_freq=eval_freq, deterministic=True, render=False, - best_model_save_path=trial_data_path, + best_model_save_path=str(trial_data_path), use_masking=use_masking, verbose=verbose, ) @@ -1008,7 +1013,7 @@ class ReforceXY(BaseReinforcementLearningModel): / "tensorboard" / dk.pair.split("/")[0] / "hyperopt" - / f"trial_{trial.number}" + / f"trial-{trial.number}" ) else: tensorboard_log_path = None -- 2.43.0