Skip to content

Commit c917713

Browse files
committed
Add log line if low confidence thermal result
1 parent 70f86c2 commit c917713

1 file changed

Lines changed: 12 additions & 4 deletions

File tree

api/Services/ResultHandlers/WorkflowResultHandlers/ThermalReadingResultHandler.cs

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,10 +50,7 @@ public async Task OnWorkflowCompleted(Workflow workflow)
5050

5151
await mqttPublisherService.PublishSaraAnalysisResultAvailable(message);
5252

53-
if (result?.Confidence is not null && result.Confidence > 0.99)
54-
{
55-
await TryUploadTimeseries(workflow, inspectionRecord, result);
56-
}
53+
await TryUploadTimeseries(workflow, inspectionRecord, result);
5754
}
5855

5956
private async Task TryUploadTimeseries(
@@ -71,6 +68,17 @@ private async Task TryUploadTimeseries(
7168
return;
7269
}
7370

71+
if (result.Confidence is not { } confidence || confidence <= 0.99)
72+
{
73+
logger.LogWarning(
74+
"Skipping thermal-reading timeseries upload for workflow {WorkflowId}: temperature {Temperature}°C, confidence {Confidence} did not meet threshold (> 0.99)",
75+
workflow.Id,
76+
result.Temperature,
77+
result.Confidence
78+
);
79+
return;
80+
}
81+
7482
var uploadRequest = new TriggerTimeseriesUploadRequest
7583
{
7684
Name =

0 commit comments

Comments
 (0)