[FFmpeg-devel] [PATCH] video stabilization plugins using vid.stab library

[Georg Martius]

On Thursday 25 April 2013 12:12:06 Clément Bœsch wrote:
> On Thu, Apr 25, 2013 at 10:13:30AM +0200, Clément Bœsch wrote:
> [...]
> 
> > > > BTW, this may be unrelated to this patch, but there is possibly a ref
> > > > counting issue somewhere; I have an undefined behaviour reproducible
> > > > with
> > > > vidstabtransform and ffplay (sometimes it has no effect). Input frame
> > > > being read-only or no doesn't seem to change anything (output being
> > > > changed is still somehow random whatever the input frame state). There
> > > > is
> > > > possibly a problem in the library (I'm using Git version), or maybe in
> > > > FFmpeg codebase.
> > > 
> > > Can you give me a cmd-line to reproduce?
> > 
> > ffplay ... -vf vidstabtransform
> 
> Problem is in the library. It seems to assume dst linesize == src
> linesize in the transform function. I can send a patch eventually.
I see. I will take a look. 
> 
> BTW, the code of the library could be ported to FFmpeg without much
> effort. Also, most of the functions you use in the library have ASM
> optimized versions in FFmpeg, so it could be made faster without much
> effort.
Sounds interesting. I will still continue with the separate lib for a while, 
because I am planning some larger changes. Afterwards I will consider.

Georg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130426/3ff75748/attachment.asc>