From 1717f867020dd7a451ca7d0ad702e5ea4ae0559d Mon Sep 17 00:00:00 2001
From: Matthias <xmatthias@outlook.com>
Date: Sun, 25 Jun 2023 17:45:01 +0200
Subject: [PATCH] Extract edge output to proper module

---
 .../optimize/optimize_reports/__init__.py     | 11 ++++----
 .../optimize/optimize_reports/bt_output.py    | 25 ++++++++++++++++++
 .../optimize_reports/optimize_reports.py      | 26 -------------------
 3 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/freqtrade/optimize/optimize_reports/__init__.py b/freqtrade/optimize/optimize_reports/__init__.py
index c8865d84d..68e222d00 100644
--- a/freqtrade/optimize/optimize_reports/__init__.py
+++ b/freqtrade/optimize/optimize_reports/__init__.py
@@ -1,5 +1,6 @@
 # flake8: noqa: F401
-from freqtrade.optimize.optimize_reports.bt_output import (show_backtest_result,
+from freqtrade.optimize.optimize_reports.bt_output import (generate_edge_table,
+                                                           show_backtest_result,
                                                            show_backtest_results,
                                                            show_sorted_pairlist,
                                                            text_table_add_metrics,
@@ -11,7 +12,7 @@ from freqtrade.optimize.optimize_reports.bt_storage import (store_backtest_analy
                                                             store_backtest_stats)
 from freqtrade.optimize.optimize_reports.optimize_reports import (
     generate_all_periodic_breakdown_stats, generate_backtest_stats, generate_daily_stats,
-    generate_edge_table, generate_exit_reason_stats, generate_pair_metrics,
-    generate_periodic_breakdown_stats, generate_rejected_signals, generate_strategy_comparison,
-    generate_strategy_stats, generate_tag_metrics, generate_trade_signal_candles,
-    generate_trading_stats, generate_wins_draws_losses)
+    generate_exit_reason_stats, generate_pair_metrics, generate_periodic_breakdown_stats,
+    generate_rejected_signals, generate_strategy_comparison, generate_strategy_stats,
+    generate_tag_metrics, generate_trade_signal_candles, generate_trading_stats,
+    generate_wins_draws_losses)
diff --git a/freqtrade/optimize/optimize_reports/bt_output.py b/freqtrade/optimize/optimize_reports/bt_output.py
index c9d4478b3..1fd1f7a34 100644
--- a/freqtrade/optimize/optimize_reports/bt_output.py
+++ b/freqtrade/optimize/optimize_reports/bt_output.py
@@ -378,3 +378,28 @@ def show_sorted_pairlist(config: Config, backtest_stats: Dict):
                 if result["key"] != 'TOTAL':
                     print(f'"{result["key"]}",  // {result["profit_mean"]:.2%}')
             print("]")
+
+
+def generate_edge_table(results: dict) -> str:
+    floatfmt = ('s', '.10g', '.2f', '.2f', '.2f', '.2f', 'd', 'd', 'd')
+    tabular_data = []
+    headers = ['Pair', 'Stoploss', 'Win Rate', 'Risk Reward Ratio',
+               'Required Risk Reward', 'Expectancy', 'Total Number of Trades',
+               'Average Duration (min)']
+
+    for result in results.items():
+        if result[1].nb_trades > 0:
+            tabular_data.append([
+                result[0],
+                result[1].stoploss,
+                result[1].winrate,
+                result[1].risk_reward_ratio,
+                result[1].required_risk_reward,
+                result[1].expectancy,
+                result[1].nb_trades,
+                round(result[1].avg_trade_duration)
+            ])
+
+    # Ignore type as floatfmt does allow tuples but mypy does not know that
+    return tabulate(tabular_data, headers=headers,
+                    floatfmt=floatfmt, tablefmt="orgtbl", stralign="right")
diff --git a/freqtrade/optimize/optimize_reports/optimize_reports.py b/freqtrade/optimize/optimize_reports/optimize_reports.py
index c217fe5f9..015f163e3 100644
--- a/freqtrade/optimize/optimize_reports/optimize_reports.py
+++ b/freqtrade/optimize/optimize_reports/optimize_reports.py
@@ -4,7 +4,6 @@ from datetime import datetime, timedelta, timezone
 from typing import Any, Dict, List, Union
 
 from pandas import DataFrame, concat, to_datetime
-from tabulate import tabulate
 
 from freqtrade.constants import BACKTEST_BREAKDOWNS, DATETIME_PRINT_FORMAT, IntOrInf
 from freqtrade.data.metrics import (calculate_cagr, calculate_calmar, calculate_csum,
@@ -215,31 +214,6 @@ def generate_strategy_comparison(bt_stats: Dict) -> List[Dict]:
     return tabular_data
 
 
-def generate_edge_table(results: dict) -> str:
-    floatfmt = ('s', '.10g', '.2f', '.2f', '.2f', '.2f', 'd', 'd', 'd')
-    tabular_data = []
-    headers = ['Pair', 'Stoploss', 'Win Rate', 'Risk Reward Ratio',
-               'Required Risk Reward', 'Expectancy', 'Total Number of Trades',
-               'Average Duration (min)']
-
-    for result in results.items():
-        if result[1].nb_trades > 0:
-            tabular_data.append([
-                result[0],
-                result[1].stoploss,
-                result[1].winrate,
-                result[1].risk_reward_ratio,
-                result[1].required_risk_reward,
-                result[1].expectancy,
-                result[1].nb_trades,
-                round(result[1].avg_trade_duration)
-            ])
-
-    # Ignore type as floatfmt does allow tuples but mypy does not know that
-    return tabulate(tabular_data, headers=headers,
-                    floatfmt=floatfmt, tablefmt="orgtbl", stralign="right")
-
-
 def _get_resample_from_period(period: str) -> str:
     if period == 'day':
         return '1d'