[FFmpeg-devel] [PATCH] tiny_ssim: add per-frame metrics and final ssim db number.

[Michael Niedermayer]

On Tue, Feb 18, 2014 at 07:27:37PM +0000, Derek Buitenhuis wrote:
> On 2/18/2014 7:22 PM, Ronald S. Bultje wrote:
> > It was a suggestion on IRC to use isatty and use \n instead of \r if not a
> > TTY. I can remove that to make it stand-alone again. I don't care much for
> > that part, I'm more interested in the db aspect - I use the per-frame stats
> > just as a measure of monitoring progress (i.e. "is it hanging" versus "my
> > video has a lot of frames").
> 
> My personal preference would be to use \n everywhere, but that's not
> great for an 'interactive' application, I suppose.
> 

> Whatever people want is OK.

+1


> 
> Patch itself LGTM.

applied

if someone wants to remove the isatty, thats fine with me ...

thx

...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Good people do not need laws to tell them to act responsibly, while bad
people will find a way around the laws. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140219/8258b434/attachment.asc>