Looking for some help if I may. I want my code to PrintWithTimeStamp when an Entry signal is generated. The following code prints both the Long and Short signals at the same time, and seems to ignore all the code above it such as if (Position.MarketPosition != MarketPosition.Flat) return;.

Here you go:

 

Code

protected override void OnBarUpdate() { EntryHandling = EntryHandling.UniqueEntries;   if (Bars.FirstBarOfSession) { // AllTrades includes virtual and real-time trades. For real-time trades please use .RealtimeTrades priorTradesCumProfit = Performance.AllTrades.TradesPerformance.Currency.CumProfit; }   if (UseSessionPnL == true && (Performance.AllTrades.TradesPerformance.Currency.CumProfit - priorTradesCumProfit >= sessionProfitTarget || Performance.AllTrades.TradesPerformance.Currency.CumProfit - priorTradesCumProfit <= (-1*sessionLossLimit))) return;   if (Position.MarketPosition != MarketPosition.Flat) return;   if (UseTradeTime==true && ToTime(Time[0]) <= tradeStartTime || ToTime(Time[0]) >= tradeEndTime) return;    if (GetRandom() == (int) RandomLimit / 2) { if (GetRandom2() % 2 == 0) GoLong(); if (!Historical) {  PrintWithTimeStamp(Instrument.FullName + " STRATEGY Erratic NT7: LONG signal generated with ASK at: " + GetCurrentAsk()); //For basic TCA } else GoShort(); if (!Historical) {  PrintWithTimeStamp(Instrument.FullName + " STRATEGY Erratic NT7:SHORT signal generated with BID at: " + GetCurrentBid()); //For basic TCA } }   }

Any thoughts on what I'm doing wrong? Thanks!

I am not sure what is the exact logic you need but I would definitely take a closer look at your brackets - indentation may prove useful.

Anyway here is my try:

 

Code

if (GetRandom() == (int) RandomLimit / 2) {     if (GetRandom2() % 2 == 0)     {         GoLong();         if (!Historical)         {              PrintWithTimeStamp(Instrument.FullName + " STRATEGY Erratic NT7: LONG signal generated with ASK at: " + GetCurrentAsk()); //For basic TCA         }     }     else     {         GoShort();         if (!Historical)         {              PrintWithTimeStamp(Instrument.FullName + " STRATEGY Erratic NT7:SHORT signal generated with BID at: " + GetCurrentBid()); //For basic TCA         }     } }

gregid that worked. Thank you! Code is something from the Elite section I'm trying to improve on.