refactor(qav3): handle missing identifier properly

Signed-off-by: Jérôme Benoit <jerome.benoit@piment-noir.org>

diff --git a/ReforceXY/user_data/freqaimodels/ReforceXY.py b/ReforceXY/user_data/freqaimodels/ReforceXY.py
index 7fc04ec9f7a188b6b19f0a526081da7c9f3381bc..e381af9ecb506a63351ff69191da904bdf932bf4 100644 (file)
--- a/ReforceXY/user_data/freqaimodels/ReforceXY.py
+++ b/ReforceXY/user_data/freqaimodels/ReforceXY.py
@@ -502,7 +502,7 @@ class ReforceXY(BaseReinforcementLearningModel):
         Runs hyperparameter optimization using Optuna and
         returns the best hyperparameters found merged with the user defined parameters
         """
-        identifier = self.freqai_info.get("identifier")
+        identifier = self.freqai_info.get("identifier", "no_id_provided")
         if self.rl_config_optuna.get("per_pair", False):
             study_name = f"{identifier}-{dk.pair}"
             storage = self.get_storage(dk.pair)

diff --git a/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py b/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
index 641a23eea9fa8bf74e5fa184f99f648e3c63b4c2..ee1fcf454f798c2eae297bde257cca84ba28a898 100644 (file)
--- a/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
+++ b/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
@@ -294,7 +294,7 @@ class LightGBMRegressorQuickAdapterV35(BaseRegressionModel):
         y_test,
         test_weights,
     ) -> tuple[dict, float] | tuple[None, None]:
-        identifier = self.freqai_info.get("identifier")
+        identifier = self.freqai_info.get("identifier", "no_id_provided")
         study_namespace = "hp"
         study_name = f"{identifier}-{study_namespace}-{pair}"
         storage = self.optuna_storage(pair)
@@ -373,7 +373,7 @@ class LightGBMRegressorQuickAdapterV35(BaseRegressionModel):
         test_weights,
         model_training_parameters,
     ) -> tuple[dict, float] | tuple[None, None]:
-        identifier = self.freqai_info.get("identifier")
+        identifier = self.freqai_info.get("identifier", "no_id_provided")
         study_namespace = "period"
         study_name = f"{identifier}-{study_namespace}-{pair}"
         storage = self.optuna_storage(pair)

diff --git a/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py b/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
index f616f523729d9aa67e06a790ad0b3e292a3d95fb..4a3f62294fbb2c5e93ba2d4d68826856bfd651fe 100644 (file)
--- a/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
+++ b/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
@@ -297,7 +297,7 @@ class XGBoostRegressorQuickAdapterV35(BaseRegressionModel):
         y_test,
         test_weights,
     ) -> tuple[dict, float] | tuple[None, None]:
-        identifier = self.freqai_info.get("identifier")
+        identifier = self.freqai_info.get("identifier", "no_id_provided")
         study_namespace = "hp"
         study_name = f"{identifier}-{study_namespace}-{pair}"
         storage = self.optuna_storage(pair)
@@ -376,7 +376,7 @@ class XGBoostRegressorQuickAdapterV35(BaseRegressionModel):
         test_weights,
         model_training_parameters,
     ) -> tuple[dict, float] | tuple[None, None]:
-        identifier = self.freqai_info.get("identifier")
+        identifier = self.freqai_info.get("identifier", "no_id_provided")
         study_namespace = "period"
         study_name = f"{identifier}-{study_namespace}-{pair}"
         storage = self.optuna_storage(pair)