On 29/07/2020 07:08, Fu, Linjie wrote:
From: ffmpeg-devel <ffmpeg-devel-boun...@ffmpeg.org> On Behalf Of
Mark Thompson
Sent: Wednesday, July 29, 2020 06:50
To: ffmpeg-devel@ffmpeg.org
Subject: [FFmpeg-devel] [PATCH 1/6] vaapi_encode_h265: Remove
confusing and redundant tile options
The tile_rows/cols options currently do a confusingly different thing to
the options of the same name on other encoders like libvpx and libaom.
There is no backward-compatibility reason to implement the log2 behaviour
as there was for libaom, so just get rid of them entirely.
Ok, previously I'm following the implementation in librav1e [1] which uses a
non-log2
value as the input of "tile-rows" and "tile-columns".
(Do we need to make it consistent as well?)
[1] https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/librav1e.c#L567
Yeah, that seems bad. I sent a patch to fix it.
Thanks,
- Mark
_______________________________________________
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".
