Skip to content

Reduce excessive debug logging in live trading #9220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Martin-Molinero merged 2 commits into from
Jan 13, 2026
+0 −12
Merged

Reduce excessive debug logging in live trading #9220

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
641010a
Reduce excessive logs
JosueNina Jan 13, 2026
aa8a60e
Revert unnecesary log changes
JosueNina Jan 13, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 0 additions & 12 deletions Engine/Results/LiveTradingResultHandler.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,6 @@ private void Update()

//Create and send back the changes in chart since the algorithm started.
var deltaCharts = new Dictionary<string, Chart>();
Log.Debug("LiveTradingResultHandler.Update(): Build delta charts");
var performanceCharts = new Dictionary<string, Chart>();
lock (ChartLock)
{
Expand All @@ -210,18 +209,15 @@ private void Update()
}
}
}
Log.Debug("LiveTradingResultHandler.Update(): End build delta charts");

//Profit loss changes, get the banner statistics, summary information on the performance for the headers.
var serverStatistics = GetServerStatistics(utcNow);
var holdings = GetHoldings(Algorithm.Securities.Values, Algorithm.SubscriptionManager.SubscriptionDataConfigService);

//Add the algorithm statistics first.
Log.Debug("LiveTradingResultHandler.Update(): Build run time stats");

var summary = GenerateStatisticsResults(performanceCharts).Summary;
var runtimeStatistics = GetAlgorithmRuntimeStatistics(summary);
Log.Debug("LiveTradingResultHandler.Update(): End build run time stats");


// since we're sending multiple packets, let's do it async and forget about it
Expand Down Expand Up @@ -613,7 +609,6 @@ protected override void Sample(string chartName, string seriesName, int seriesIn
return;
}

Log.Debug("LiveTradingResultHandler.Sample(): Sampling " + chartName + "." + seriesName);
lock (ChartLock)
{
//Add a copy locally:
Expand All @@ -633,7 +628,6 @@ protected override void Sample(string chartName, string seriesName, int seriesIn
//Add our value:
series.Values.Add(value);
}
Log.Debug("LiveTradingResultHandler.Sample(): Done sampling " + chartName + "." + seriesName);
}

/// <summary>
Expand All @@ -643,7 +637,6 @@ protected override void Sample(string chartName, string seriesName, int seriesIn
/// <seealso cref="Sample(string,string,int,SeriesType,ISeriesPoint,string)"/>
protected void SampleRange(IEnumerable<Chart> updates)
{
Log.Debug("LiveTradingResultHandler.SampleRange(): Begin sampling");
lock (ChartLock)
{
foreach (var update in updates)
Expand Down Expand Up @@ -679,7 +672,6 @@ protected void SampleRange(IEnumerable<Chart> updates)
}
}
}
Log.Debug("LiveTradingResultHandler.SampleRange(): Finished sampling");
}

/// <summary>
Expand Down Expand Up @@ -1047,8 +1039,6 @@ public virtual void ProcessSynchronousEvents(bool forceProcess = false)

if (time > _nextSample || forceProcess)
{
Log.Debug("LiveTradingResultHandler.ProcessSynchronousEvents(): Enter");

//Set next sample time: 4000 samples per backtest
_nextSample = time.Add(ResamplePeriod);

Expand Down Expand Up @@ -1090,8 +1080,6 @@ public virtual void ProcessSynchronousEvents(bool forceProcess = false)
}
}
}

Log.Debug("LiveTradingResultHandler.ProcessSynchronousEvents(): Exit");
}

/// <summary>
Expand Down