ATR introduces look-ahead bias
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 9k
- Forks
- 1.5k
- PR merge metrics
- No merged PRs in 30d
Description
Expected Behavior
When we trade with trailing stop loss the ATR is calculated. Fo example suppose that we calculate the ATR for 100 periods the first 100 periods should be removed because they don't have values and if you try to open a trade in the period 10 there isn't an available ATR value therefore isn't possible to determine the stop loss. So in that case all the values that contains NaN values should be removed.
Actual Behavior
In the code when the ATR is calculated the values are backward filled with the future values and according to our previous example if a trade is opened in the period 10, then the ATR used is the ATR in the period 101
def set_atr_periods(self, periods: int = 100):
"""
Set the lookback period for computing ATR. The default value
of 100 ensures a _stable_ ATR.
"""
h, l, c_prev = self.data.High, self.data.Low, pd.Series(self.data.Close).shift(1)
tr = np.max([h - l, (c_prev - h).abs(), (c_prev - l).abs()], axis=0)
atr = pd.Series(tr).rolling(periods).mean().bfill().values # Is look-ahead bias introduced using bfill?
self.__atr = atr
Additional info
It maintins the ATR calculation and changes until the period 101
My code is the following
class MyStrategy(TrailingStrategy):
size = 0.3
atr = 3
atr_period = 100
def init(self):
super().init()
self.set_trailing_sl(n_atr=self.atr)
self.set_atr_periods(periods=self.atr_period)
def next(self):
super().next()
if not self.position.is_long:
self.buy(size=self.size)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at TrailingStrategy.set_atr_periods and inspect how the rolling ATR series is passed into trailing-stop calculations. Reproduce the 100-period example with the provided MyStrategy, then verify that unavailable initial ATR values are not filled from future periods and that trades cannot use look-ahead data. Add or update the relevant regression coverage if the repository has existing tests for trailing stops or ATR.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- fintech-quant
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100