From: Jérôme Benoit Date: Mon, 26 Jan 2026 12:49:21 +0000 (+0100) Subject: refactor(quickadapter): remove verbose optuna results logging X-Git-Url: https://git.piment-noir.org/?a=commitdiff_plain;h=63a4a97d6a98b93760bfed32d76a6521d8bc038a;p=freqai-strategies.git 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. --- 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)"