From 6220aaf5bd9b66422ccacf165f54d5b20c79cb22 Mon Sep 17 00:00:00 2001
From: Antonio Ospite <ao2@ao2.it>
Date: Mon, 5 Dec 2016 15:35:31 +0100
Subject: [PATCH 1/1] vidi-player.py: add an option to add a background layer
 with the "rest" sample

Adding a background representing rests looks a lot better than having
a black screen flashing in and out.
---
 vidi-player.py | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/vidi-player.py b/vidi-player.py
index fed0687..4ac51a6 100755
--- a/vidi-player.py
+++ b/vidi-player.py
@@ -22,6 +22,9 @@ import sys
 import mido
 import vidi
 
+# TODO: turn that into a command line option
+ADD_REST_BACKGROUND = True
+
 
 def is_note(msg):
     return msg.type == 'note_on' or msg.type == 'note_off'
@@ -69,6 +72,10 @@ def timeline_from_midi(midi_file, video_font_path):
 
             timeline.add_clip(video_sample_path, start_time, duration)
 
+    if ADD_REST_BACKGROUND:
+        rest_sample_path = "%s/sample_rest.png" % video_font_path
+        timeline.add_layer_clip(rest_sample_path, 0, elapsed_time)
+
     return timeline
 
 
-- 
2.1.4