[FFmpeg-trac] #8862(ffplay:reopened): Wrong color space selected

FFmpeg trac at avcodec.org
Thu Apr 15 06:48:38 EEST 2021


#8862: Wrong color space selected
------------------------------------+------------------------------------
             Reporter:  airbete     |                    Owner:
                 Type:  defect      |                   Status:  reopened
             Priority:  normal      |                Component:  ffplay
              Version:  git-master  |               Resolution:
             Keywords:              |               Blocked By:
             Blocking:              |  Reproduced by developer:  0
Analyzed by developer:  0           |
------------------------------------+------------------------------------
Changes (by Balling):

 * status:  closed => reopened
 * resolution:  invalid =>


Comment:

 Okay, so this is just a bug here.
 https://github.com/FFmpeg/FFmpeg/blob/573f05a7533cd9aed3ed895b4fa4ad8fcba4e56a/fftools/ffplay.c#L968

 See https://github.com/libsdl-org/SDL/issues/2860

 So frame->colorspace == AVCOL_SPC_SMPTE170M (SMPTE 170M matrix) should
 force that mode, but it does not.

 Also why AVCOL_SPC_SMPTE240M matrix uses BT.601?? It is a different matrix
 as derived from SMPTE 170M primaries (yeah, SMPTE 170M matrix is not
 derived from SMPTE 170M primaries, I know, confusing).
 P.S.
 Obviously it is not primaries that define the matrix, even though it
 should be (not the case for JPEG (BT.601 a.k.a. SMPTE 170M matrix is
 derived from System M primaries ACTUALLY, see SMPTE 177 and
 https://github.com/AOMediaCodec/libavif/pull/430#discussion_r610920396)
 but BT.709 is the default in JPEG, since that is used in the default sRGB,
 also not the case for DCI-P3 in BT.2020 matrix as Netflix uses).

 I think we should try to fix that. Or at least deprecate ffplaying sYCC
 files...

 Since they migrated to github I will try to open an issue there too.

--
Ticket URL: <https://trac.ffmpeg.org/ticket/8862#comment:7>
FFmpeg <https://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list