-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[🕹️]Added the notebook for Backtesting Momentum Strategies #6759
[🕹️]Added the notebook for Backtesting Momentum Strategies #6759
Conversation
Hi @jmaslek, please review my PR. |
Can you supress the pandas warnings then I will approve and merge and award |
Hey @jmaslek, I have addressed the pandas warning. You can review the PR. |
/award 300 |
Awarding SanchitMahajan236: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/SanchitMahajan236 |
Awarding SanchitMahajan236: 300 points 🕹️ Well done! Check out your new contribution on oss.gg/SanchitMahajan236 |
Expected to enhance user capability to test and refine momentum strategies, leading to more informed trading decisions.
Minimal impact on system performance, with backtesting execution time expected to increase by ~5%.
Impact score: 8
Closes #6712