From 97e015fa8e5a57a6fc6fcb9d727b6262048e30ea Mon Sep 17 00:00:00 2001 From: Artem Titov Date: Wed, 31 Jul 2019 14:31:18 +0200 Subject: [PATCH] Update metrics units for better fit in DefaultVideoQualityAnalyzer Bug: webrtc:10138 Change-Id: I823e21bd4e570511cffff1797ebdaae9adb2e4fd Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/147646 Reviewed-by: Mirko Bonadei Commit-Queue: Artem Titov Cr-Commit-Position: refs/heads/master@{#28719} --- .../pc/e2e/analyzer/video/default_video_quality_analyzer.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc index 499d04e2d5..30d021bbeb 100644 --- a/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc +++ b/test/pc/e2e/analyzer/video/default_video_quality_analyzer.cc @@ -619,16 +619,16 @@ void DefaultVideoQualityAnalyzer::ReportResults( stats.time_between_freezes_ms, "ms"); ReportResult("freeze_time_ms", test_case_name, stats.freeze_time_ms, "ms"); ReportResult("pixels_per_frame", test_case_name, - stats.resolution_of_rendered_frame, "unitless"); + stats.resolution_of_rendered_frame, "count"); test::PrintResult("min_psnr", "", test_case_name, stats.psnr.IsEmpty() ? 0 : stats.psnr.GetMin(), "dB", /*important=*/false); ReportResult("decode_time", test_case_name, stats.decode_time_ms, "ms"); test::PrintResult("dropped_frames", "", test_case_name, - frame_counters.dropped, "unitless", + frame_counters.dropped, "count", /*important=*/false); ReportResult("max_skipped", test_case_name, stats.skipped_between_rendered, - "unitless"); + "count"); } void DefaultVideoQualityAnalyzer::ReportResult(