]> Piment Noir Git Repositories - freqai-strategies.git/commitdiff
refactor(quickadapter): remove verbose optuna results logging
authorJérôme Benoit <jerome.benoit@piment-noir.org>
Mon, 26 Jan 2026 12:49:21 +0000 (13:49 +0100)
committerJérôme Benoit <jerome.benoit@piment-noir.org>
Mon, 26 Jan 2026 12:49:21 +0000 (13:49 +0100)
format_dict() now handles study_best_results in a single line, making
the per-key logging loop redundant.

quickadapter/user_data/freqaimodels/QuickAdapterRegressorV3.py

index cdee30510a89cf3b168389ff4756557bd7b3b36a..3a0bef9683166a99c05d55dd585ea84f1aa54ca1 100644 (file)
@@ -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)"