From 0e699b87af3bf3c9097c7774689f0a29f35196a8 Mon Sep 17 00:00:00 2001 From: xmatthias Date: Thu, 7 Jun 2018 20:08:46 +0200 Subject: [PATCH] don't sum percentage, but use mean instead (aligned to backtesting) --- freqtrade/rpc/rpc.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/freqtrade/rpc/rpc.py b/freqtrade/rpc/rpc.py index f48666748..c2f097319 100644 --- a/freqtrade/rpc/rpc.py +++ b/freqtrade/rpc/rpc.py @@ -9,6 +9,7 @@ from typing import Dict, Tuple, Any import arrow import sqlalchemy as sql from pandas import DataFrame +from numpy import mean, nan_to_num from freqtrade import exchange from freqtrade.misc import shorten_date @@ -209,14 +210,14 @@ class RPC(object): fiat = self.freqtrade.fiat_converter # Prepare data to display profit_closed_coin = round(sum(profit_closed_coin), 8) - profit_closed_percent = round(sum(profit_closed_percent) * 100, 2) + profit_closed_percent = round(nan_to_num(mean(profit_closed_percent)) * 100, 2) profit_closed_fiat = fiat.convert_amount( profit_closed_coin, stake_currency, fiat_display_currency ) profit_all_coin = round(sum(profit_all_coin), 8) - profit_all_percent = round(sum(profit_all_percent) * 100, 2) + profit_all_percent = round(nan_to_num(mean(profit_all_percent)) * 100, 2) profit_all_fiat = fiat.convert_amount( profit_all_coin, stake_currency,