Move strategies with known lookahead bias to a separate directory.

closes #240
closes #238
closes #237
This commit is contained in:
Matthias
2022-03-01 19:56:16 +01:00
parent 484569e9c9
commit e4350c79d2
4 changed files with 29 additions and 0 deletions
@@ -0,0 +1,29 @@
Warning, Strategies in this folder do have a lookahead bias.
Please see these as practice to see if you can spot the lookahead bias.
<details>
<summary>Expand for spoilers / solution</summary>
Please Click on each strategy to see details of the mistakes.
<details>
<summary>DevilStra</summary>
`normalize()` uses `.min()` and `.max()`. This uses the full dataframe, not just past data.
</details>
<details>
<summary>GodStraNew</summary>
`normalize()` uses `.min()` and `.max()`. This uses the full dataframe, not just past data.
</details>
<details>
<summary>Zeus</summary>
uses `.min()` and `.max()` to normalize `trend_ichimoku_base` as well as `trend_kst_diff`.
</details>
</details>