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 <mbonadei@webrtc.org> Commit-Queue: Artem Titov <titovartem@webrtc.org> Cr-Commit-Position: refs/heads/master@{#28719}
This commit is contained in:
@ -619,16 +619,16 @@ void DefaultVideoQualityAnalyzer::ReportResults(
|
|||||||
stats.time_between_freezes_ms, "ms");
|
stats.time_between_freezes_ms, "ms");
|
||||||
ReportResult("freeze_time_ms", test_case_name, stats.freeze_time_ms, "ms");
|
ReportResult("freeze_time_ms", test_case_name, stats.freeze_time_ms, "ms");
|
||||||
ReportResult("pixels_per_frame", test_case_name,
|
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,
|
test::PrintResult("min_psnr", "", test_case_name,
|
||||||
stats.psnr.IsEmpty() ? 0 : stats.psnr.GetMin(), "dB",
|
stats.psnr.IsEmpty() ? 0 : stats.psnr.GetMin(), "dB",
|
||||||
/*important=*/false);
|
/*important=*/false);
|
||||||
ReportResult("decode_time", test_case_name, stats.decode_time_ms, "ms");
|
ReportResult("decode_time", test_case_name, stats.decode_time_ms, "ms");
|
||||||
test::PrintResult("dropped_frames", "", test_case_name,
|
test::PrintResult("dropped_frames", "", test_case_name,
|
||||||
frame_counters.dropped, "unitless",
|
frame_counters.dropped, "count",
|
||||||
/*important=*/false);
|
/*important=*/false);
|
||||||
ReportResult("max_skipped", test_case_name, stats.skipped_between_rendered,
|
ReportResult("max_skipped", test_case_name, stats.skipped_between_rendered,
|
||||||
"unitless");
|
"count");
|
||||||
}
|
}
|
||||||
|
|
||||||
void DefaultVideoQualityAnalyzer::ReportResult(
|
void DefaultVideoQualityAnalyzer::ReportResult(
|
||||||
|
Reference in New Issue
Block a user