ClearOutputWindow with multiple output windows

[jeronymite]

ClearOutputWindow with multiple output windows

Thanks, Matthew.

I totally agree with you that simplicity as a design concept is ideal. The caveat to that is clarity and effectiveness of function.

Thanks for listening.

[NinjaTrader_Matthew]

We are in favor of simplicity. Providing an overload still does not prevent situations where another script clears the output window unexpectedly. I will not argue the system can't stand to be improved, but we're interested in seeing what sort of enhanced add-on output window or debugging tools the community can come up with. We'll definitely put your feedback down to be reviewed at the point our plate is cleared to revisit the output system.

[jeronymite]

ClearOutputWindow with multiple output windows

Thanks, Matthew and particularly, thanks koganam!

Matthew, I would recommend that in this case "simplicity" needs to be superseded in favour of accuracy and extensibility. The potential is not insignificant for "confusing" situations where multiple strategies writing to both windows could cause "unexpected" clearing.

As koganam so eloquently suggests, the functionality really should be to allow the method to have an overload specifying the Output window to be cleared (by index or whatever is appropriate). Of course, two different signatures could be created to allow a convenient form of backwards compatibility -- one with no parameter, as currently, that would function as you indicate; the other with the Output window parameter.

In a separate thread (http://www.ninjatrader.com/support/f...ad.php?t=74170), I recommend extending the Output window functionality, and if this is implemented (please ), such an update to ClearOutputWindow would be essential.

Thanks.

[koganam]

For simplicity, ClearOutputWindow will work as stated, however if you'd like, you can always call NinjaTrader.Code.Output.Reset(PrintTo outputTab) (which will be available in the next beta build)

Which I shall promptly stick in a partial class as

protected void ClearOutputWindow(PrintTo outputTab)
{
NinjaTrader.Code.Output.Reset(outputTab);
return;
}

So that works really nicely too.

[NinjaTrader_Matthew]

Maybe better to provide an overload where we can specify the exact window to clear?

For simplicity, ClearOutputWindow will work as stated, however if you'd like, you can always call NinjaTrader.Code.Output.Reset(PrintTo outputTab) (which will be available in the next beta build)

[koganam]

Thanks for your report. There is an issue where the method is defaulting to the Output tab 1. Beta 2 will include a fix where the ClearOutputWindow() method will clear the output tab that was previously set by PrintTo variable.

NTEIGHT-8254

Maybe better to provide an overload where we can specify the exact window to clear?

[NinjaTrader_Matthew]

Thanks for your report. There is an issue where the method is defaulting to the Output tab 1. Beta 2 will include a fix where the ClearOutputWindow() method will clear the output tab that was previously set by PrintTo variable.

NTEIGHT-8254

[jeronymite]

It's not clear from the documentation how one invokes ClearOutputWindow() now that there is more than one output window. Obviously, one wants to be able to clear any single window or both.