[FFmpeg-devel] [PATCH v12 0/8] [WIP] webp: add support for animated WebP decoding
Thilo Borgmann
thilo.borgmann at mail.de
Tue May 21 19:50:16 EEST 2024
Hi,
[...]
>>> Tests mostly work for me. There are a few images (that I reported
>>> earlier) that give:
>>
>> thanks for testing!
>>
>>
>>> Canvas change detected. The output will be damaged. Use -threads 1
>>> to try decoding with best effort.
>>> They don't animate without that option and with it render incorrectly.
>>
>> That issue yields from the canvas frame being the synchronization object
>> (ThreadFrame) - doing so prevents the canvas size changed mid-stream.
>> _Maybe_ this can be fixed switching the whole frame multithreading away
>> from ThreadFrame to sth else, not sure though and no experience with the
>> alternatives (AVExecutor?). Maybe Andreas can predict if it's
>> worth/valid to change that whole part of it? I'm not against putting
>> more effort into it to get it right.
I could fix 488x488.webp and have an almost identical output to libwebp.
488x488.webp features an ARGB canvas and has both, ARGB & YUVA420P
p-frames.
Do you have more files with other variations of canvas & p-frames? If
they at all exist... e.g. canvas YUV and p-frames RGB?
Pinged Meta as well for real-world samples. Will take some more days
until I get feedback. Will then post the next iteration...
Thanks,
Thilo
More information about the ffmpeg-devel
mailing list