From d95345484d33e9bd644a614bd73b8e1773e5aa47 Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Wed, 21 Sep 2022 14:36:12 +0200 Subject: [PATCH] [PCLF] Fix exported test case for audio analyzer through new metrics API Bug: b/246095034 Change-Id: I94592e65f1bd33e82af83bf4f839351efcc42e62 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/276265 Commit-Queue: Artem Titov Reviewed-by: Mirko Bonadei Cr-Commit-Position: refs/heads/main@{#38157} --- .../audio/default_audio_quality_analyzer.cc | 27 ++++++++++--------- 1 file changed, 14 insertions(+), 13 deletions(-) diff --git a/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc b/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc index 63b5960e6a..83ae8f2655 100644 --- a/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc @@ -136,26 +136,27 @@ void DefaultAudioQualityAnalyzer::Stop() { item.second.preferred_buffer_size_ms, "ms", ImproveDirection::kNone); } else { - metrics_logger_->LogMetric("expand_rate", item.first, + metrics_logger_->LogMetric("expand_rate", GetTestCaseName(item.first), item.second.expand_rate, Unit::kUnitless, ImprovementDirection::kSmallerIsBetter); - metrics_logger_->LogMetric("accelerate_rate", item.first, + metrics_logger_->LogMetric("accelerate_rate", GetTestCaseName(item.first), item.second.accelerate_rate, Unit::kUnitless, ImprovementDirection::kSmallerIsBetter); - metrics_logger_->LogMetric("preemptive_rate", item.first, + metrics_logger_->LogMetric("preemptive_rate", GetTestCaseName(item.first), item.second.preemptive_rate, Unit::kUnitless, ImprovementDirection::kSmallerIsBetter); metrics_logger_->LogMetric( - "speech_expand_rate", item.first, item.second.speech_expand_rate, - Unit::kUnitless, ImprovementDirection::kSmallerIsBetter); - metrics_logger_->LogMetric("average_jitter_buffer_delay_ms", item.first, - item.second.average_jitter_buffer_delay_ms, - Unit::kMilliseconds, - ImprovementDirection::kNeitherIsBetter); - metrics_logger_->LogMetric("preferred_buffer_size_ms", item.first, - item.second.preferred_buffer_size_ms, - Unit::kMilliseconds, - ImprovementDirection::kNeitherIsBetter); + "speech_expand_rate", GetTestCaseName(item.first), + item.second.speech_expand_rate, Unit::kUnitless, + ImprovementDirection::kSmallerIsBetter); + metrics_logger_->LogMetric( + "average_jitter_buffer_delay_ms", GetTestCaseName(item.first), + item.second.average_jitter_buffer_delay_ms, Unit::kMilliseconds, + ImprovementDirection::kNeitherIsBetter); + metrics_logger_->LogMetric( + "preferred_buffer_size_ms", GetTestCaseName(item.first), + item.second.preferred_buffer_size_ms, Unit::kMilliseconds, + ImprovementDirection::kNeitherIsBetter); } } }