From: Jérôme Benoit <jerome.benoit@piment-noir.org>
Date: Sat, 19 Jul 2025 19:09:14 +0000 (+0200)
Subject: refactor: trivial code cleanup
X-Git-Url: https://git.piment-noir.org/?a=commitdiff_plain;h=ff9e30c1050fba11d462cb3f7261f6568e43b563;p=freqai-strategies.git

refactor: trivial code cleanup

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

diff --git a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
index c48c472..3da8560 100644
--- a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
+++ b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
@@ -1243,6 +1243,15 @@ def get_optuna_study_model_parameters(
         "min_child_samples": (10, 100),
     }
 
+    log_scaled_params = {
+        "learning_rate",
+        "min_child_weight",
+        "reg_alpha",
+        "reg_lambda",
+        "gamma",
+        "min_split_gain",
+    }
+
     ranges = copy.deepcopy(default_ranges)
     if model_training_best_parameters:
         for param, (default_min, default_max) in default_ranges.items():
@@ -1255,14 +1264,7 @@ def get_optuna_study_model_parameters(
             ):
                 continue
 
-            if param in [
-                "learning_rate",
-                "min_child_weight",
-                "reg_alpha",
-                "reg_lambda",
-                "gamma",
-                "min_split_gain",
-            ]:
+            if param in log_scaled_params:
                 new_min = center_value / (1 + expansion_factor)
                 new_max = center_value * (1 + expansion_factor)
             else: