Building a NinjaTrader 8 Indicator: What Actually Went Wrong

Alexandru Stefan — Mon, 23 Feb 2026 20:04:56 +0000

I'm an IT Engineer with a background in banking systems who trades futures on the side for several years now. At some point the two worlds collided: I was spending too much time drawing levels by hand before each session. Previous day high/low, weekly open, initial balance, Globex session ranges; by the time I had everything laid out on a fresh chart, I'd already missed the first move. The software instinct kicked in: automate it!

So I built my own NinjaTrader 8 indicator. It turned out to be a lot harder than expected. NT8's rendering model, threading constraints, and some very specific C# compatibility quirks bit me repeatedly. Here's what actually went wrong and how I fixed it.

The rendering problem

NT8's standard Draw.HorizontalLine creates persistent chart objects. With 40–50 levels across all providers (previous day/week/month OHLC, opening range, initial balance, VWAP bands, Globex Asia/London), panning the chart starts to feel sluggish. The objects accumulate, and NT has to manage them all.

The fix was to skip the draw object system entirely for labels and hook directly into the native render cycle via SharpDX.Direct2D1. Labels are redrawn every frame at their current pixel position. No chart objects, no stale cleanup logic, and no pan lag.

protected override void OnRender(ChartControl chartControl, ChartScale chartScale)
{
    base.OnRender(chartControl, chartScale);

    if (!VisualPinLevelLabelsToRightEdge
        || chartControl == null || chartScale == null
        || ChartPanel == null || RenderTarget == null
        || _pinnedLineLabels.Count == 0)
    {
        return;
    }

    EnsurePinnedLineLabelResources();
    if (_pinnedLineLabelTextFormat == null || _pinnedLineLabelBrush == null)
        return;

    // Calculate dynamic layout bounds so labels don't clip off-screen
    float right = ChartPanel.X + ChartPanel.W - 6f;
    float left = Math.Max(ChartPanel.X + 2f, right - 420f);
    float width = Math.Max(20f, right - left);
    float height = Math.Max(12f, VisualLevelLabelFontSize + 6f);

    PinnedLineLabel[] labels = _pinnedLineLabels.ToArray();
    foreach (PinnedLineLabel label in labels)
    {
        // Translate price value to Y pixel coordinates in real-time
        float y = (float)chartScale.GetYByValue(label.Price);
        float top = y - (height / 2f);
        DxRectangleF layoutRect = new DxRectangleF(left, top, width, height);

        // Feed into SharpDX RenderTarget
        _pinnedLineLabelBrush.Color = ToDxColor(label.Brush, Colors.GhostWhite);
        RenderTarget.DrawText(
            label.Text,
            _pinnedLineLabelTextFormat,
            layoutRect,
            _pinnedLineLabelBrush,
            DxDrawTextOptions.NoSnap);
    }
}

The tradeoff: you now own the resource lifecycle. EnsurePinnedLineLabelResources() creates the TextFormat and SolidColorBrush on first use and re-creates them on window resize events. You have to dispose them manually or you'll leak GPU memory. Not hard, just something NT's normal drawing API would have handled for you.

Getting weekly and monthly levels without loading a year of bars

Previous week and month OHLC is where most indicators take a shortcut that hurts performance. The naive approach is to load hundreds of bars on the primary chart series and scan backward. On a tick chart or 1-minute chart with months of history, that's a lot of data.

Instead, I use AddDataSeries in State.Configure to add dedicated Weekly and Monthly bar series. NT sources those independently from the primary chart, so the indicator gets completed prior-period bars without any extra chart history. When the HTF series doesn't have enough completed bars yet (fresh chart load, limited data subscription), a BarsRequest fallback fires asynchronously and populates the same override slot once the data arrives.

The tricky part is that BarsRequest completes on a background thread. You can't touch NinjaScript objects from there; everything that touches the indicator state has to be marshalled back to the UI thread. Getting that callback chain right without deadlocking or writing stale data took a few iterations.

The bug that inflated Volume Profile by ~3x

Volume Profile requires tick data: every trade's price and size. NT8 delivers ticks via OnMarketData, and I was collecting MarketDataType.Last ticks inside that handler.

The problem: when you add secondary data series (AddDataSeries), OnMarketData fires for all series, not just the primary chart. The Week and Month series are completed historical bars with no real-time subscription, but the 1-minute series does receive live ticks, which meant each tick was arriving multiple times across series. The volume profile was showing roughly 3× actual volume, and the Point of Control was drifting to wherever the noise concentrated.

The fix is a single guard:

protected override void OnMarketData(MarketDataEventArgs e)
{
    if (BarsInProgress != 0) return; // primary series only
    if (e.MarketDataType != MarketDataType.Last) return;
    // ... collect tick
}

One line. Took longer than it should have to find because the profile looked reasonable, just slightly off. The tell was comparing POC against a reference chart with known values.

The startup freeze on dense charts

Load a week of 1-second bars on NQ and you're looking at hundreds of thousands of bars. For each bar, I was rendering active confluence zones, and each zone needed to know how many bars ago its session started, so I could anchor the zone's left edge at the session open.

My first implementation scanned Time[] backwards on every bar for every zone. On a fresh load that's O(bars × zones), and with that many bars and 20+ zones it was spending several seconds just on that loop during the historical pass. The chart appeared frozen.

The fix was a dictionary keyed by session ID, populated once per session boundary and looked up in O(1) for all subsequent bars. The scan still happens once per session, and after that it's a simple dictionary lookup.

NT8 will reject modern C# syntax at export time

NinjaTrader's compiled-assembly export doesn't support all C# language features, even when your local dotnet build passes clean. Specific things that broke at NT export time:

record types
with expressions
init-only setters
DateOnly / TimeOnly
Target-typed new() in some contexts

The indication is that the indicator imports fine as source but fails when you try to export as a compiled assembly. You won't see a meaningful error in NT; it just fails to export or silently misbehaves. The fix was to go through the codebase and replace every instance with older C# equivalents. Not glamorous, but necessary if you want to ship a compiled binary.

The result

After all of that, the indicator loads a dense chart cleanly, shows levels from all sources without hand-drawing anything, and clusters proximate levels into visual zones so it's obvious at a glance where multiple timeframes agree.

If you trade futures on NinjaTrader 8 and want to stop manually maintaining your reference levels, check out Key Levels PRO.