adelorenzo/stable
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

removed trade_count inside if statement

Browse Source 
i removed trade_count inside if statement. Even though it helps overfitting, It is not useful when running hyperopt on small datasets.
This commit is contained in:
Robert Roman 2021-09-22 09:21:43 -05:00 committed by GitHub
parent b946f8e7f1
commit c6b684603c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
freqtrade/optimize/hyperopt_loss_calmar.py
View File

@ -42,7 +42,7 @@ class CalmarHyperOptLoss(IHyperOptLoss):
except ValueError: except ValueError:
max_drawdown = 0 max_drawdown = 0
if max_drawdown != 0 and trade_count > 1000: if max_drawdown != 0:
calmar_ratio = expected_returns_mean / max_drawdown * np.sqrt(365) calmar_ratio = expected_returns_mean / max_drawdown * np.sqrt(365)
else: else:
calmar_ratio = -20. calmar_ratio = -20.