mirror of
https://github.com/freqtrade/freqtrade.git
synced 2024-11-10 10:21:59 +00:00
Add comment with reasoning to ignore leverage in min_amount calculation
This commit is contained in:
parent
988f4d5190
commit
fb519a5b39
|
@ -606,12 +606,14 @@ class FreqtradeBot(LoggingMixin):
|
|||
if not enter_limit_requested:
|
||||
raise PricingError(f'Could not determine {side} price.')
|
||||
|
||||
# Min-stake-amount should actually include Leverage - this way our "minimal"
|
||||
# stake- amount might be higher than necessary.
|
||||
# We do however also need min-stake to determine leverage, therefore this is ignored as
|
||||
# edge-case for now.
|
||||
min_stake_amount = self.exchange.get_min_pair_stake_amount(
|
||||
pair,
|
||||
enter_limit_requested,
|
||||
self.strategy.stoploss,
|
||||
# TODO-lev: This is problematic... we need stake-amount to determine max_leverage
|
||||
# leverage=leverage
|
||||
)
|
||||
|
||||
if not self.edge:
|
||||
|
|
|
@ -368,6 +368,10 @@ class Backtesting:
|
|||
|
||||
def _get_sell_trade_entry_for_candle(self, trade: LocalTrade,
|
||||
sell_row: Tuple) -> Optional[LocalTrade]:
|
||||
# TODO-lev: add interest / funding fees to trade object ->
|
||||
# Must be done either here, or one level higher ->
|
||||
# (if we don't want to do it at "detail" level)
|
||||
|
||||
sell_candle_time = sell_row[DATE_IDX].to_pydatetime()
|
||||
enter = sell_row[SHORT_IDX] if trade.is_short else sell_row[LONG_IDX]
|
||||
exit_ = sell_row[ESHORT_IDX] if trade.is_short else sell_row[ELONG_IDX]
|
||||
|
|
Loading…
Reference in New Issue
Block a user