From 45be693f8c6e188d47ae64023db69b540f8c5e62 Mon Sep 17 00:00:00 2001
From: James Rowe <jroweboy@gmail.com>
Date: Mon, 12 Aug 2019 22:21:37 -0600
Subject: [PATCH] Add telemetry field for mean frametime
Previously, telemetry results couldn't give a good estimate for
performance over time, because it didn't include any fields related to
performance. With this, devs should be able to query metabase for mean
frametime to check for performance regressions after a change is made.
---
src/core/telemetry_session.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/core/telemetry_session.cpp b/src/core/telemetry_session.cpp
index 1049dd8c8..93a4136d7 100644
--- a/src/core/telemetry_session.cpp
+++ b/src/core/telemetry_session.cpp
@@ -99,6 +99,8 @@ TelemetrySession::~TelemetrySession() {
std::chrono::system_clock::now().time_since_epoch())
.count()};
AddField(Telemetry::FieldType::Session, "Shutdown_Time", shutdown_time);
+ AddField(Telemetry::FieldType::Session, "Mean_Frametime_MS",
+ Core::System::GetInstance().perf_stats->GetMeanFrametime());
#ifdef ENABLE_WEB_SERVICE
auto backend = std::make_unique<WebService::TelemetryJson>(Settings::values.web_api_url,