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

[Georg Martius]

On Wednesday 24 April 2013 18:48:00 Clément Bœsch wrote:
> On Tue, Apr 23, 2013 at 01:22:15AM +0200, Clément Bœsch wrote:
> > On Tue, Apr 23, 2013 at 12:33:38AM +0200, Georg Martius wrote:
> > > On Friday 19 April 2013 02:04:20 Georg Martius wrote:
> > > [...]
> > > Sorry for the self-reply. I just wanted to remind about the patch
> > > (before it is not applyable again ;-))
> > 
> > Unless someone has an objection I will apply this (possibly with local
> > cosmetics) in the next 48h after another checking of the code.
> 
> Applied.
Great!
> 
> Note that I took the liberty to do some various cosmetics changes in the
> following commits.
Thanks.
> 
> Also, I fixed and simplified the direct path for vidstabdetect.
Okay, I see. Well if the show option is disabled, then the filter does not 
write something, so we don't need to make 'in' writeable. 
Should this simplification also be done for the transform plugin?
> 
> 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?

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/20130425/bf848636/attachment.asc>