Use of ISeries<double> as Input

[aligator]

Dear Support,

Using an ISeries double as input to MAX generates CS1502 and CS1503 errors. What is the proper code for use an ISeries<double> as input to MAX?

double Vrg = SMA(Volume, Period)[0];
double Vro = (Volume[0] / Vrg);
double Vmax = MAX(Vro, Period);

Many thanks.

[bltdavid]

Vro is not an ISeries.

Did you try,

double Vmax = Math.Max(Vro, Period);

[aligator]

Vro is not an ISeries.

Did you try,

double Vmax = Math.Max(Vro, Period);

Thanks, but totally two different things.
Math.Max() finds the highest of two values whereas MAX() finds the highest value of a series over a period.

[NinjaTrader_JoshG]

Hello aligator,
Thanks for your post.

You need to use a Series as an input. In your snippet "Vro" is not a series, it's a simple variable.

Also, make sure to add the index your are checking from at the end of MAX()

double Vrg = SMA(Volume, Period)[0];
Vro[0] = (Volume[0] / Vrg);
double Vmax = MAX(Vro, Period)[0];

[aligator]

Hello aligator,
Thanks for your post.

You need to use a Series as an input. In your snippet "Vro" is not a series, it's a simple variable.

Also, make sure to add the index your are checking from at the end of MAX()

double Vrg = SMA(Volume, Period)[0];
Vro[0] = (Volume[0] / Vrg);
double Vmax = MAX(Vro, Period)[0];

Thanks Josh. Here is the code and (I think) I am using a Series as input. But still getting the same error. I must be overlooking something.

namespace NinjaTrader.NinjaScript.Indicators
{
    public class ET : Indicator
    {
        private Series<double>        Vro;
        private SMA                    Vrg;
        private MAX                    Vmax;
        private MIN                  Vmin;

        protected override void OnStateChange()
        {
            if (State == State.SetDefaults)
            {
                Description                    = @"";
                Name                        = "ET";
                IsSuspendedWhileInactive    = true;
                Period                        = 40;

                AddPlot(new Stroke(Brushes.Orange, 4), PlotStyle.Bar, "Value");
                AddLine(Brushes.SlateBlue,    0,    NinjaTrader.Custom.Resource.NinjaScriptIndicatorZeroLine);
            }

            else if (State == State.DataLoaded)
            {
                Vro        = new Series<double>(this);
                Vrg     = SMA(Volume, Period);
            }            
        }

        protected override void OnBarUpdate()
        {
            if (CurrentBar < Period)
                    return;

            // ET parameters and Plots
            double Vrg    = SMA(Volume, Period)[0];
            Vro[0]        = (Volume[0] / Vrg);
            double Vmax    = MAX(Vro, Period);
            double Vmin    = MIN(Vro, Period);

            Value[0]     = (Vro - Vmin) * 100 / (Vmax - Vmin);

        }

        #region Properties

        [Browsable(false)]
        [XmlIgnore()]
        public Series<double> Value
        {
            get { return Values[0]; }
        }    

        [Range(1, int.MaxValue), NinjaScriptProperty]
        [Display(ResourceType = typeof(Custom.Resource), Name = "Period", GroupName = "NinjaScriptParameters", Order = 0)]
        public int Period
        { get; set; }
        #endregion
    }
}

[NinjaTrader_JoshG]

aligator,

You are overlooking the index for MAX() and MIN() similar to my sample.

 double Vmax = MAX(Vro, Period)[SIZE=18px][B][0][/B][/SIZE];

You can use any index you require for your logic, but if you want the Period to be calculated from the most recent bar you need to use a BarsAgo of 0

[aligator]

aligator,

You are overlooking the index for MAX() and MIN() similar to my sample.

double Vmax = MAX(Vro, Period)[SIZE=18px][B][0][/B][/SIZE];

You can use any index you require for your logic, but if you want the Period to be calculated from the most recent bar you need to use a BarsAgo of 0

Josh,

Thanks a bunch!