Strategy Builder creates incorrect indicator signature

[saltminer]

Hi,
in debugging an inquiry from a customer, it appears that the Strategy Builder creates an incorrect signature for an indicator it is required to use, if that indicator has properties (used for parameter inputs) that are commented out. The Strategy Builder still uses them in the signature, which is incorrect.
Example:

//		[TypeConverter(typeof(SoundConverter))]
//		[NinjaScriptProperty]
//		[Display(Description = "Select audio alert sound.  Custom sound files must be .wav files and be added to the 'sounds' directory under the Program Files/NinjaTrader 8 install location.", ResourceType = typeof(Custom.Resource), Name = "Continuation", GroupName = "Alerts")]
//		public string Sound_ExtremeRepeat
//		{
//            get { return sound_ExtremeRepeat; }
//            set { sound_ExtremeRepeat= value; }
//		}

The correct signature for the indicator in this example, visible from Intellisense, is 8 parameters. The Strategy Builder has created a signature of ten parameters, the last two of which should not be there.

Cheers,
saltminer

[NinjaTrader_Jim]

Hello saltminer,

Thanks for writing in.

Could you provide me with a sample indicator and sample strategy that exhibit this behavior so I may test it on my end?

Please export the strategy and indicator and send the zip file as an attachment to platformsupport[at]ninjatrader[dot]com. Please reference the ticket number 1671518 and include the message "Attention Jim" and I will follow up with you via email.

To export a NinjaScript from NinjaTrader 8, please do the following:

• From the Control Center window select Tools > Export > NinjaScript Add-On...
• Click Add > Select the indicator/strategy > OK > Export.
• Choose a location to save the file.
• Then attach that file you saved.

I look forward to assisting you further.

[saltminer]

Hi Jim,
in preparing the resources to send to you I have determined the problem is a custom add-on that was adding parameters to the signature within Builder.
So, false alarm.

Cheers,
saltminer