Skip to content

time zone mismatch #31

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

Open
chillmelon opened this issue Apr 17, 2025 · 3 comments
Open

time zone mismatch #31

chillmelon opened this issue Apr 17, 2025 · 3 comments

Comments

@chillmelon
Copy link

Mt5.history_deals_get((datetime.today() - timedelta(days=1)), datetime.now())[-1]
This line will cause index out of range error because datetime use local time zone and mt5 use UTC.

@Joaopeuko
Copy link
Owner

Joaopeuko commented Apr 28, 2025

@chillmelon , I just release a new version. Would you mind to test it and let me know if it is fixed?

pip install mqpy

https://pypi.org/project/mqpy/

I also added new documentation. Still have some things to improve,

@chillmelon
Copy link
Author

chillmelon commented May 8, 2025

@chillmelon , I just release a new version. Would you mind to test it and let me know if it is fixed?

pip install mqpy

https://pypi.org/project/mqpy/

I also added new documentation. Still have some things to improve,

I've tested the new version 0.6.11. AFAIK the time issue is solved, that's nice. But here are still some issues I found.

  • The hardcoded type_filling will result in a invalid request.
  • Mt5.symbol_info(self.symbol).trade_mode seems to return wrong value everytime. I believe this is a bug of MT5 API.
  • This library does not support hedging account yet it's not mentioned in the documentation.

Thank you for building this project, I'm happy to help.

@Joaopeuko
Copy link
Owner

Thanks for the message and for taking the time to run those checks. I appreciate it. I haven’t had much time lately to improve this project, so your help is welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants