heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

media_types/video/processing.py: Move store_metadata

Browse Source 
Move store_metadata to self.common_setup() so that it's easy
to parallelize self.transcode() later.
This commit is contained in:
vijeth-aradhya 2017-06-02 01:55:42 +05:30
parent f7e1bfea7a
commit 2963b0a182
1 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
mediagoblin/media_types/video/processing.py
View File

@ -178,6 +178,14 @@ class CommonVideoProcessor(MediaProcessor):
self.transcoder = transcoders.VideoTranscoder() self.transcoder = transcoders.VideoTranscoder()
self.did_transcode = False self.did_transcode = False
# Extract metadata and keep a record of it
self.metadata = transcoders.discover(self.process_filename)
# metadata's stream info here is a DiscovererContainerInfo instance,
# it gets split into DiscovererAudioInfo and DiscovererVideoInfo;
# metadata itself has container-related data in tags, like video-codec
store_metadata(self.entry, self.metadata)
def copy_original(self): def copy_original(self):
# If we didn't transcode, then we need to keep the original # If we didn't transcode, then we need to keep the original
if not self.did_transcode or \ if not self.did_transcode or \
@ -246,20 +254,12 @@ class CommonVideoProcessor(MediaProcessor):
if self._skip_processing('webm_video', **file_metadata): if self._skip_processing('webm_video', **file_metadata):
return return
# Extract metadata and keep a record of it orig_dst_dimensions = (self.metadata.get_video_streams()[0].get_width(),
metadata = transcoders.discover(self.process_filename) self.metadata.get_video_streams()[0].get_height())
# metadata's stream info here is a DiscovererContainerInfo instance,
# it gets split into DiscovererAudioInfo and DiscovererVideoInfo;
# metadata itself has container-related data in tags, like video-codec
store_metadata(self.entry, metadata)
orig_dst_dimensions = (metadata.get_video_streams()[0].get_width(),
metadata.get_video_streams()[0].get_height())
# Figure out whether or not we need to transcode this video or # Figure out whether or not we need to transcode this video or
# if we can skip it # if we can skip it
if skip_transcode(metadata, medium_size): if skip_transcode(self.metadata, medium_size):
_log.debug('Skipping transcoding') _log.debug('Skipping transcoding')
dst_dimensions = orig_dst_dimensions dst_dimensions = orig_dst_dimensions