136 RC3 - wrong fibo levels show in AW

Wrong fibolevel is read in to AW:

//--------------------------------------------------------------------
// Trading rule: Long entry (On Bar Open)
//--------------------------------------------------------------------                   
if ((LongEntrySignal
   and Not LongExitSignal)
   and (MarketPosition("Current", MagicNumber, "") is Not Long))
{
    // Action #1
    Open Long order at (Fibo(Main chart, Open-Close previous week, Fibo Level = -38.2)[] + (ATR(Main chart,ATRPeriod1)[5] * PriceEntryMult1)) Stop;
        Order valid for BarsValid bars;
        Duplicate trades: allowed; 
        Replacing pending orders: allowed; 
        Stop Loss = SLm * ATR(mATRp);
        Profit target = TPm * ATR(mATRp);

        Trailing Stop = TSm * ATR(mATRp);

        Exit After ExitAfterBars bars;
}


//--------------------------------------------------------------------
// Trading rule: Short entry (On Bar Open)
//--------------------------------------------------------------------                   
if ((ShortEntrySignal
   and Not ShortExitSignal)
   and (MarketPosition("Current", MagicNumber, "") is Not Short))
{
    // Action #1
    Open Short order at (Fibo(Main chart, Open-Close previous week, Fibo Level = 138.2)[] - (ATR(Main chart,ATRPeriod1)[5] * PriceEntryMult1)) Stop;
        Order valid for BarsValid bars;
        Duplicate trades: allowed; 
        Replacing pending orders: allowed; 
        Stop Loss = SLm * ATR(mATRp);
        Profit target = TPm * ATR(mATRp);

        Trailing Stop = TSm * ATR(mATRp);

        Exit After ExitAfterBars bars;
}

Attachments
Screenshot_396.png
(60.72 KiB)
  • Votes +2
  • Project AlgoWizard
  • Type Bug
  • Status Fixed
  • Priority Normal

History

b
#1

bentra

06.12.2022 10:58

Task created

b
#2

bentra

06.12.2022 11:11
Voted for this task.
TT
#3

Tamas

06.12.2022 14:17

Status changed from New to Waiting for information

Please attach the problematic strategy to reproduce the bug.
b
#4

bentra

06.12.2022 15:08
I emailed them because they were too big.
b
#5

bentra

07.12.2022 16:34
OK email didnt work so here is the sharing link for the strategies:

https://1drv.ms/u/s!AmIAw-1nBHjgkrQZTOBCQ93e7vdkCw?e=p2LH7K


TT
#6

Tamas

26.04.2023 13:34

Status changed from Waiting for information to Fixed

No file attached,  I am closing it
b
#7

bentra

26.04.2023 22:57

Attachment Screenshot_657.png added

Screenshot_657.png
(59.56 KiB)
You don't need a file. You can see the problem in the code 138.2 is missing. (derived as the negation of -38.2)
So too is
176.4 (negated from  -76.4)
361.8 (negated from -261.8)
and many others....

It looks like the coder failed to derive the negations properly for line #81. For instance, it looks like the coder is assuming the negation of 423.6 is -423.6 but actually it is -323.6. So you are missing many selectable options.

It might be easier just to make it a numerical input instead of a selector....

g
#8

Lee Guan Chuan

27.04.2023 10:55
Hello,


Our team has discussed on this issue and decided to add in the missing numbers. We are looking for references on the for the missing numbers.


Thank you.

E
#9

Emmanuel

27.04.2023 14:53
Voted for this task.
b
#10

bentra

27.04.2023 14:56
I totally forgot the fibo calculations were changed here:
https://roadmap.strategyquant.com/tasks/sq4_9109

So, this problem is already solved!


The new strategies will not be negated this way! The negation of -76.4 is actually -76.4!!!


You can actually just close this ticket! Sorry for wasting time! 



Votes: +2

Drop files to upload

or

choose files

Max size: 5MB

Not allowed: exe, msi, application, reg, php, js, htaccess, htpasswd, gitignore

...
Wait please