From c009f661e6f389fac532a9a794a95ff3f79a92f1 Mon Sep 17 00:00:00 2001
From: Jana Mizikova <mizikova.jana@gmail.com>
Date: Fri, 1 Nov 2019 17:06:35 +0100
Subject: [PATCH] fix(nks): thumbnails, build workfile with preview mov

 - thumbnail for clip is taken from middle of duration
---
 pype/nuke/lib.py                                   |  4 +++-
 pype/plugins/nukestudio/publish/collect_plates.py  |  6 ++++--
 pype/plugins/nukestudio/publish/collect_reviews.py | 11 ++++++++---
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/pype/nuke/lib.py b/pype/nuke/lib.py
index f182088457..cd809bf640 100644
--- a/pype/nuke/lib.py
+++ b/pype/nuke/lib.py
@@ -927,7 +927,7 @@ class BuildWorkfile(WorkfileSettings):
     def process(self,
                 regex_filter=None,
                 version=None,
-                representations=["exr", "dpx", "lutJson"]):
+                representations=["exr", "dpx", "lutJson", "mov", "preview"]):
         """
         A short description.
 
@@ -984,6 +984,8 @@ class BuildWorkfile(WorkfileSettings):
                                    version=version,
                                    representations=representations)
 
+        log.info("__ subsets: `{}`".format(subsets))
+
         nodes_backdrop = list()
 
         for name, subset in subsets.items():
diff --git a/pype/plugins/nukestudio/publish/collect_plates.py b/pype/plugins/nukestudio/publish/collect_plates.py
index 2ebbfde551..f9eb126772 100644
--- a/pype/plugins/nukestudio/publish/collect_plates.py
+++ b/pype/plugins/nukestudio/publish/collect_plates.py
@@ -196,12 +196,14 @@ class CollectPlatesData(api.InstancePlugin):
 
         thumb_file = head + ".png"
         thumb_path = os.path.join(staging_dir, thumb_file)
+        thumb_frame = instance.data["sourceIn"] + ((instance.data["sourceOut"] - instance.data["sourceIn"])/2)
 
-        thumbnail = item.thumbnail(instance.data["sourceIn"]).save(
+        thumbnail = item.thumbnail(thumb_frame).save(
             thumb_path,
             format='png'
         )
-        self.log.debug("__ thumbnail: {}".format(thumbnail))
+        self.log.debug("__ sourceIn: `{}`".format(instance.data["sourceIn"]))
+        self.log.debug("__ thumbnail: `{}`, frame: `{}`".format(thumbnail, thumb_frame))
 
         thumb_representation = {
             'files': thumb_file,
diff --git a/pype/plugins/nukestudio/publish/collect_reviews.py b/pype/plugins/nukestudio/publish/collect_reviews.py
index 9fab0f0741..f9032b2ca4 100644
--- a/pype/plugins/nukestudio/publish/collect_reviews.py
+++ b/pype/plugins/nukestudio/publish/collect_reviews.py
@@ -106,7 +106,6 @@ class CollectReviews(api.InstancePlugin):
 
     def create_thumbnail(self, instance):
         item = instance.data["item"]
-        source_in = instance.data["sourceIn"]
 
         source_path = instance.data["sourcePath"]
         source_file = os.path.basename(source_path)
@@ -119,11 +118,17 @@ class CollectReviews(api.InstancePlugin):
         thumb_file = head + ".png"
         thumb_path = os.path.join(staging_dir, thumb_file)
         self.log.debug("__ thumb_path: {}".format(thumb_path))
-        self.log.debug("__ source_in: {}".format(source_in))
-        thumbnail = item.thumbnail(source_in).save(
+
+        thumb_frame = instance.data["sourceIn"] + ((instance.data["sourceOut"] - instance.data["sourceIn"])/2)
+
+        thumbnail = item.thumbnail(thumb_frame).save(
             thumb_path,
             format='png'
         )
+        
+        self.log.debug("__ sourceIn: `{}`".format(instance.data["sourceIn"]))
+        self.log.debug("__ thumbnail: `{}`, frame: `{}`".format(thumbnail, thumb_frame))
+
         self.log.debug("__ thumbnail: {}".format(thumbnail))
 
         thumb_representation = {