-
Notifications
You must be signed in to change notification settings - Fork 230
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
MQL5 support #45
base: master
Are you sure you want to change the base?
MQL5 support #45
Conversation
I added new type of MQL application called 'Service' with MQL5 support for DWX_ZeroMQ.
Fix small bugs at ServiceInit and DWX_GetTickData functions.
I added modified DWX_ZeroMQ_Connector file ready to work with service implemented in DWX_ZeroMQ_Service_v1.0.0.mq5 file.
I added Execution and Reporting modules modified to work with api implemented in DWX_ZeroMQ_Connector_v1_0.py file
I added Strategy module modified to work with api implemented in DWX_ZeroMQ_Connector_v1_0.py file.
I added coin flip traders strategy modified to work with api implemented in DWX_ZeroMQ_Connector_v1_0.py file.
I added few examples.
I corrected few spelling mistakes and reformatted data presentation.
Hi @artbert , Thank you so much for your contribution! 😃 We'll review this as soon as we can - many thanks! |
Hi, |
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.
Proper README doc. Tested the code and it works as expected on a standard fresh Windows 10 machine with MT5 64bit installed.
GOD Bless u, mate |
Run a separate pull request for a dockerize version of this connector and I'll help with the review. |
Any idea when this PR could be merged into master ? |
This would be great to merge into |
The pull request includes proposal of my version of MQL5 support.
The idea was to:
Please see [mql5/README.md] to read detailed information about my implementation.
I created new python files to not make mess in the project. Their names have '_MT5' suffix.
Final explanation of the ideas was placed as a commentary to the mql5 code.