mrlukekr / automated-trader Goto Github PK

Luke,

Your random forest is using a randomized train / test split. Your technical indicators embed prior day information into the calculations, which is leaking data into your randomized splits. Your reference [10] paper has this same issue (https://www.reddit.com/r/algotrading/comments/cv83yh/overfitting).

For time series analysis, you should not use randomized splits, as that is how not data would be received in a real environment.

hi,
tried playing with code, however, it no longer builds