EA31337 / Strategy-ZigZag

Strategy based on the ZigZag indicator.
7 stars 4 forks source link

Zigzag Strategy just open buy positon ,not sell #5

Open mavidelisi opened 1 year ago

mavidelisi commented 1 year ago

Hi Kenorb ;

when i test robot for zigzag strategy regardless timeframe , it just open buy position , not sell position.

Thanks.

kenorb commented 1 year ago

Thanks for the report.

EA Advanced v2.013.1 with MT5:

I've checked MT4 tests for ZigZag strategy, same issue for MT4:

Symbol                             EURUSD (Euro vs US Dollar)
Period                             1 Minute (M1) 2019.01.01 23:42 - 2019.01.30 23:59 (2019.01.01 - 2019.01.31)
Total trades                  1934 Short positions (won %)                  0 (0.00%) Long positions (won %)                             1934 (49.69%)

Signal logic can be found at https://github.com/EA31337/Strategy-ZigZag/blob/v1.013/Stg_ZigZag.mqh#L107-L116

I'll check and fix it in the next release.

kenorb commented 1 year ago

Fixed in a6f505e517ffa7eeb16068dcbe3174bcdac7f3f7.