diff options
author | Alessio Balsini <balsini@google.com> | 2024-03-12 00:52:19 +0000 |
---|---|---|
committer | Alessio Balsini <balsini@google.com> | 2024-03-12 00:52:19 +0000 |
commit | e8cea65741e47490bed5987aeb9d26fb25118821 (patch) | |
tree | c512e10bf3449eae0c01f8ede8b0591eb20f2228 | |
parent | f49a1949a48863babbba66081096fdc39d298502 (diff) | |
download | dittosuite-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.cpp | 3 |
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); } |