Skip to content

Questions on minor discrepancies w/ paper #4

@ebagjones

Description

@ebagjones

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.

  1. I can't find TP, SL descriptions. To Purchase, SelL?
  2. 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.
  3. 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!)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions