Convert to Python3, also clean up the code

This commit is contained in:
PokestarFan 2019-04-12 21:14:38 -04:00 committed by Alex Natskovich
parent f246344cc7
commit 3e0b659e3d

View file

@ -1,11 +1,13 @@
#!/usr/bin/env python
from __future__ import print_function
import csv
import subprocess
import math
import json
import math
import os
import shlex
import subprocess
from optparse import OptionParser
@ -43,7 +45,6 @@ def split_by_manifest(filename, manifest, vcodec="copy", acodec="copy",
except IndexError as e:
raise IndexError("No . in filename. Error: " + str(e))
for video_config in config:
split_str = ""
split_args = []
try:
split_start = video_config["start_time"]
@ -55,11 +56,11 @@ def split_by_manifest(filename, manifest, vcodec="copy", acodec="copy",
filebase = ".".join(filebase.split(".")[:-1])
split_args += ["-ss", str(split_start), "-t",
str(split_length), filebase + "." + fileext]
str(split_length), filebase + "." + fileext]
print("########################################################")
print("About to run: "+" ".join(split_cmd+split_args))
print("About to run: " + " ".join(split_cmd + split_args))
print("########################################################")
subprocess.check_output(split_cmd+split_args)
subprocess.check_output(split_cmd + split_args)
except KeyError as e:
print("############# Incorrect format ##############")
if manifest_type == "json":
@ -72,17 +73,20 @@ def split_by_manifest(filename, manifest, vcodec="copy", acodec="copy",
print(e)
raise SystemExit
def get_video_length(filename):
output = subprocess.check_output(("ffprobe", "-v", "error", "-show_entries", "format=duration", "-of", "default=noprint_wrappers=1:nokey=1", filename)).strip()
def get_video_length(filename):
output = subprocess.check_output(("ffprobe", "-v", "error", "-show_entries", "format=duration", "-of",
"default=noprint_wrappers=1:nokey=1", filename)).strip()
video_length = int(float(output))
print("Video length in seconds: "+str(video_length))
print("Video length in seconds: " + str(video_length))
return video_length
def ceildiv(a, b):
return int(math.ceil(a / float(b)))
def split_by_seconds(filename, split_length, vcodec="copy", acodec="copy",
extra="", video_length=None, **kwargs):
if split_length and split_length <= 0:
@ -92,7 +96,7 @@ def split_by_seconds(filename, split_length, vcodec="copy", acodec="copy",
if not video_length:
video_length = get_video_length(filename)
split_count = ceildiv(video_length, split_length)
if(split_count == 1):
if split_count == 1:
print("Video length is less then the target split length.")
raise SystemExit
@ -110,83 +114,83 @@ def split_by_seconds(filename, split_length, vcodec="copy", acodec="copy",
split_start = split_length * n
split_args += ["-ss", str(split_start), "-t", str(split_length),
filebase + "-" + str(n+1) + "-of-" + \
str(split_count) + "." + fileext]
print("About to run: "+" ".join(split_cmd+split_args))
subprocess.check_output(split_cmd+split_args)
filebase + "-" + str(n + 1) + "-of-" +
str(split_count) + "." + fileext]
print("About to run: " + " ".join(split_cmd + split_args))
subprocess.check_output(split_cmd + split_args)
def main():
parser = OptionParser()
parser.add_option("-f", "--file",
dest = "filename",
help = "File to split, for example sample.avi",
type = "string",
action = "store"
)
dest="filename",
help="File to split, for example sample.avi",
type="string",
action="store"
)
parser.add_option("-s", "--split-size",
dest = "split_length",
help = "Split or chunk size in seconds, for example 10",
type = "int",
action = "store"
)
dest="split_length",
help="Split or chunk size in seconds, for example 10",
type="int",
action="store"
)
parser.add_option("-c", "--split-chunks",
dest = "split_chunks",
help = "Number of chunks to split to",
type = "int",
action = "store"
)
dest="split_chunks",
help="Number of chunks to split to",
type="int",
action="store"
)
parser.add_option("-S", "--split-filesize",
dest = "split_filesize",
help = "Split or chunk size in bytes (approximate)",
type = "int",
action = "store"
)
dest="split_filesize",
help="Split or chunk size in bytes (approximate)",
type="int",
action="store"
)
parser.add_option("--filesize-factor",
dest = "filesize_factor",
help = "with --split-filesize, use this factor in time to" \
" size heuristics [default: %default]",
type = "float",
action = "store",
default = 0.95
)
dest="filesize_factor",
help="with --split-filesize, use this factor in time to"
" size heuristics [default: %default]",
type="float",
action="store",
default=0.95
)
parser.add_option("--chunk-strategy",
dest = "chunk_strategy",
help = "with --split-filesize, allocate chunks according to" \
" given strategy (eager or even)",
type = "choice",
action = "store",
choices = ['eager', 'even'],
default = 'eager'
)
dest="chunk_strategy",
help="with --split-filesize, allocate chunks according to"
" given strategy (eager or even)",
type="choice",
action="store",
choices=['eager', 'even'],
default='eager'
)
parser.add_option("-m", "--manifest",
dest = "manifest",
help = "Split video based on a json manifest file. ",
type = "string",
action = "store"
)
dest="manifest",
help="Split video based on a json manifest file. ",
type="string",
action="store"
)
parser.add_option("-v", "--vcodec",
dest = "vcodec",
help = "Video codec to use. ",
type = "string",
default = "copy",
action = "store"
)
dest="vcodec",
help="Video codec to use. ",
type="string",
default="copy",
action="store"
)
parser.add_option("-a", "--acodec",
dest = "acodec",
help = "Audio codec to use. ",
type = "string",
default = "copy",
action = "store"
)
dest="acodec",
help="Audio codec to use. ",
type="string",
default="copy",
action="store"
)
parser.add_option("-e", "--extra",
dest = "extra",
help = "Extra options for ffmpeg, e.g. '-e -threads 8'. ",
type = "string",
default = "",
action = "store"
)
dest="extra",
help="Extra options for ffmpeg, e.g. '-e -threads 8'. ",
type="string",
default="",
action="store"
)
(options, args) = parser.parse_args()
def bailout():
@ -197,7 +201,7 @@ def main():
bailout()
if options.manifest:
split_by_manifest(**(options.__dict__))
split_by_manifest(**options.__dict__)
else:
video_length = None
if not options.split_length:
@ -214,7 +218,8 @@ def main():
options.split_length = int(split_filesize / float(file_size) * video_length)
if not options.split_length:
bailout()
split_by_seconds(video_length=video_length, **(options.__dict__))
split_by_seconds(video_length=video_length, **options.__dict__)
if __name__ == '__main__':
main()