> -----Original Message----- > From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > Thilo Borgmann > Sent: Thursday, December 12, 2019 22:15 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and cbf_cr > for transform depth 4 > > Hi, > > Am 12.12.19 um 08:27 schrieb Fu, Linjie: > >> -----Original Message----- > >> From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of > >> Carl Eugen Hoyos > >> Sent: Wednesday, December 11, 2019 18:19 > >> To: FFmpeg development discussions and patches <ffmpeg- > >> de...@ffmpeg.org> > >> Subject: Re: [FFmpeg-devel] [PATCH] lavc/hevc_cabac: fix cbf_cb and > cbf_cr > >> for transform depth 4 > >> > >> Am Mi., 11. Dez. 2019 um 09:57 Uhr schrieb Linjie Fu <linjie...@intel.com>: > >>> > >>> The max transform depth is 5(from 0 to 4), so we need 5 cabac states for > >>> cbf_cb and cbf_cr. > >>> > >>> See Table 9-4 for details. > >> > >> Does this change the output of a file? > >> If yes, please provide it / implement a fate test. > > > > Yes, this issue was found in some license-limited clips and does affect the > decoding. > > Hence it takes some efforts to generate an available file for verifying/test > in the > > community. > > > > I've created a ticket for this and attached the clips and the decode result > with/without > > this patch: > > https://trac.ffmpeg.org/ticket/8425 > > > > Please help to verify. > > please add a fate test that uses your .h265 sample to verify decoding. > Ping for this patch and the test in fate, thx.
- linjie _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
