-
Notifications
You must be signed in to change notification settings - Fork 155
✨ Feature/timeframe zoom #5
✨ Feature/timeframe zoom #5
Conversation
Some hyperopt results with this PR:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems to have hurt the total profit hyperopt spits out badly but at least now we should be able to get way more reliable test results!
I will pull this in for review/merging after I finish up the unclogger 😉
For readers unfamiliar with that this would fix:
https://brookmiles.github.io/freqtrade-stuff/backtesting-traps/
since `&` doesn't short circut.
Just updated the unclogger so it should be compatible with this PR now, about to start the merging 😉 However, I noticed that the hyperopted version with the used unclogger doesn't appear to be having issues with backtest exploitation at first sight. |
What this feature tested with the 'OnlyProfitHyperOptLoss' objective? It seems like it exacerbate the backtest trap |
I have been testing this feature with If needed this feature can easily be turned off by setting both |
Sounds good, I will probably start dry-running myself now that I now the trick about removing 'timeframe' from the config JSON file :) |
This PR adds a timeframe zoom feature. When backtesting it will check that you are using 1 or 5m timeframe and throws an error if otherwise. It's a bit annoying to change all the time but I don't think there is a good solution to do it automatically (without a bunch of overhead when running live). It also makes sure that you don't forget to change it back and start running the strat live at 5m.
This is pretty much ripped off of Obelisk with a few small improvements.
startup_candle_count
conversion when backtestingHere are some backtesting results using v0.8.1 weights and original coin list:
Running a hyperopt with this new feature gives less tight results for trailing stoploss: