help with division by 0 error

• 04/28/2021 at 1:16 PM #168284
  Mishap

  DEFPARAM CumulateOrders = TRUE // Cumulating positions deactivated DEFPARAM preloadbars = 1000 //Trigger to enter position (all other criteria must be met) //current time frame rsi1 = RSI[8](close) rsi2 = RSI[1](close) c1 = (rsi2 CROSSES over rsi1) //c2 = (close >= MA2) Timeframe (2 minutes) //base moving average indicator percentages 3 candle look back Ind1 = Average[15] (close[3]) Ind2 = Average[20] (close[3]) Ind3 = Average[25] (close[3]) Ind4 = Average[30] (close[3]) //moving average candle indicator percentages Ind5 = Average[15] (close) Ind6 = Average[20] (close) Ind7 = Average[25] (close) Ind8 = Average[30] (close) //get distance from averages in percentage relative to one above or below DistPer1 = abs(Ind1-Ind2)*100/Ind1 DistPer2 = abs(Ind2-Ind3)*100/Ind2 DistPer3 = abs(Ind3-Ind4)*100/Ind3 DistPer4 = abs(Ind5-Ind6)*100/Ind5 DistPer5 = abs(Ind6-Ind7)*100/Ind6 DistPer6 = abs(Ind7-Ind8)*100/Ind7 c3 = DistPer4-DistPer1 c4 = DistPer5-DistPer2 c5 = DistPer6-DistPer3 IF Ind5 > Ind6 AND Ind6 > Ind7 AND Ind7 > Ind8 THEN UpTrend = 1 ELSE UpTrend = 0 ENDIF Per1 = .001 Per2 = .002 Per3 = .003 // Conditions to enter long positions IF c1 AND c3 >= Per1 AND c4 >= Per2 AND c5 >= Per3 AND UpTrend THEN BUY 1 CONTRACTS AT MARKET ENDIF

  1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50

  DEFPARAM CumulateOrders = TRUE // Cumulating positions deactivated DEFPARAM preloadbars = 1000   //Trigger to enter position (all other criteria must be met)   //current time frame rsi1 = RSI[8](close) rsi2 = RSI[1](close) c1 = (rsi2 CROSSES over rsi1) //c2 = (close >= MA2) Timeframe (2 minutes) //base moving average indicator percentages 3 candle look back Ind1 = Average[15] (close[3]) Ind2 = Average[20] (close[3]) Ind3 = Average[25] (close[3]) Ind4 = Average[30] (close[3])     //moving average candle indicator percentages Ind5 = Average[15] (close) Ind6 = Average[20] (close) Ind7 = Average[25] (close) Ind8 = Average[30] (close)   //get distance from averages in percentage relative to one above or below DistPer1 = abs(Ind1-Ind2)*100/Ind1 DistPer2 = abs(Ind2-Ind3)*100/Ind2 DistPer3 = abs(Ind3-Ind4)*100/Ind3 DistPer4 = abs(Ind5-Ind6)*100/Ind5 DistPer5 = abs(Ind6-Ind7)*100/Ind6 DistPer6 = abs(Ind7-Ind8)*100/Ind7   c3 = DistPer4-DistPer1 c4 = DistPer5-DistPer2 c5 = DistPer6-DistPer3   IF Ind5 > Ind6 AND Ind6 > Ind7 AND Ind7 > Ind8 THEN UpTrend = 1 ELSE UpTrend = 0 ENDIF   Per1 = .001 Per2 = .002 Per3 = .003   // Conditions to enter long positions IF c1 AND c3 >= Per1 AND c4 >= Per2 AND c5 >= Per3 AND UpTrend THEN BUY 1 CONTRACTS AT MARKET ENDIF

  Hi Guys,

  Im hoping someone can point me in the right direction with where my code is causing division by 0 error. i think its in the “DistPer” section of the code. essentially im trying to get 3 different averages from 3 candles back & compare to the current average. if there is an increase in percentage as at the “Per” value. it keeps throwing the div by 0 error which i cant seem to clear.

  I have reviewed other threads around div by 0 error but haven’t found a workable solution as yet.

  any help is appreciated.

  cheers

  

  Mishap

  Participant

  Topics: 17

  Replies: 16

  Been thanked: 2 times
  04/28/2021 at 1:40 PM #168289
  robertogozzi

  Post your topic in the correct forum:
  _ ProRealTime Platform Support: only platform related issues.
  _ ProOrder: only strategy topics.
  _ ProBuilder: only indicator topics.
  _ ProScreener: only screener topics
  _ General Discussion: any other topics.
  _ Welcome New Members: for new forum members to introduce themselves.

  Thank you 🙂

  

  robertogozzi

  Moderator

  Topics: 69

  Replies: 17891

  Been thanked: 4505 times
  04/28/2021 at 1:57 PM #168290
  robertogozzi

  Don’t you need

  DistPerN = abs(Ind4-Ind3)*100/Ind4

  1	DistPerN = abs(Ind4-Ind3)*100/Ind4

  ?

  Try using GRAPH to monitor the variables involved in division:

  GRAPH Ind1 GRAPH Ind2 GRAPH Ind3 GRAPH Ind4 GRAPH Ind5 GRAPH Ind6 GRAPH Ind7

  1 2 3 4 5 6 7

  GRAPH Ind1 GRAPH Ind2 GRAPH Ind3 GRAPH Ind4 GRAPH Ind5 GRAPH Ind6 GRAPH Ind7

  or this version:

  GRAPH Ind1 = 0 GRAPH Ind2 = 0 GRAPH Ind3 = 0 GRAPH Ind4 = 0 GRAPH Ind5 = 0 GRAPH Ind6 = 0 GRAPH Ind7 = 0

  1 2 3 4 5 6 7

  GRAPH Ind1 = 0 GRAPH Ind2 = 0 GRAPH Ind3 = 0 GRAPH Ind4 = 0 GRAPH Ind5 = 0 GRAPH Ind6 = 0 GRAPH Ind7 = 0

   

  

  robertogozzi

  Moderator

  Topics: 69

  Replies: 17891

  Been thanked: 4505 times
  04/28/2021 at 7:55 PM #168310
  Jean-Christophe

  Hello Mishap, the division by 0 error is due to

  RSI[1](close)

  1	RSI[1](close)

  I had exactly the same error when running a strategy live and I searched during a long time before finding the solution.

  Try to use at least

  RSI[2](close)

  1	RSI[2](close)

  as the parameter must be an integer greater than 1

  1 user thanked author for this post.

  GraHal

  

  Jean-Christophe

  Participant

  Topics: 5

  Replies: 11

  Been thanked: 2 times
  09/12/2022 at 11:07 AM #200609
  thomas_price

  Jean-Christophe wrote:

  Hello Mishap, the division by 0 error is due to

  RSI[1](close)

  1	RSI[1](close)

  I had exactly the same error when running a strategy live and I searched during a long time before finding the solution.

  Try to use at least

  RSI[2](close)

  1	RSI[2](close)

  as the parameter must be an integer greater than 1

  is it true ? someone else has the same issue with [1] previous value ?

  🙂

  

  thomas_price

  Participant

  Topics: 5

  Replies: 15

  Been thanked: 0 times
  09/12/2022 at 2:09 PM #200619
  JC_Bywan

  Hi, you shouldn’t read [1] in RSI[1](close) as a previous value here, but as the RSI period.

  For  [1] to be a previous value, it would have to be at the end: RSI[period](close)[1]

  

  JC_Bywan

  Moderator

  Topics: 20

  Replies: 2779

  Been thanked: 1057 times
• Author
  Posts