Ftrack and review.

pype/plugins/harmony/publish/extract_render.py

@@ -25,7 +25,10 @@ class ExtractRender(pyblish.api.InstancePlugin):
             return [
                 about.getApplicationPath(),
                 scene.currentProjectPath(),
-                scene.currentScene()
+                scene.currentScene(),
+                scene.getFrameRate(),
+                scene.getStartFrame(),
+                scene.getStopFrame()
             ]
         }
         func
@@ -36,6 +39,9 @@ class ExtractRender(pyblish.api.InstancePlugin):
         application_path = result[0]
         project_path = result[1]
         scene_path = os.path.join(result[1], result[2] + ".xstage")
+        frame_rate = result[3]
+        frame_start = result[4]
+        frame_end = result[5]
 
         # Set output path to temp folder.
         path = tempfile.mkdtemp()
@@ -54,15 +60,18 @@ class ExtractRender(pyblish.api.InstancePlugin):
         harmony.save_scene()
 
         # Execute rendering.
-        output = pype.lib._subprocess([application_path, "-batch", scene_path])
-        self.log.info(output)
+        import subprocess
+        subprocess.call([application_path, "-batch", scene_path])
+        #output = pype.lib._subprocess([application_path, "-batch", scene_path])
+        #self.log.info(output)
 
         # Collect rendered files.
         files = os.listdir(path)
         collections, remainder = clique.assemble(files, minimum_items=1)
         assert not remainder, (
-            "There shouldn't have been a remainder for '%s': "
-            "%s" % (instance[0], remainder)
+            "There should not be a remainder for {0}: {1}".format(
+                instance[0], remainder
+            )
         )
         assert len(collections) == 1, (
             "There should only be one image sequence in {}. Found: {}".format(
@@ -76,7 +85,18 @@ class ExtractRender(pyblish.api.InstancePlugin):
             "ext": extension,
             "files": list(collections[0]),
             "stagingDir": path,
+            "frameStart": frame_start,
+            "frameEnd": frame_end,
+            "fps": frame_rate,
+            "preview": True,
+            "tags": ["review"]
         }
         instance.data["representations"] = [representation]
+        self.log.info(frame_rate)
+
+        # Required for extract_review plugin (L222 onwards).
+        instance.data["frameStart"] = frame_start
+        instance.data["frameEnd"] = frame_end
+        instance.data["fps"] = frame_rate
 
         self.log.info("Extracted {instance} to {path}".format(**locals()))