Unwanted mtultiple position entries/exits in one bar

[Waldo]

Hello,

recently I have noticed unwated behavior of my strategy. I think it is connected to the changes in code which solved my previous problem within this thread some months ago: http://www.ninjatrader.com/support/f...ad.php?t=68028

The problem is that now it can happen that multiple trades are executed within the same day even though the strategy is intended to execute orders only at the beginning of each trading day.

Please see attached print screen from the executions and also the whole strategy script.

Could you please advice me how to adjust the script so that only 1 position per day will be permitted, but at the same time I would like to prevent the problem from previous thread to occur once again.

Should you need additional information from me, please let me know.

Thank you and regards,
Waldo

[NinjaTrader_ChelseaB]

Hi Waldo,

Thanks for your post.

You will need to add logic to your script to prevent further entries after there has been an entry.

For example:
In #region Variables:
private bool tradeExecuted = false;

In OnBarUpdate():
if (Bars.BarsSinceSession == 0)
tradeExecuted = false;

if (tradeExecuted == true)
return;

Within the namespace:
protected override void OnExecution(IExecution execution)
{
if (execution.Order.OrderAction == OrderAction.Buy)
tradeExecuted = true;
}

[Waldo]

Thank you for a fast reply. I have tried to add the code into the script, but unfortunately the result is the same. Please find attached the adjusted script. Could you please check if it was added in the correct sections and if not then edit it accordingly? I have marked the respective rows with comment "// ADDED" within the code.

With regards,
Waldo

[NinjaTrader_ChelseaB]

Hi Waldo,

I'd like schedule a call with you to investigate on your end.

I'm testing on my end and only getting one trade per day so it does appear to work OK. (Even though this should allow for multiple trades at the beginning of the session and then not again until the next session, I am not actually getting multiple trades at the beginning of the session)

Please send an email to platformsupport [at] ninjatrader [dot] com. In the email please add a link to this forum thread.

[NinjaTrader_ChelseaB]

Waldo,

Thanks for the time on the phone.

For anyone reading this thread, I did not understand the goal.

The goal was not to allow 1 trade per instrument at the beginning of the session, the goal was to allow only 1 instrument 1 trade per session.

To do this, instead of changing the bool to true in OnExecution, change the bool to true in each condition that allows for an order to be placed.

[Waldo]

Hello Chelsea,

I have added the change of bool to each condition as discussed, but the result is still the same. Any further ideas or support will be appreciated.

With regards,
Waldo

[NinjaTrader_ChelseaB]

Hi Waldo,

Please send me an email when you would like me to give you a call back.

[NinjaTrader_ChelseaB]

Hi Waldo,

Running this on a Daily chart exposed that the Bars.BarsSinceSession is always 0 and causing the tradeExecuted bool to always be reset.

To correct this we added code to require a new session and a new primary bar for the reset to occur.

[Waldo]

Hello Chelsea,

thank you once again for the support. This has done the trick, therefore you can close the thread.

With regards,
Waldo

[CactusMo]

Hello ChelseaB, I would like to have just one trade per day. Is there a way to use the Ninja Wizard strategy builder to make my automated strategy make just one trade per session or day? I know how to use the wizard but not so good with scripting.

[NinjaTrader_ChelseaB]

Hello CactusMo,

You can use a User Variable instead of a bool if you are using the wizard and you do not want the user to be able to change the bool.

After the first order is placed set Variable0 to 1. Then use the Misc -> Session break to reset to this to 0 on the new session. Only allow for orders if Variable0 is 0.

[CactusMo]

Hello Chelsea, I tried following your instructions as you so kindly relied, but I keep getting an error message after clicking the "Compile" button. Here is what I did.

Clicked: "User variables" then "Variable0" then "==" then "Misc" then "Numeric value" then change value from "0" to "1"
Clicked: "Session break" then "==" then "Misc" then "Numeric value" then left value at "0"
This is how I understand your instructions, but I am sure I have not executed your instructions properly. I did not include the rest of my code since it works just fine. The only problem is it executes more than one trade per day or per market session.

[NinjaTrader_ChelseaB]

Hello CactusMo,

Attached is an example.

Follow these steps to import the NinjaScript:

1. Download the script to your desktop, keep it in the compressed .zip file.
2. From the Control Center window select the menu File > Utilities > Import NinjaScript
3. Select the downloaded .zip file
4. NinjaTrader will then confirm if the import has been successful.

Critical *Note that on any files that say "File already exists on your PC" that start with an "@" symbol are the ones that came preloaded inside of NinjaTrader so you would say "No" so that you do not override those files.

[CactusMo]

Hello Chelsea, works perfect. Thanks!

[ttodua]

Is `if(FirstTickOfBar)` the same alternative? i use that to reset, and is that different that your solution?