fix(qav3): properly test for previous study best_params property

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

diff --git a/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py b/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
index 6a849bb9599c26e2d3bfbb00e78a8d3c45246adf..0718c847ecbc8c426d101c891d5985f5fce45a2a 100644 (file)
--- a/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
+++ b/quickadapter/user_data/freqaimodels/LightGBMRegressorQuickAdapterV35.py
@@ -285,7 +285,7 @@ class LightGBMRegressorQuickAdapterV35(BaseRegressionModel):
             direction=optuna.study.StudyDirection.MINIMIZE,
             storage=storage,
         )
-        if previous_study:
+        if previous_study and hasattr(previous_study, "best_params"):
             study.enqueue_trial(previous_study.best_params)
         start = time.time()
         try:
@@ -342,7 +342,7 @@ class LightGBMRegressorQuickAdapterV35(BaseRegressionModel):
             direction=optuna.study.StudyDirection.MINIMIZE,
             storage=storage,
         )
-        if previous_study:
+        if previous_study and hasattr(previous_study, "best_params"):
             study.enqueue_trial(previous_study.best_params)
         start = time.time()
         try:
@@ -382,7 +382,6 @@ class LightGBMRegressorQuickAdapterV35(BaseRegressionModel):
     ) -> optuna.study.Study | None:
         try:
             previous_study = optuna.load_study(study_name=study_name, storage=storage)
-            previous_study.best_params
         except Exception:
             previous_study = None
         try:

diff --git a/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py b/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
index 79666e25fcc211503695f752e0a8a58a6a47fafe..3493740658507a305684e301639c04aeb33764c0 100644 (file)
--- a/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
+++ b/quickadapter/user_data/freqaimodels/XGBoostRegressorQuickAdapterV35.py
@@ -286,7 +286,7 @@ class XGBoostRegressorQuickAdapterV35(BaseRegressionModel):
             direction=optuna.study.StudyDirection.MINIMIZE,
             storage=storage,
         )
-        if previous_study:
+        if previous_study and hasattr(previous_study, "best_params"):
             study.enqueue_trial(previous_study.best_params)
         start = time.time()
         try:
@@ -343,7 +343,7 @@ class XGBoostRegressorQuickAdapterV35(BaseRegressionModel):
             direction=optuna.study.StudyDirection.MINIMIZE,
             storage=storage,
         )
-        if previous_study:
+        if previous_study and hasattr(previous_study, "best_params"):
             study.enqueue_trial(previous_study.best_params)
         start = time.time()
         try:
@@ -383,7 +383,6 @@ class XGBoostRegressorQuickAdapterV35(BaseRegressionModel):
     ) -> optuna.study.Study | None:
         try:
             previous_study = optuna.load_study(study_name=study_name, storage=storage)
-            previous_study.best_params
         except Exception:
             previous_study = None
         try: