[FFmpeg-devel] [PATCH] avfilter/vf_transpose adding NPP transpose filter
Timo Rothenpieler
timo at rothenpieler.org
Mon Sep 3 13:42:33 EEST 2018
Minus the missing configure dependency, documentation entry and minor
bump, the filter looks fine to me code wise.
Two issues though:
From my understanding, it's not a transpose filter, but a rotation one,
fixed to 90 degree angles. Unless I'm missing something in the code,
it's not doing any actual flipping, like vf_transpose does.
It should also be calling nppiTranspose_8u_C1R somewhere after/before
the call to nppiRotate_8u_C1R. Or use a transformation matrix to do the
transposition in one step, assuming libnpp has a function for that.
It's possible that I missed it doing the transpose somewhere, so please
correct me if I'm wrong.
I'd also strongly prefer this filter to be compatible with the arguments
of normal vf_transpose, and behave the same for the same arguments.
Additional parameters with more functions are fine.
Timo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4538 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180903/b513cfaa/attachment.bin>
More information about the ffmpeg-devel
mailing list