Heavy CPU and memory usage using Dispatcher.InvokeAsync

[set2win]

I finally find a nice solution to my button issues but it is eating up computer resources. I did all the development on my desktop which has a good process or and 32gb ram and did notice a little lag in the strategy opening in the chart but then tried to use it on my VPS computer and it basically just hangs up or crashes it. Wondering if I am doing something wrong or there is another method that doesn't kill a computer. as soon as I comment out this code the computer and strategy are normal again.

Dispatcher.InvokeAsync((() =>
{
if(golongButton != null && goshortButton != null && exitButton !=null)
{
if(Position.MarketPosition == MarketPosition.Long)
{
golongButton.Content = "Go Long";
golongButton.Name = "GoLongButton";
golongButtonClicked = false;
longEnable = false;
}
else if (Position.MarketPosition == MarketPosition.Short)
{
goshortButton.Content = "Go Short";
goshortButton.Name = "GoShortButton";
goshortButtonClicked = false;
shortEnable = false;
}
else if (Position.MarketPosition == MarketPosition.Flat)
{
exitButton.Content = "Close All";
exitButtonClicked = false;
exitEnable = false;
}

}}));

[NinjaTrader_BrandonH]

Hello set2win,

Thanks for your post.

Is this code being used frequently?

Are you able to move that code to be in the button event instead of in OnBarUpdate()? If so, do you see improved performance when the code is moved?

I look forward to assisting further.

[set2win]

I added If (State == State.RealTime) prior to this Dispatcher as it is only needed to set the button state in realtime. This fixed the problem.