Report profit only on filled entries.
This commit is contained in:
parent
c9498d0117
commit
bfc7898654
@ -177,6 +177,8 @@ class RPC:
|
|||||||
current_rate = NAN
|
current_rate = NAN
|
||||||
else:
|
else:
|
||||||
current_rate = trade.close_rate
|
current_rate = trade.close_rate
|
||||||
|
if len(trade.select_filled_orders(trade.entry_side)) > 0:
|
||||||
|
logger.warning(trade.select_filled_orders(trade.entry_side))
|
||||||
current_profit = trade.calc_profit_ratio(current_rate)
|
current_profit = trade.calc_profit_ratio(current_rate)
|
||||||
current_profit_abs = trade.calc_profit(current_rate)
|
current_profit_abs = trade.calc_profit(current_rate)
|
||||||
current_profit_fiat: Optional[float] = None
|
current_profit_fiat: Optional[float] = None
|
||||||
@ -187,6 +189,8 @@ class RPC:
|
|||||||
self._freqtrade.config['stake_currency'],
|
self._freqtrade.config['stake_currency'],
|
||||||
self._freqtrade.config['fiat_display_currency']
|
self._freqtrade.config['fiat_display_currency']
|
||||||
)
|
)
|
||||||
|
else:
|
||||||
|
current_profit = current_profit_abs = current_profit_fiat = 0.0
|
||||||
|
|
||||||
# Calculate guaranteed profit (in case of trailing stop)
|
# Calculate guaranteed profit (in case of trailing stop)
|
||||||
stoploss_entry_dist = trade.calc_profit(trade.stop_loss)
|
stoploss_entry_dist = trade.calc_profit(trade.stop_loss)
|
||||||
@ -235,8 +239,12 @@ class RPC:
|
|||||||
trade.pair, side='exit', is_short=trade.is_short, refresh=False)
|
trade.pair, side='exit', is_short=trade.is_short, refresh=False)
|
||||||
except (PricingError, ExchangeError):
|
except (PricingError, ExchangeError):
|
||||||
current_rate = NAN
|
current_rate = NAN
|
||||||
|
if len(trade.select_filled_orders(trade.entry_side)) > 0:
|
||||||
trade_profit = trade.calc_profit(current_rate)
|
trade_profit = trade.calc_profit(current_rate)
|
||||||
profit_str = f'{trade.calc_profit_ratio(current_rate):.2%}'
|
profit_str = f'{trade.calc_profit_ratio(current_rate):.2%}'
|
||||||
|
else:
|
||||||
|
trade_profit = 0.0
|
||||||
|
profit_str = f'{0.0:.2f}'
|
||||||
direction_str = ('S' if trade.is_short else 'L') if nonspot else ''
|
direction_str = ('S' if trade.is_short else 'L') if nonspot else ''
|
||||||
if self._fiat_converter:
|
if self._fiat_converter:
|
||||||
fiat_profit = self._fiat_converter.convert_amount(
|
fiat_profit = self._fiat_converter.convert_amount(
|
||||||
@ -244,7 +252,7 @@ class RPC:
|
|||||||
stake_currency,
|
stake_currency,
|
||||||
fiat_display_currency
|
fiat_display_currency
|
||||||
)
|
)
|
||||||
if fiat_profit and not isnan(fiat_profit):
|
if not isnan(fiat_profit):
|
||||||
profit_str += f" ({fiat_profit:.2f})"
|
profit_str += f" ({fiat_profit:.2f})"
|
||||||
fiat_profit_sum = fiat_profit if isnan(fiat_profit_sum) \
|
fiat_profit_sum = fiat_profit if isnan(fiat_profit_sum) \
|
||||||
else fiat_profit_sum + fiat_profit
|
else fiat_profit_sum + fiat_profit
|
||||||
|
Loading…
Reference in New Issue
Block a user