Beautify this code?

[tinfox]

RSI()."RSI" is less than 30 and
RSI()."RSI" from 1 bars ago is greater than 30 and
RSI()."RSI" from 2 bars ago is greater than 30 and
RSI()."RSI" from 3 bars ago is greater than 30 and
RSI()."RSI" from 4 bars ago is greater than 30 and
RSI()."RSI" from 5 bars ago is greater than 30 and
RSI()."RSI" from 6 bars ago is greater than 30 and
RSI()."RSI" from 7 bars ago is greater than 30

I want to take this to 20 to eliminate redundant alerts. Can I do something like "within 20 bars" or "within 1-20"?

 

[Svanoy]

@tinfox

def RSI = RSI().RSI;
def RSI20 = fold r = 1 to 21 with intr = 0 do if GetValue(RSI,r) > 30 then intr + 1 else intr;
def Cond = RSI20 == 20 and RSI < 30;

 

[armybender]

@tinfox

def RSI = RSI().RSI;
def RSI20 = fold r = 1 to 21 with intr = 0 do if GetValue(RSI,r) > 30 then intr + 1 else intr;
def Cond = RSI20 == 20 and RSI < 30;

Yep, this is it. Except, I think the 21 should be 20. I do like that you simplified your logical / boolean value as "RSI20 == 20" instead of putting the unnecessary "if RSI20 == 20 then 1 else 0" in there. Nice job. This should be marked as the answer.

 

[halcyonguy]

Yep, this is it. Except, I think the 21 should be 20. I do like that you simplified your logical / boolean value as "RSI20 == 20" instead of putting the unnecessary "if RSI20 == 20 then 1 else 0" in there. Nice job. This should be marked as the answer.

21 is correct. fold doesn't evaluate the 'to' number, it stops 1 number before it.

 

[Joshua]

RSI() < 30 and !Sum(RSI()[1] < 30,20)