New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 593892 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Mar 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

PSNR, color pre- and post- spitzer

Project Member Reported by crouleau@chromium.org, Mar 10 2016

Issue description

Version: 50.0.2661.9
OS: Android

Nexus 5:
  Without Spitzer flag: Avg PSNR is 36.64 (https://av-analysis.corp.google.com/#/get-testpass-details/2/Video%20Stack/96681)

  With Spitzer flag: Avg PSNR is 30.55 (https://av-analysis.corp.google.com/#/get-testpass-details/2/Video%20Stack/96682)

If you look at the videos, you can see that the color is hugely different between the two videos. I assume that this is the reason for the PSNR difference. I manually reviewed results for Samsung Galaxy S3 and saw the same difference.

Color is complicated because it is subjective to some extent. I have added hubbe@ and cliffordcheng@, our resident color experts, to weigh in.
 
Cc: w...@chromium.org
Does this only happen for vpx content? IIRC we don't have color space information for our VP9 clips, so it's possible Android is assuming one always.
Oh sorry, yes, this is only for VP9 content (We don't have any automated VP8 tests).

Comment 3 by w...@chromium.org, Mar 10 2016

Yeah, I think this is most likely a difference in what color space is assumed in the absence of metadata.

Spitzer will default to BT709 for HD content. Presumably MediaPlayer is assuming BT601.

Even if re-encode the test videos to contain the metadata, I have no idea whether MediaPlayer will respect it. Its libvpx could be too old, and even if it's not, it might ignore it.

Comment 4 by w...@chromium.org, Mar 10 2016

One way to test would be to add a BT601 flag to the test video and see if the colors match. 

I can't test it right now, but if someone else wants to, I believe the vpxenc flag is --color-space=bt601
Status: WontFix (was: Untriaged)
I'm going to close this bug as expected behavior. Thanks for looking into it, watk@.
Which one looks better? :)
Labels: Hotlist-AVA

Sign in to add a comment