Indicator 'MyDrawRegionTest': Error on calling 'OnBarUpdate' method on bar 0: The cal

[sledge]

I think I need someone with some experience to help on this one...


NT8 output window:

Indicator 'MyDrawRegionTest': Error on calling 'OnBarUpdate' method on bar 0: The calling thread cannot access this object because a different thread owns it.



It is the DrawRegion command that is the issue... I'm not sure if it's complaining about "this" or myDataSeries1/2.

And I'm not sure which thread owns what where.

I'm at a loss.

			Draw.Region(this, "Zone1", CurrentBar, 0, myDataSeries1, myDataSeries2, Zone_1_Border, Zone_1_Color,Opacity );

[NinjaTrader_Matthew]

Does this work if you pass a Brush in to the Draw.Region() method directly, rather than the public brush you created?

Draw.Region(this, "Zone1", CurrentBar, 0, myDataSeries1, myDataSeries2, Brushes.Green, Brushes.LimeGreen, Opacity );

[sledge]

Does this work if you pass a Brush in to the Draw.Region() method directly, rather than the public brush you created?

Draw.Region(this, "Zone1", CurrentBar, 0, myDataSeries1, myDataSeries2, Brushes.Green, Brushes.LimeGreen, Opacity );

I hard coded Brushes.Red instead and it no longer gives that error message.

I'll work backwards from here.

[koganam]

I hard coded Brushes.Red instead and it no longer gives that error message.

I'll work backwards from here.

Try making the brush static.

[sledge]

I found VolumeZones which does it properly.

I couldn't find a Draw.Region example in the standard examples, and got fixated on that with the error message.


This fixes things..


Zone_1_Color = Brushes.Green;
Zone_1_Border = Brushes.LimeGreen;


Now to fix the code because I have no region.

[koganam]

I found VolumeZones which does it properly.

I couldn't find a Draw.Region example in the standard examples, and got fixated on that with the error message.


This fixes things..


Zone_1_Color = Brushes.Green;
Zone_1_Border = Brushes.LimeGreen;


Now to fix the code because I have no region.

Interesting and strange. There does not seem to be any Intellisense for Brushes, or indeed anything in the SharpDX context. Does not even tell you if it is a namespace or a class. Hm.

[sledge]

INteresting and strange. There does not seem to be any Intellisense for Brushes, or indeed anything in the SharpDX context. Does not even tell you if it is a namespace or a class. Hm.

Which is why I panic'd and found some other stuff to try, as I figured I didn't have the right USING.

See the other reported post in the bug section...

[NinjaTrader_Matthew]

For the intelliprompt SharpDX and Media.Brushes stuff, we are looking into this further. This issue is tracked with NTEIGHT-8179 but not ETA on a resolution. In the mean time, Visual Studio will give you full intellisense support...

Regarding the way you were creating a new SolidColorBrush(), this approach would work if you did that >= State.DataLoaded, since OnBarUpdate and the object would be running from the same thread at that point (your State.SetDefaults is running on the UI).

Also if you still do not have a region, I think you need to swap your 0 and Current Bar arguments and it should work... I see that our documentation suggests the other way and that this used to work in NT7 so I will also have that checked out.

[gregid]

Is there a recommended way of creating Brush from Color that avoids the pitfalls of the op's original approach (ie: new SolidColorBrush(Colors.LimeGreen))

[sledge]

Also if you still do not have a region, I think you need to swap your 0 and Current Bar arguments and it should work... I see that our documentation suggests the other way and that this used to work in NT7 so I will also have that checked out.

It wasn't drawing until I moved my bars because of the bug in this post:

http://www.ninjatrader.com/support/forum/showthread.php?t=73746

And opacity appears to be 0 to 100 now, not 0 to 10... so 5 was pretty light.

[NinjaTrader_Matthew]

It wasn't drawing until I moved my bars because of the bug in this post:

http://www.ninjatrader.com/support/forum/showthread.php?t=73746

And opacity appears to be 0 to 100 now, not 0 to 10... so 5 was pretty light.

Yes - you are right. And I can confirm that was fixed with the new @Regions DrawingTool which will be available in next built.

Also correct about opacity.

[NinjaTrader_Matthew]

Is there a recommended way of creating Brush from Color that avoids the pitfalls of the op's original approach (ie: new SolidColorBrush(Colors.LimeGreen))

If you're just pushing a brush to the UI, you should just be able to use a predefined SolidColorBrush object

MyBrush = Brushes.LimeGreen;

If for any reason you need to create a new Brush object, you'll have to do that from something > State.Configure

[gregid]

If you're just pushing a brush to the UI, you should just be able to use a predefined SolidColorBrush object

MyBrush = Brushes.LimeGreen;

If for any reason you need to create a new Brush object, you'll have to do that from something > State.Configure

Is it > State.Configure or >= State.Configure?

[NinjaTrader_Matthew]

> State.Configure.

You would want to make sure we're out of the configure state, which reside on the UI. So anything beyond State.Configure, such as DataLoad, Historical, etc.

Be aware that resources created in State.Configure are not disposed of here immediately and will be kept utilized by NS objects that are in grids (NS strategies in strategy grid) even if the object is not enabled.

Try to create resources in State.Historical or State.DataLoaded though some scenarios require the creation in State.Configure.

[RJay]

[QUOTE=sledge;409688]I think I need someone with some experience to help on this one...


NT8 output window:

Indicator 'MyDrawRegionTest': Error on calling 'OnBarUpdate' method on bar 0: The calling thread cannot access this object because a different thread owns it.


I have come across a similar situation.

This line of code compiles but sends the same error to output window when the indicator is loaded to a chart.

string BarHeight = Bars.BarsPeriod.Value.ToString();

Error on calling 'OnBarUpdate' method on bar 5567: The calling thread cannot access this object because a different thread owns it.

.