Help with compile error when importing indicator from NT6.5 to NT7

August 14th, 2014, 02:39 PM

Hi,
I found an old indicator on my old laptop, and I was curious to see if I could have better use of it with a few years more trading experience.
I have ZERO programming experience though, so when I look through the NT support forum, it's like martian to me.

When I compile the indicator I get the following message:
No overload for method 'DrawText' takes '10' arguments

The error code is CS1501

The same error shows in 7 different places. I am cutting and pasting the 7 different lines.

DrawText("tot"+totcntr, ". " + tot.ToString() + " .", 0, getY(lowestprice)-TickSize, Color.White, fnt2, StringAlignment.Center, Color.Black, Color.Black, 6);
else if (tot < 0)
DrawText("tot"+totcntr, ". " + tot.ToString() + " .", 0, getY(lowestprice)-TickSize, Color.White, fnt2, StringAlignment.Center, Color.Black, Color.Red, 6);
else
DrawText("tot"+totcntr, ". " + tot.ToString() + " .", 0, getY(lowestprice)-TickSize, Color.Black, fnt2, StringAlignment.Center, Color.Black, Color.LimeGreen, 6);
}

DrawText("tag"+cnt, "." + getStr(vol) + "*" + getStr(vol_diff), 0, getY(pr), Color.White, fnt, StringAlignment.Center, Color.Black, Color.Black, 8);
else if (vol_diff < 0)
DrawText("tag"+cnt, "." + getStr(vol) + "*" + getStr(abs), 0, getY(pr), Color.White, fnt, StringAlignment.Center, Color.Black, Color.Red, shade);
else
DrawText("tag"+cnt, "." + getStr(vol) + "*" + getStr(abs), 0, getY(pr), Color.Black, fnt, StringAlignment.Center, Color.Black, Color.LimeGreen, shade);
}

DrawText("tag"+cnt, "." + getStr(vol) + "*" + getStr(vol_diff), 0, getY(pr), Color.Black, fnt, StringAlignment.Center, Color.Black, Color.Orange, 8);
}

I would appreciate any help, Anders

August 14th, 2014, 04:08 PM

The signatures of the Draw methods changed between NT6.5 and NT7 so the ordering and number of parameters is slightly different. It is easy to fix.

August 18th, 2014, 03:07 PM

Zondor

The signatures of the Draw methods changed between NT6.5 and NT7 so the ordering and number of parameters is slightly different. It is easy to fix.

Hi Zondor,

Unfortunately I have zero programming skills, so what might seem easy to someone is not that clear to me. I don't even know where to start...

Anders

August 18th, 2014, 03:33 PM

bergvall63

Hi Zondor,

Unfortunately I have zero programming skills, so what might seem easy to someone is not that clear to me. I don't even know where to start...

Anders

If you post the cs file I will fix it.

If you open the file in the Ninjascript editor, the Intellisense popup feature will show you the parameters needed by the signature, which is how I will fix it. Which will only take a few minutes.

I won't import it. I will copy the cs file into the indicator folder and fix the compile errors one by one until they are all gone.

Lots of people went through this exact same exercise a few years ago, after NT7 first came out and legacy indicators stopped working. A similar situation looms in the near future with NT8, I can't wait.....

August 18th, 2014, 06:59 PM

Thanks Zondor

I really appreciate it.

August 20th, 2014, 11:25 AM

For NT7, two additional parameters, the boolean AutoScale and the integer y pixel offset, need to be included in the signatures of the DrawText method. This should work.

August 21st, 2014, 03:42 AM

Zondor

For NT7, two additional parameters, the boolean AutoScale and the integer y pixel offset, need to be included in the signatures of the DrawText method. This should work.

Correct about the params and thanks because I wasn't aware of pixelOffset as I don't use this overload, but you still need to pass in a double (usually price) for Y, as well as the pixel offset, so I don't think the GetY calls are what you want in that code as the value returned is inappropriate but the int returned would still get recast to double by the compiler.

e.g.

Code

        protected override void OnBarUpdate()
        {
			if (CurrentBar % 100 == 0)
            {
				DrawText("tag"+CurrentBar, true, "show tag"+CurrentBar, 0, High[0] /*double Y*/, 0 /*int pixelOffset*/, 
							Color.Black, ChartControl.Font, StringAlignment.Center, Color.Black, Color.Blue, 5);
			}
        }

You could use the ChartControl/Plot bounds and work in integer pixel space using the Windows direct draw methods instead but that's not usually necessary.

August 21st, 2014, 05:50 AM

Zondor

For NT7, two additional parameters, the boolean AutoScale and the integer y pixel offset, need to be included in the signatures of the DrawText method. This should work.

Thank you so much for taking this on. Unfortunately I still get 4 error messages, new ones this time. I'm attaching an image of the errors. If you feel you don't have time to mess with this I totally understand. Again thanks for your help

Anders

Disregard the new error, I screwed up when I imported the Indicator!!
Again THANK YOU for helping out!

August 21st, 2014, 05:59 AM

ratfink

Correct about the params and thanks because I wasn't aware of pixelOffset as I don't use this overload, but you still need to pass in a double (usually price) for Y, as well as the pixel offset, so I don't think the GetY calls are what you want in that code as the value returned is inappropriate but the int returned would still get recast to double by the compiler.

e.g.

Code

        protected override void OnBarUpdate()
        {
			if (CurrentBar % 100 == 0)
            {
				DrawText("tag"+CurrentBar, true, "show tag"+CurrentBar, 0, High[0] /*double Y*/, 0 /*int pixelOffset*/, 
							Color.Black, ChartControl.Font, StringAlignment.Center, Color.Black, Color.Blue, 5);
			}
        }

You could use the ChartControl/Plot bounds and work in integer pixel space using the Windows direct draw methods instead but that's not usually necessary.

Thanks Ratfink for helping out. It's amazing to see how many nice guys/girls there are in this forum helping out us non-programmers. Much appreciated!

Anders

August 21st, 2014, 11:30 AM

What's really interesting about this indicator is that it opens a new window that appears to be an enhanced DOM and Time and Sales combination.

The Drawing of items on the price panel seems to work fine without having to make any changes to the GetY methods. A quick look shows prices are being multiplied by 10,000 and then being converted to integers, consistent with this indicator being specific to the Euro currency. Despite this, the objects seem to be appearing at the proper ordinates on the price panel of the chart.

The only thing I looked at originally was the syntax of the signatures generating the compile errors. Now it might be worthwhile to figure out what this indicator is trying to accomplish. It is accumulating real time data and does not backfill anything when it starts up.

Looks like it may be a cumulative time and sales indicator showing trades at the bid and ask. If this is the case, as it continues to run it will create a volume profile.

Help with compile error when importing indicator from NT6.5 to NT7

Discussion in NinjaTrader

Help with compile error when importing indicator from NT6.5 to NT7