Shift candles after pushing them to dataprovider
this will ensure that the signals are not shifted in callbacks closes #6234
This commit is contained in:
parent
4b9d55dbe2
commit
5d0c2bcb44
@ -275,6 +275,13 @@ class Backtesting:
|
|||||||
# Trim startup period from analyzed dataframe
|
# Trim startup period from analyzed dataframe
|
||||||
df_analyzed = processed[pair] = pair_data = trim_dataframe(
|
df_analyzed = processed[pair] = pair_data = trim_dataframe(
|
||||||
df_analyzed, self.timerange, startup_candles=self.required_startup)
|
df_analyzed, self.timerange, startup_candles=self.required_startup)
|
||||||
|
# Update dataprovider cache
|
||||||
|
self.dataprovider._set_cached_df(pair, self.timeframe, df_analyzed)
|
||||||
|
|
||||||
|
# Create a copy of the dataframe before shifting, that way the buy signal/tag
|
||||||
|
# remains on the correct candle for callbacks.
|
||||||
|
df_analyzed = df_analyzed.copy()
|
||||||
|
|
||||||
# To avoid using data from future, we use buy/sell signals shifted
|
# To avoid using data from future, we use buy/sell signals shifted
|
||||||
# from the previous candle
|
# from the previous candle
|
||||||
df_analyzed.loc[:, 'buy'] = df_analyzed.loc[:, 'buy'].shift(1)
|
df_analyzed.loc[:, 'buy'] = df_analyzed.loc[:, 'buy'].shift(1)
|
||||||
@ -282,9 +289,6 @@ class Backtesting:
|
|||||||
df_analyzed.loc[:, 'buy_tag'] = df_analyzed.loc[:, 'buy_tag'].shift(1)
|
df_analyzed.loc[:, 'buy_tag'] = df_analyzed.loc[:, 'buy_tag'].shift(1)
|
||||||
df_analyzed.loc[:, 'exit_tag'] = df_analyzed.loc[:, 'exit_tag'].shift(1)
|
df_analyzed.loc[:, 'exit_tag'] = df_analyzed.loc[:, 'exit_tag'].shift(1)
|
||||||
|
|
||||||
# Update dataprovider cache
|
|
||||||
self.dataprovider._set_cached_df(pair, self.timeframe, df_analyzed)
|
|
||||||
|
|
||||||
df_analyzed = df_analyzed.drop(df_analyzed.head(1).index)
|
df_analyzed = df_analyzed.drop(df_analyzed.head(1).index)
|
||||||
|
|
||||||
# Convert from Pandas to list for performance reasons
|
# Convert from Pandas to list for performance reasons
|
||||||
|
Loading…
Reference in New Issue
Block a user