New issue
Advanced search Search tips

Issue 719597 link

Starred by 1 user
Status: Fixed
Owner:
crouleau@chromium.org
Closed: Jul 2017
Cc:
jrumm...@chromium.org
johnchen@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

decoded_frame_count perf test misleading

Project Member Reported by jrumm...@chromium.org, May 8 2017 
The linux-release/media.tough_video_cases/decoded_frame_count tests note that "lower is better". If this number represents the number of frames actually decoded, then it should be "higher is better".

Example: https://chromeperf.appspot.com/report?sid=9255d86c9c9a1a55029f5e741cf674c48ae3cda707d50781cbb9854abb519c5e&rev=467944

When it dropped from 550 to 300 that should be a problem, not an improvement.

Comment 2 by crouleau@chromium.org, May 8 2017

Cc: johnchen@chromium.org jrumm...@chromium.org
Status: Started (was: Assigned)
I'm going to look into the original design documents to try to understand what was intended here:

https://docs.google.com/document/d/1r7UY9n7ng7Z0VtDIWjhC9LbuIhMSiIdHgb-yTthsfKs/edit#

https://docs.google.com/document/d/1hPYo9BFwqFgHXrM0OslXOjjMLwDA0QWIiLwaWsfTHOY/edit#

Comment 3 by crouleau@chromium.org, Jun 7 2017 

Work is progressing here: https://docs.google.com/document/d/1TvUWPl6diK3DTJdSWMOcdz-6qnKC6KEHAZqoBBD8lIc/edit#heading=h.pbl9canvrafx

I think the right solution is to simply stop alerting on these.

Comment 4 by crouleau@chromium.org, Jul 10 2017

Status: Fixed (was: Started)
I fixed this to no longer alert on decoded_frame_count.
►

Sign in to add a comment