-
Notifications
You must be signed in to change notification settings - Fork 12
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
Super jupyternb for backtesting! #2
Comments
Thank you so much for the time reading this report and code carefully. I will try to answer your questions here. q2 if you check the pnl graphs both Entry TRUE & FALSE trades are being included: A:
**q3: The article mentions "holding period was no more than 5 minutes per trade". A: From the picture, you can see the whole trade holding period (from entry = True to exit = True)
**q4: May I confirm what are cex_price & dex_price in the csv? Mid prices?". A: Hope this explanation clarifies the questions and am happy to answer if you have any. |
Thank you so much for the insights! Much appreciated. |
For Q2 - apologies I should have specified - it was the pnl bar chart I was referring to. Attached screenpic on left is from article where 30May22 pnl > $4k vs. actual pnl $3.614k (I adjusted to show only Entry: TRUE on right). Agreed - the Cumulative PnL chart (red line) looks fine. The Sharpe Ratio also is impacted by inclusion of Entry: FALSE trades fyi. @FunnyBlue |
Hi @unclecharliecrypto Add this line of code under the function groupby_date_data(df) will solve the issue and the chart for both PnL sum/avg and the Sharp will be updated accordingly. Thanks for pointing this out!
|
@FunnyBlue Thanks FunnyBlue! |
@unclecharliecrypto happy to collaborate/discuss if you have some interesting strategy ideas to explore together. |
@FunnyBlue fyi. Been live testing several of the hummingbot.io strats in a personal capacity. In particular working through the code for the Avellaneda MM strat. It has gone through several hummingbot version modeling interations. |
Great visualization for potential arb opps.
(1) I have verified a total of 19 trades placed over 28 days where trades submitted on 9 days (syntax "Entry:TRUE")
(2) There were an additional 10 trades (syntax "Entry: FALSE") which occurred shortly after the above trades. The python code (drop duplicates) excludes these trades which is fine. However - if you check the pnl graphs both Entry TRUE & FALSE trades are being included. For example, pnl on 30May22 is $3.6k (TRUE) however the pnl chart is showing > $4k due to combining both TRUE&FALSE trades. Sharpe ratio impacted as well.
(3) The article mentions "holding period was no more than 5 minutes per trade". Unsure how to interpret this? Screenpic of 19 trade timestamps attached.
(4)May I confirm what are cex_price & dex_price in the csv? Mid prices?
The text was updated successfully, but these errors were encountered: