• If this is your first visit, you will have to register before you can post. To view messages, please scroll below and select the forum that you would like to visits. Questions? Be sure to check out the Forum FAQ.

Announcement

Collapse
No announcement yet.

Partner 728x90

Collapse

Time Limit for 2nd condition

Collapse
X
  • Filter
  • Time
  • Show
Clear All
new posts

    #16
    Originally posted by NinjaTrader_Paul View Post
    Hello GeorgeW,

    Thanks for your post.

    While we do not provide debugging service I would suggest having a careful review of the following statement:

    Code:
    			if(!check && (CrossAbove(myBsc.BuySellChangePlot, overboughtBSChange, 2)) 
    			&& (myBsc.BuySellChangePlot[0] > overboughtBSChange) 
    			&& myBsc.BuySellChangePlot[0]) > (myBsc.BuySellChangePlot[1]) 
    			&& (myBsl.BuySellLine[0] > OverboughtBSL) 
    			&& ((Close[0] > Open[0]) || ((Open[0] == Close[0]) 
    			&& (Math.Abs(High[0] - Open[0]) < Math.Abs(Low[0] - Open[0])))))				
                {                	
    				lvtBullBar = (Math.Round((myBsc[0]),2));
    				firstSig = Time[0];
    				check = true;
                }
    I've shown that the red "()" may not be correctly placed.
    Thanks, Paul. That's a copy and paste error on my part, as the original code has an opening bracket after && on your 3rd line.

    Comment


      #17
      Hi GeorgeW,

      Thanks for your reply.

      At this point I think you would need to embed your code with print statements to output each an every value and/or each and every condition so that you can evaluate bar by bar (or tick by tick) to understand how the logic is processing the data and correct from there.

      Here is a reference to our debug guide: http://ninjatrader.com/support/forum...ead.php?t=3418
      Paul H.NinjaTrader Customer Service

      Comment


        #18
        Solved the outstanding issues by:
        1. Removing "!check" from the first if statement. This results in all lvtBullBars being flagged as intended,
        2. Then in order to stop tovBullBars which occur before a lvtBullBar from being included in the netLvtTOV calculation, after the curly braces for the tovBullBar calculation has been closed, inserting a section of code to Array.Clear if there is another lvtBullBar. This in addition to the other statement later in the code to Array.Clear if a certain time limit is exceeded.

        Thanks for your assistance on this matter.

        Comment

        Latest Posts

        Collapse

        Topics Statistics Last Post
        Started by TraderCAD, Today, 04:27 PM
        1 response
        5 views
        0 likes
        Last Post TraderCAD  
        Started by DavidHP, Today, 04:11 PM
        0 responses
        1 view
        0 likes
        Last Post DavidHP
        by DavidHP
         
        Started by alexstox, 03-24-2019, 01:07 PM
        6 responses
        55 views
        0 likes
        Last Post NinjaTrader_PatrickG  
        Started by fredfred, Today, 11:29 AM
        3 responses
        12 views
        0 likes
        Last Post NinjaTrader_JoshG  
        Started by swcooke, Today, 10:38 AM
        3 responses
        12 views
        0 likes
        Last Post NinjaTrader_Jesse  
        Working...
        X