From fac27ba7d50aee0f7b899a95f03dda9248539fa6 Mon Sep 17 00:00:00 2001
From: Christophe-Marie Duquesne <chm.duquesne@gmail.com>
Date: Wed, 17 Jul 2013 21:56:26 +0200
Subject: [PATCH] improved encoding options

---
 sigal/video.py | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/sigal/video.py b/sigal/video.py
index 559a860..e0a807b 100644
--- a/sigal/video.py
+++ b/sigal/video.py
@@ -33,15 +33,18 @@ def generate_video(source, outname, size, options=[]):
     :param options: array of options passed to ffmpeg
 
     """
-    # Until we find a better solution, we don't keep the h param: We want
-    # to preserve the ratio of the original file (resizeToFit is not
-    # available in ffmpeg) see
+    # 2013-07-16: Until we find a better solution, we don't keep the h
+    # param: We want to preserve the ratio of the original file
+    # (resizeToFit is not available in ffmpeg) see
     # http://stackoverflow.com/questions/8218363/maintaining-ffmpeg-aspect-ratio
+    #
+    # 2013-07-17: encoding options improved, thanks to
+    # http://ffmpeg.org/trac/ffmpeg/wiki/vpxEncodingGuide
     (w, h) = size
     with open("/dev/null") as devnull:
-        subprocess.call(['ffmpeg', '-i', source, '-y', '-vf',
-            "scale=%i:trunc(ow/a/2)*2'" % w] + options + [outname],
-            stderr=devnull)
+        subprocess.call(['ffmpeg', '-i', source, '-y', '-crf', '10',
+            '-b:v', '1.6M','-vf', "scale=%i:trunc(ow/a/2)*2'" % w] +
+            options + [outname], stderr=devnull)
 
 def generate_thumbnail(source, outname, box, format, fit=True, options=None):
     "Create a thumbnail image"