From 63a4a97d6a98b93760bfed32d76a6521d8bc038a Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=B4me=20Benoit?= Date: Mon, 26 Jan 2026 13:49:21 +0100 Subject: [PATCH] refactor(quickadapter): remove verbose optuna results logging format_dict() now handles study_best_results in a single line, making the per-key logging loop redundant. --- .../freqaimodels/QuickAdapterRegressorV3.py | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py index cdee305..3a0bef9 100644 --- a/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py +++ b/quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py @@ -2908,27 +2908,10 @@ class QuickAdapterRegressorV3(BaseRegressionModel): f"[{pair}] Optuna {namespace} {objective_type} objective hyperopt completed" f"{metric_log_msg} ({time_spent:.2f} secs)" ) - max_study_results_key_length = ( - max(len(str(key)) for key in study_best_results.keys()) - if study_best_results - else 20 - ) if study_best_results: logger.info( f"[{pair}] Optuna {namespace} {objective_type} objective hyperopt best params: {format_dict(study_best_results, style='dict')}" ) - for key, value in study_best_results.items(): - if isinstance(value, list): - formatted_value = ( - f"[{', '.join([format_number(item) for item in value])}]" - ) - elif isinstance(value, (int, float)): - formatted_value = format_number(value) - else: - formatted_value = repr(value) - logger.info( - f"[{pair}] Optuna {namespace} {objective_type} objective hyperopt | {key:>{max_study_results_key_length}s} : {formatted_value}" - ) if not self.optuna_validate_params(pair, namespace, study): logger.warning( f"[{pair}] Optuna {namespace} {objective_type} objective hyperopt best params found has invalid optimization target value(s)" -- 2.53.0