aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlessio Balsini <balsini@google.com>2024-03-12 00:52:19 +0000
committerAlessio Balsini <balsini@google.com>2024-03-12 00:52:19 +0000
commite8cea65741e47490bed5987aeb9d26fb25118821 (patch)
treec512e10bf3449eae0c01f8ede8b0591eb20f2228
parentf49a1949a48863babbba66081096fdc39d298502 (diff)
downloaddittosuite-e8cea65741e47490bed5987aeb9d26fb25118821.tar.gz
Fix missing mean value
For benchmarks that spawn multiple processes, the returned aggregated data was lacking the mean value. This was due to the mean value correctly exported by the child process, but mistakenly not copied into the parent. This patch fixes this issue. Test: manual execution of a multiprocessing workload Change-Id: If643932a88c95ae41145c58badfef4a61bfe1d71 Signed-off-by: Alessio Balsini <balsini@google.com>
-rw-r--r--src/result.cpp3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/result.cpp b/src/result.cpp
index 6a07bff..0726d34 100644
--- a/src/result.cpp
+++ b/src/result.cpp
@@ -426,7 +426,8 @@ std::unique_ptr<Result> Result::FromPb(const dittosuiteproto::Result& pb) {
auto result = std::make_unique<Result>(pb.name(), 1);
for (const auto& m : pb.metrics()) {
- Result::Statistics stats = {.min = m.min(), .max = m.max(), .median = m.median(), .sd = m.sd()};
+ Result::Statistics stats = {
+ .min = m.min(), .max = m.max(), .mean = m.mean(), .median = m.median(), .sd = m.sd()};
result->SetStatistics(m.name(), stats);
}