From 75faada18d45e4661126c94c0783ea0eb5026e14 Mon Sep 17 00:00:00 2001
From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= <jerome.benoit@piment-noir.org>
Date: Sat, 3 Jan 2026 00:10:44 +0100
Subject: [PATCH] refactor: remove redundant 'Only' prefix from namespace
 comments

Simplify code comments by removing the word 'Only' from namespace
identifier comments. The context already makes it clear that these
are the supported namespaces.
---
 .../freqaimodels/QuickAdapterRegressorV3.py   | 20 ++++++++-----------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
index a3f32b2..4103310 100644
--- a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
+++ b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py
@@ -1184,7 +1184,7 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
         else:
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[0]!r}"  # Only "hp"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[0]!r}"  # "hp"
             )
         return value
 
@@ -1196,7 +1196,7 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
         else:
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[0]!r}"  # Only "hp"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[0]!r}"  # "hp"
             )
 
     def get_optuna_values(
@@ -1207,7 +1207,7 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
         else:
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # Only "label"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # "label"
             )
         return values
 
@@ -1219,7 +1219,7 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
         else:
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # Only "label"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # "label"
             )
 
     def init_optuna_label_candle_pool(self) -> None:
@@ -1364,12 +1364,10 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
         namespace: OptunaNamespace,
         callback: Callable[[], Optional[optuna.study.Study]],
     ) -> None:
-        if namespace not in {
-            QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]
-        }:  # Only "label"
+        if namespace not in {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}:  # "label"
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # Only "label"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # "label"
             )
         if not callable(callback):
             raise ValueError(
@@ -2688,12 +2686,10 @@ class QuickAdapterRegressorV3(BaseRegressionModel):
     def _get_multi_objective_study_best_trial(
         self, namespace: OptunaNamespace, study: optuna.study.Study
     ) -> Optional[optuna.trial.FrozenTrial]:
-        if namespace not in {
-            QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]
-        }:  # Only "label"
+        if namespace not in {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}:  # "label"
             raise ValueError(
                 f"Invalid namespace {namespace!r}. "
-                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # Only "label"
+                f"Supported: {QuickAdapterRegressorV3._OPTUNA_NAMESPACES[1]}"  # "label"
             )
         n_objectives = len(study.directions)
         if n_objectives < 2:
-- 
2.53.0