-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Python: TestGenerator: unmatched data error (low value X at Y is not reached from the least timeframe, low price Z mismatches) #86
Comments
I'm very sorry, and I want to apologize, but I'm not able to create the environment to run the tests. I would take a long time to get set up, and a whole day is gone, so I'm giving up the task. Once again, I'm sorry, thank you. |
I'm sorry about it. If you're still interested, I can give you the access to the remote Linux where the issue can be easily reproduced. Or this can be tested using Travis CI. |
Is this still an issue? |
@Ryu0 I think it is. If the steps aren't clear, I can double check them this evening. I think now the approach is to use Docker container, but syntax using |
@Ryu0 I've tested and the problem can be still reproduced. I've updated description to use Here is the example:
The main error which appears:
Please check also other timeframes (use
In summary, the error happens, because of some problem in the data which has been converted from CSV into FXT and MT4 complains about it, and it doesn't happen for other years for some reason. Basically I need to find the source of this error and fix it. To test your fixes, you can login to Docker container by:
then you can run:
Once data has been downloaded first time, second run should use the existing files. To replace FXT data, you can generate using script and replace in the platform dir.
To test only specific month, change it using If you've any questions, please ask. For quicker answer, contact me at http://t.me/kenorb |
Hi. Is this still an issue, right? |
@brunofarina I think it's still the issue. |
Lol so is this the code challenge? Or we haven't gotten there yet |
@kenorb I'm able to working on this issue. |
After fixing #85, this error start happening at M30 timeframe when backtesting:
Similar error at TestPeriod M5 (plus volume error, similar as per: #70):
And M15:
This can be tested using Docker container, e.g.
-T
). And all months:-m 1-12
. Remove-b
and-M
params to avoid re-downloading the data when running it again.docker run -it ea31337/ea-tester bash
.-x
for debug.After fix in
convert_csv_to_mt.py
, clone and convert the CSV data e.g. via (see:Makefile
as example):and move generated files and replace with the existing HST files in
~/.wine/drive_c/Program Files/MetaTrader 4/history/default
and re-run the test.To read the converted HST files, you can use
convert_mt_to_csv.py
tool, e.g.The problem is with low value mismatch as per error. The fix shouldn't break the fixed logic in dbb096e, which wasn't working before as per #85. The solution should include the proper calculation of OLHCV values (open/low/high/close/volume) across the different timeframes to avoid any unmatched data errors.
See also:
Requirements
Basically these commands needs to work with no data error after complete testing:
docker run ea31337/ea-tester run_backtest -v -t -M4.0.0.1010 -d 1000 -p EURUSD -m 1-12 -y 2015 -s 10 -b DS -D5 -e TestFXTHeader
(see: Build #1087).docker run ea31337/ea-tester run_backtest -T M1 -v -t -e MA -y 2015 -m 7
(this works)docker run ea31337/ea-tester run_backtest -T M5 -v -t -e MA -y 2015 -m 7
docker run ea31337/ea-tester run_backtest -T M15 -v -t -e MA -y 2015 -m 7
docker run ea31337/ea-tester run_backtest -T M30 -v -t -e MA -y 2015 -m 7
docker run ea31337/ea-tester run_backtest -T M30 -v -t -e MA -y 2015 -m 1-12
(test all months just in case)Resources
--
Est. 10h
The text was updated successfully, but these errors were encountered: