Skip to content

Commit 8b47b51

Browse files
author
aba
committed
In this python script, I added "import rospy" to be able to run this script in a Catkin environment.
Then, I removed the option "sameq" of the ffmpeg process. In ubuntu 14.04, ffmpeg is now called avconv, on my computer I have a symlink from avconv to ffmpeg, so calling ffmpeg still work. But with this version of FFMPEG (avconv 9.16), there is nolonger the sameq argument. Maybe it could be interesting to call avconc and rather than ffmpeg is ffmpeg executable is not found...
1 parent 6bc8a1d commit 8b47b51

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

bag_tools/scripts/bag_tools/make_video.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
PKG = 'bag_tools' # this package name
3434

3535
import roslib; roslib.load_manifest(PKG)
36-
36+
import rospy
3737
import tempfile
3838
import subprocess
3939
import glob
@@ -56,7 +56,7 @@ def create_video(tmp_dir, args):
5656
i = i + 1
5757

5858
rospy.loginfo('Creating video...')
59-
cmd = ["ffmpeg", "-f", "image2", "-r", str(args.fps), "-i", tmp_dir + "/img-%d.jpg", "-sameq", args.output]
59+
cmd = ["ffmpeg", "-f", "image2", "-r", str(args.fps), "-i", tmp_dir + "/img-%d.jpg", args.output]
6060
rospy.loginfo(' {}'.format(' '.join(cmd)))
6161
subprocess.call(cmd)
6262

@@ -81,4 +81,4 @@ def create_video(tmp_dir, args):
8181
import traceback
8282
traceback.print_exc()
8383
rospy.loginfo('Cleaning up temp files...')
84-
shutil.rmtree(tmp_dir)
84+
#shutil.rmtree(tmp_dir)

0 commit comments

Comments
 (0)