MediaSDK icon indicating copy to clipboard operation
MediaSDK copied to clipboard

The SDK doesn't set mfxExtVideoSignalInfo::VideoFullRange for jpeg decoding.

Open xhaihao opened this issue 2 years ago • 8 comments

call MFXVideoDECODE_DecodeHeader with an attached MFX_EXTBUFF_VIDEO_SIGNAL_INFO buffer for jpeg decoding, however mfxExtVideoSignalInfo::VideoFullRange is not set for a full-range jpeg file.

xhaihao avatar Mar 31 '22 00:03 xhaihao

@chuanli1 Could you please take a look? If confirm it, we can assign one guy to fix it. Thanks

wangyan-intel avatar Mar 31 '22 01:03 wangyan-intel

@chuanli1 Could you please take a look? If confirm it, we can assign one guy to fix it. Thanks Sure, I will check it.

chuanli1 avatar Mar 31 '22 01:03 chuanli1

Hi @xhaihao, could you provide related clip and an app patch of attached MFX_EXTBUFF_VIDEO_SIGNAL_INFO buffer? Thanks.

chuanli1 avatar Apr 02 '22 02:04 chuanli1

@chuanli1 you may use a yuvj420p jpeg file

$ ffprobe in.jpeg 2>&1 | grep yuvj420p 

$ ffmpeg -hwaccel qsv -c:v mjpeg_qsv -i in.jpeg -f null -

A MFX_EXTBUFF_VIDEO_SIGNAL_INFO buffer is attached when using FFmpeg with mjpeg_qsv decoder to decode a JPEG file

xhaihao avatar Apr 02 '22 02:04 xhaihao

@chuanli1 @dmitryermilov any update on this issue?

uartie avatar Jul 19 '22 15:07 uartie

I check jpeg spec(itu-t81) and don’t find this color range syntax, so RT don't parser this flag from Jpeg header, and HW don’t need this flag for decoder, and this issue is not a bug, but a new feature.

chuanli1 avatar Jul 20 '22 00:07 chuanli1

@chuanli1 If so , may you set mfxExtVideoSignalInfo::VideoFullRange to 1 by default for jpeg decoding ?

xhaihao avatar Jul 25 '22 02:07 xhaihao

Yes, this is feasible, we will put this task in the to-do list.

chuanli1 avatar Aug 09 '22 07:08 chuanli1