-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Support multiple instruments(symbols) backtest #639
base: master
Are you sure you want to change the base?
Conversation
How do you think @kernc? If you think it is inappropriate I will withdraw it, thanks. |
It looks like the current attempt is not backwards compatible. For example, the signature of Maybe we can include a flag which enables multi-symbol mode and return |
There is also something wrong with |
I've made a parallel attempt here #641 |
How is this coming along? |
60eff81
to
109c352
Compare
Not related in anyway with him but @dodid did an awesome job extending this backtesting.py library to his minitrade package which enables multiasset backtesting, check it out: https://github.com/dodid/minitrade |
428c361
to
0ce6cab
Compare
70abc06
to
c79ffb0
Compare
Fixes #20
An incomplete attempt to add multiple-symbol backtest, which is mentioned in #20. Not sure if this fits the current structure.
Lots of necessary changes haven't been done yet, including docs and code related graphing part.