-
Notifications
You must be signed in to change notification settings - Fork 85
Open
Description
This is a great project and I like what has been done here, but I've been going through the code and paper and have some questions.
- I can't find TP, SL descriptions. To Purchase, SelL?
- Trading fees are excluded when buying, selling at the same price - fractionally, a large number of positions are opened, closed at the same trade price, but this doesn't account for fees or slippage. This can be significant drag on the PnL.
- Just a note, your Histograms have a bug, the Buy and Hold and DDDQN labels are swapped. I think this error might have even carried into your paper.
Thanks again (and in advance!)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels