base period type enum

[e-man]

greetings,

reviewing the Kagi code and would like to know how to expose Range bars as a valid selection for BasePeriodType. any helpful pointers most appreciated.

cheers,
-e

[NinjaTrader_Brett]

Hello,

This would be officially unsupported unfortunately, but mabye others can assist here. Its most likely unsupported as the two would contradict eachother/ cause issue correct?

Let me know if I can be of further assistance.

[e-man]

was hoping for something similar to this:

        public override Gui.Chart.ChartStyleType[] ChartStyleTypesSupported
        {
            get
            {
                return new Gui.Chart.ChartStyleType[]
                {
                    Gui.Chart.ChartStyleType.Box,
                    Gui.Chart.ChartStyleType.CandleStick,
                    Gui.Chart.ChartStyleType.HiLoBars,
                    Gui.Chart.ChartStyleType.LineOnClose,
                    Gui.Chart.ChartStyleType.OHLC,
                };
            }
        }

where i could just plug in the BasePeriodType[] values that were applicable.

anyone else run into this limitation before? i understand the official unsupported response, but would still like to explore how to do this.

cheers,
-e

EDIT: hmmm, i was close ... Supported : PeriodType[] may do the trick ... we'll see.

[e-man]

... denied ...

'NinjaTrader.Data.NewBarType.Supported.get': cannot override inherited member 'NinjaTrader.Data.BarsType.Supported.get' because it is not marked virtual, abstract, or override

any chance we can get this marked as virtual?

cheers,
-e

[NinjaTrader_RyanM1]

Thank you for the suggestion, e-man. Currently you can't use range bars for the Kagi bar type. I will forward this to our development team for consideration in a future release.

[e-man]

you can disregard that suggestion.

i coded it up over here and made it work, but there is no business use for this type of bar. the bar-heights can get ridiculous large in range-bound markets.

cheers,
-e

[NinjaTrader_Brett]

Hello,

Thanks for letting us know.