Parameters

[elitetradernyc]

Hey guys how do I turn my variables into parameters so that they can be optimized in the optimizer?

[NinjaTrader_AdamP]

elitetradernyw,

There is a section of code near the bottom of each indicator / strategy. It is in the Properties section. This needs to be added.

[Description("Slow period for a moving average")]
        [GridCategory("Parameters")]
        public int SlowPeriod
        {
            get { return slowPeriod; }
            set { slowPeriod = Math.Max(1, value); }
        }

[elitetradernyc]

Thanks, but when I add the variables and the code here, and I try to compile it gives me an error message, that I may have recursive properties that may cause ninjatrader to crash, how do I avoid this message?

[NinjaTrader_Lance]

Hello,


Could you please post a sample .cs file that is generating this issue?
Strategies are located in: (My)Documents\NinjaTrader 7\bin\Custom\Strategy

Additionally, please follow the instructions below to see where the errors are coming from after compiling the indicator. This will allow you to debug the indicator/strategy or remove it from your PC. If you are wondering why you receive an error when compiling only one indicator, it is because NinjaTrader always compiles all indicators and strategies - not just one. This is done to give you the highest runtime performance possible.

• Open NinjaTrader
• From the Control Center select the Tools menu--> select the Edit NinjaScript menu item--> select Indicator
• Select any indicator and double click on it (we need only to open and compile one script to see all compile errors for all files).
• A new window will appear and you will need to right click in the window and select Compile to compile the indicators.
• At the bottom of the window a new section will appear where you can find the error locations.
• From there you have the option to comment out offending code sections, remove the complete indicator or debug it to be able to compile again.

If you are unsure as to what the error is indicating, please send me a screenshot of the error with the name and description fields clearly readable.

To send a screenshot press Alt + PRINT SCREEN to take a screen shot of the selected window. Then go to Start--> Accessories--> Paint, and press CRTL + V to paste the image. Lastly, save as a jpeg file and send the file as an attachment.

We have also collected more comprehensive steps for resolving NinjaScript programming errors in this tip on our forums - http://www.ninjatrader.com/support/f...ead.php?t=4678

[elitetradernyc]

Lance, let me tell you what I'm doing and maybe you can help me from here. I simply add a new variable like:

private int slowAverage = 35; // This one already works.
private int MediumMarketEMA = 4; // This is a new one that does not work -Note not used in code.

Here is the code I add for the new one, that tells me i have a recursive problem:

[Description("MediumMarketEMA")]
[GridCategory("Parameters")]
public int MediumMarketEMA
{
get { return MediumMarketEMA; }
set { MediumMarketEMA = Math.Max(1, value); }
}

Here is error message:


your strategy likely holds one
or more recurisve properties which
could cause NinjaTrader to crash MediumMarketEMA

Do you want to edit these properties?

Any ideas?

[NinjaTrader_Lance]

Hello,

Please change to the following capitalization within your code:

//Variable name
[B]mediumMarketEMA [/B]= 4;

[Description("MediumMarketEMA")]
        [GridCategory("Parameters")]
        public int MediumMarketEMA
        {
            get { return [B]mediumMarketEMA[/B]; }
            set { [B]mediumMarketEMA [/B]= Math.Max(1, value); }
        }

Let me know if I can be of further assistance.

[elitetradernyc]

Hi Lance, unfortunately the error is still there with lower case caps. I even tried restarting my computer to see if the error would clear up. Any other ideas?

[NinjaTrader_Lance]

Hello,

Could you please post a sample .cs file that is generating this issue?
Strategies are located in: (My)Documents\NinjaTrader 7\bin\Custom\Strategy

I look forward to assisting you further.

[elitetradernyc]

Nevermind, it was caps, thanks for the tip though.