From ede951f5ec254b1c20531d6eb44d227eab353fa1 Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Sat, 21 Jun 2025 02:20:14 +0200 Subject: [PATCH] refactor(qav3): add clause guard at candle pool handling MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Signed-off-by: Jérôme Benoit --- .../user_data/freqaimodels/QuickAdapterRegressorV3.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py index 8a40fee..c5d91ca 100644 --- a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py +++ b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py @@ -78,7 +78,7 @@ class QuickAdapterRegressorV3(BaseRegressionModel): def _optuna_label_candle_pool_full(self) -> list[int]: if not hasattr(self, "pairs") or not self.pairs: raise RuntimeError( - "Failed to initialize optuna label candle pool full: pairs property is not defined" + "Failed to initialize optuna label candle pool full: pairs property is not defined or empty" ) n_pairs = len(self.pairs) label_frequency_candles = max( @@ -211,7 +211,10 @@ class QuickAdapterRegressorV3(BaseRegressionModel): raise ValueError(f"Invalid namespace: {namespace}") def init_optuna_label_candle_pool(self) -> None: - self._optuna_label_candle_pool = self._optuna_label_candle_pool_full + optuna_label_candle_pool_full = self._optuna_label_candle_pool_full + if len(optuna_label_candle_pool_full) == 0: + raise RuntimeError("Failed to initialize optuna label candle pool full") + self._optuna_label_candle_pool = optuna_label_candle_pool_full random.shuffle(self._optuna_label_candle_pool) if len(self._optuna_label_candle_pool) == 0: raise RuntimeError("Failed to initialize optuna label candle pool") -- 2.43.0