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

Derek Buitenhuis derek.buitenhuis at gmail.com
Tue Feb 18 20:27:37 CET 2014


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.

Patch itself LGTM.

I will be nice not to have to deal with Xiph's "special" dump_ssim.

- Derek


More information about the ffmpeg-devel mailing list