Incorrect Pip Value on Micro Futures

[GitHubTrader]

Hello,

In version 2.27 this worked correctly in version 2.33 the Pip Value is wrong. I'm not much of a coder but I am trying to find the difference or what changed to make this happen, so if I find I'll post again. If someone has time to look into this it would be appreciated.

I've included screen shots showing the difference. The left side is the correct Pip value. I also imagine this may mess up the SL/TP on futures like S&P and Nasdaq as the SL/TP has to be set at .25 levels. I haven't tested to see if the SL/TP is properly place as of yet.

This is MT5 with Futures broker ampfutures

[EarnForex]

Hi!

Thanks for reporting the issue!
Unfortunately, I cannot test this as AMP Futures wouldn't send me the confirmation email to open an account. Would it be possible for you to open another demo account there and send me the credentials (to webmaster@earnforex.com)?

[GitHubTrader]

Hello, Do you have a google voice or some other number capable of receiving texts, I can use to setup an account? The require this and verify it one time. Thanks

…

On Wed, Feb 10, 2021, 09:06 EarnForex ***@***.***> wrote: Hi! Thanks for reporting the issue! Unfortunately, I cannot test this as AMP Futures wouldn't send me the confirmation email to open an account. Would it be possible for you to open another demo account there and send me the credentials (to ***@***.***)? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASZONZ62N5P65QARI5YZTF3S6KOGNANCNFSM4XMDUS7Q> .

[EarnForex]

What's the email address I can send you my phone number to?

[GitHubTrader]

Hi, I'll use it to setup a demo and hopefully it all works out so you'll have access going forward. I'll email you the login info right after, where you able to download mt5 from them?

…

On Thu, Feb 11, 2021, 04:07 EarnForex ***@***.***> wrote: What's the email address I can send you my phone number to? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASZONZYWDDQPXK6HGOSA5PDS6OT5HANCNFSM4XMDUS7Q> .

[GitHubTrader]

It's strange that you aren't able to setup an account as I know they have used from around the world. I ment to mention I tried applying for an account using your email address check your email to see if you received anything.

…

On Thu, Feb 11, 2021, 09:49 Gregory Boudro ***@***.***> wrote: Hi, ***@***.*** I'll use it to setup a demo and hopefully it all works out so you'll have access going forward. I'll email you the login info right after, where you able to download mt5 from them? On Thu, Feb 11, 2021, 04:07 EarnForex ***@***.***> wrote: > What's the email address I can send you my phone number to? > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <#10 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ASZONZYWDDQPXK6HGOSA5PDS6OT5HANCNFSM4XMDUS7Q> > . >

[EarnForex]

OK, thanks to @GitHubTrader I now have a working demo account.
I've looked into the issue with incorrect pip value and position size calculation. It seems, the problem is caused by either incorrect return of the TickValue for this symbols from the trading server or by incorrect MQL5 documentation on the calculation method (I don't remember if I tested Exchange Futures symbols with any other brokers). Most likely, it is the former as the TickValue right now returns 0.5, while this symbol is a $5 futures where $1 movement in the underlying index should cost $5.

Anyway, a rough fix can be applied by replacing line 5019 in PositionSizeCalculator.mqh:
if ((CalcMode == SYMBOL_CALC_MODE_FUTURES) || (CalcMode == SYMBOL_CALC_MODE_EXCH_FUTURES) || (CalcMode == SYMBOL_CALC_MODE_EXCH_FUTURES_FORTS))
with:
if ((CalcMode == SYMBOL_CALC_MODE_FUTURES) /*|| (CalcMode == SYMBOL_CALC_MODE_EXCH_FUTURES)*/ || (CalcMode == SYMBOL_CALC_MODE_EXCH_FUTURES_FORTS))

I will later look into the issue with the spread.

[EarnForex]

OK, I've looked into the spread issue, and it doesn't seem incorrect to me. The spread is calculated as Ask - Bid, which is exactly 25 points here:

Please let me know if I am missing something.

[EarnForex]

Solved in 2.34.