Draw.Region(this, "TMA Band", CurrentBar, 0, TMA(High, Period)[0], TMA(Low, Period)[0], Color, Opacity);
Announcement
Collapse
Looking for a User App or Add-On built by the NinjaTrader community?
Visit NinjaTrader EcoSystem and our free User App Share!
Have a question for the NinjaScript developer community? Open a new thread in our NinjaScript File Sharing Discussion Forum!
Have a question for the NinjaScript developer community? Open a new thread in our NinjaScript File Sharing Discussion Forum!
See more
See less
Partner 728x90
Collapse
NinjaTrader
What is the best way to pass an indicator value as series to Draw.Region?
Collapse
X
-
What is the best way to pass an indicator value as series to Draw.Region?
Draw.Region(this, "TMA Band", CurrentBar, 0, TMA(High, Period)[0], TMA(Low, Period)[0], Color, Opacity); -
Hello, thanks for writing in. That is the correct way to reference the latest indicator value. Are you getting any errors in the Log tab of the Control Center about the indicator? Also use Print() to print the values to confirm the indicator is seeing these values properly.
//right before calling Draw.Region:
Print("TMA(High, Period)[0] " + TMA(High, Period)[0]);
Print("TMA(Low, Period)[0] " + TMA(Low, Period)[0]);Chris L.NinjaTrader Customer Service
-
-
namespace NinjaTrader.NinjaScript.Indicators.My
{
public class MyTMABand : Indicator
{
protected override void OnStateChange()
{
if (State == State.SetDefaults)
{
Description = @"My TMA Band.";
Name = "MyTMABand";
Calculate = Calculate.OnPriceChange;
IsOverlay = true;
DisplayInDataBox = true;
DrawOnPricePanel = true;
DrawHorizontalGridLines = true;
DrawVerticalGridLines = true;
PaintPriceMarkers = true;
ScaleJustification = NinjaTrader.Gui.Chart.ScaleJustification.Right;
IsSuspendedWhileInactive = true;
Period = 13;
Color = Brushes.White;
Opacity = 50;
}
else if (State == State.Configure)
{
}
else if (State == State.DataLoaded)
{
}
}
protected override void OnBarUpdate()
{
Draw.Region(this, "TMA Band", CurrentBar, 0, TMA(High, Period)[0], TMA(Low, Period)[0], Color, Opacity);
}
region Properties
[NinjaScriptProperty]
[Range(1, int.MaxValue)]
[Display(Name="Period", Description="Period", Order=1, GroupName="Parameters")]
public int Period
{ get; set; }
[NinjaScriptProperty]
[XmlIgnore]
[Display(Name="Color", Description="Color", Order=2, GroupName="Parameters")]
public Brush Color
{ get; set; }
[Browsable(false)]
public string ColorSerializable
{
get { return Serialize.BrushToString(Color); }
set { Color = Serialize.StringToBrush(value); }
}
[NinjaScriptProperty]
[Range(0, 100)]
[Display(Name="Opacity", Description="Opacity", Order=3, GroupName="Parameters")]
public int Opacity
{ get; set; }
#endregion
}
}
Comment
-
Hi, this means you have the parameter list for Draw.Region incorrect. See all the valid overloads here:
Make sure you are using a valid override for this method:
Code:Draw.Region(NinjaScriptBase owner, string tag, int startBarsAgo, int endBarsAgo, ISeries<double> series, double price, Brush areaBrush, int areaOpacity, int displacement = 0) Draw.Region(NinjaScriptBase owner, string tag, int startBarsAgo, int endBarsAgo, ISeries<double> series1, ISeries<double> series2, Brush outlineBrush, Brush areaBrush, int areaOpacity, [int displacement]) Draw.Region(NinjaScriptBase owner, string tag, DateTime startTime, DateTime endTime, ISeries<double> series, double price, Brush areaBrush, int areaOpacity) Draw.Region(NinjaScriptBase owner, string tag, DateTime startTime, DateTime endTime, ISeries<double> series1, ISeries<double> series2, Brush outlineBrush, Brush areaBrush, int areaOpacity)
Chris L.NinjaTrader Customer Service
Comment
-
-
Hi, Please look at the first overload. The Region method needs two series, not a singular indicator value, which is a double value. It is also missing one more Brush color:
Draw.Region(this, "TMA Band", CurrentBar, 0, TMA(High, 14), TMA(Low, 14), Brushes.Red, Brushes.Blue, 10);Chris L.NinjaTrader Customer Service
- Likes 1
Comment
-
Yup, thank you so much...
Draw.Region(this, "TMA Band", CurrentBar, 0, TMA(High, Period), TMA(Low, Period), null, Color, Opacity);
I apparently deleted it the first couple of go arounds, but that did the trick.
Comment
Latest Posts
Collapse
Topics | Statistics | Last Post | ||
---|---|---|---|---|
Started by johnMoss, 03-04-2024, 01:53 PM
|
3 responses
46 views
0 likes
|
Last Post
by AndyM
Today, 06:13 PM
|
||
Started by algospoke, Today, 06:01 PM
|
0 responses
8 views
0 likes
|
Last Post
by algospoke
Today, 06:01 PM
|
||
Started by cre8able, Today, 05:53 PM
|
0 responses
8 views
0 likes
|
Last Post
by cre8able
Today, 05:53 PM
|
||
Started by Wessel, 03-23-2024, 02:37 PM
|
3 responses
24 views
0 likes
|
Last Post Today, 04:58 PM | ||
Started by martini, Yesterday, 04:45 PM
|
4 responses
32 views
0 likes
|
Last Post
by martini
Today, 04:48 PM
|
Comment