From 6ae5e2d8a54c76240169d1cb3d5513738448a48d Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 15:13:57 +0100
Subject: [PATCH 01/29] fix(nuke): separate write render and prerender create
 plugins

---
 ...ate_write.py => create_write_prerender.py} | 107 ++----------------
 .../nuke/create/create_write_render.py        | 101 +++++++++++++++++
 2 files changed, 108 insertions(+), 100 deletions(-)
 rename pype/plugins/nuke/create/{create_write.py => create_write_prerender.py} (54%)
 create mode 100644 pype/plugins/nuke/create/create_write_render.py

diff --git a/pype/plugins/nuke/create/create_write.py b/pype/plugins/nuke/create/create_write_prerender.py
similarity index 54%
rename from pype/plugins/nuke/create/create_write.py
rename to pype/plugins/nuke/create/create_write_prerender.py
index 74e450f267..f8210db9db 100644
--- a/pype/plugins/nuke/create/create_write.py
+++ b/pype/plugins/nuke/create/create_write_prerender.py
@@ -1,103 +1,11 @@
 from collections import OrderedDict
-from pype.nuke import plugin
+from pype.nuke import (
+    plugin,
+    lib as pnlib
+    )
 import nuke
 
 
-class CreateWriteRender(plugin.PypeCreator):
-    # change this to template preset
-    name = "WriteRender"
-    label = "Create Write Render"
-    hosts = ["nuke"]
-    n_class = "write"
-    family = "render"
-    icon = "sign-out"
-    defaults = ["Main", "Mask"]
-
-    def __init__(self, *args, **kwargs):
-        super(CreateWriteRender, self).__init__(*args, **kwargs)
-
-        data = OrderedDict()
-
-        data["family"] = self.family
-        data["families"] = self.n_class
-
-        for k, v in self.data.items():
-            if k not in data.keys():
-                data.update({k: v})
-
-        self.data = data
-        self.nodes = nuke.selectedNodes()
-        self.log.debug("_ self.data: '{}'".format(self.data))
-
-    def process(self):
-        from pype.nuke import lib as pnlib
-
-        inputs = []
-        outputs = []
-        instance = nuke.toNode(self.data["subset"])
-        selected_node = None
-
-        # use selection
-        if (self.options or {}).get("useSelection"):
-            nodes = self.nodes
-
-            if not (len(nodes) < 2):
-                msg = ("Select only one node. The node you want to connect to, "
-                      "or tick off `Use selection`")
-                log.error(msg)
-                nuke.message(msg)
-
-            selected_node = nodes[0]
-            inputs = [selected_node]
-            outputs = selected_node.dependent()
-
-            if instance:
-                if (instance.name() in selected_node.name()):
-                    selected_node = instance.dependencies()[0]
-
-        # if node already exist
-        if instance:
-            # collect input / outputs
-            inputs = instance.dependencies()
-            outputs = instance.dependent()
-            selected_node = inputs[0]
-            # remove old one
-            nuke.delete(instance)
-
-        # recreate new
-        write_data = {
-            "class": self.n_class,
-            "families": [self.family],
-            "avalon": self.data
-        }
-
-        if self.presets.get('fpath_template'):
-            self.log.info("Adding template path from preset")
-            write_data.update(
-                {"fpath_template": self.presets["fpath_template"]}
-            )
-        else:
-            self.log.info("Adding template path from plugin")
-            write_data.update({
-                "fpath_template": "{work}/renders/nuke/{subset}/{subset}.{frame}.{ext}"})
-
-        write_node = pnlib.create_write_node(
-            self.data["subset"],
-            write_data,
-            input=selected_node)
-
-        # relinking to collected connections
-        for i, input in enumerate(inputs):
-            write_node.setInput(i, input)
-
-        write_node.autoplace()
-
-        for output in outputs:
-            output.setInput(0, write_node)
-
-        return write_node
-
-
 class CreateWritePrerender(plugin.PypeCreator):
     # change this to template preset
     name = "WritePrerender"
@@ -125,8 +33,6 @@ class CreateWritePrerender(plugin.PypeCreator):
         self.log.debug("_ self.data: '{}'".format(self.data))
 
     def process(self):
-        from pype.nuke import lib as pnlib
-
         inputs = []
         outputs = []
         instance = nuke.toNode(self.data["subset"])
@@ -137,8 +43,9 @@ class CreateWritePrerender(plugin.PypeCreator):
             nodes = self.nodes
 
             if not (len(nodes) < 2):
-                msg = ("Select only one node. The node you want to connect to, "
-                      "or tick off `Use selection`")
+                msg = ("Select only one node. The node "
+                       "you want to connect to, "
+                       "or tick off `Use selection`")
                 self.log.error(msg)
                 nuke.message(msg)
 
diff --git a/pype/plugins/nuke/create/create_write_render.py b/pype/plugins/nuke/create/create_write_render.py
new file mode 100644
index 0000000000..c3b60ba2b0
--- /dev/null
+++ b/pype/plugins/nuke/create/create_write_render.py
@@ -0,0 +1,101 @@
+from collections import OrderedDict
+from pype.nuke import (
+    plugin,
+    lib as pnlib
+    )
+import nuke
+
+
+class CreateWriteRender(plugin.PypeCreator):
+    # change this to template preset
+    name = "WriteRender"
+    label = "Create Write Render"
+    hosts = ["nuke"]
+    n_class = "write"
+    family = "render"
+    icon = "sign-out"
+    defaults = ["Main", "Mask"]
+
+    def __init__(self, *args, **kwargs):
+        super(CreateWriteRender, self).__init__(*args, **kwargs)
+
+        data = OrderedDict()
+
+        data["family"] = self.family
+        data["families"] = self.n_class
+
+        for k, v in self.data.items():
+            if k not in data.keys():
+                data.update({k: v})
+
+        self.data = data
+        self.nodes = nuke.selectedNodes()
+        self.log.debug("_ self.data: '{}'".format(self.data))
+
+    def process(self):
+
+        inputs = []
+        outputs = []
+        instance = nuke.toNode(self.data["subset"])
+        selected_node = None
+
+        # use selection
+        if (self.options or {}).get("useSelection"):
+            nodes = self.nodes
+
+            if not (len(nodes) < 2):
+                msg = ("Select only one node. "
+                       "The node you want to connect to, "
+                       "or tick off `Use selection`")
+                self.log.error(msg)
+                nuke.message(msg)
+
+            selected_node = nodes[0]
+            inputs = [selected_node]
+            outputs = selected_node.dependent()
+
+            if instance:
+                if (instance.name() in selected_node.name()):
+                    selected_node = instance.dependencies()[0]
+
+        # if node already exist
+        if instance:
+            # collect input / outputs
+            inputs = instance.dependencies()
+            outputs = instance.dependent()
+            selected_node = inputs[0]
+            # remove old one
+            nuke.delete(instance)
+
+        # recreate new
+        write_data = {
+            "class": self.n_class,
+            "families": [self.family],
+            "avalon": self.data
+        }
+
+        if self.presets.get('fpath_template'):
+            self.log.info("Adding template path from preset")
+            write_data.update(
+                {"fpath_template": self.presets["fpath_template"]}
+            )
+        else:
+            self.log.info("Adding template path from plugin")
+            write_data.update({
+                "fpath_template": "{work}/renders/nuke/{subset}/{subset}.{frame}.{ext}"})
+
+        write_node = pnlib.create_write_node(
+            self.data["subset"],
+            write_data,
+            input=selected_node)
+
+        # relinking to collected connections
+        for i, input in enumerate(inputs):
+            write_node.setInput(i, input)
+
+        write_node.autoplace()
+
+        for output in outputs:
+            output.setInput(0, write_node)
+
+        return write_node

From 5dfbe54df1b33b6391dc1b7bf718495982da234d Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:01:09 +0100
Subject: [PATCH 02/29] clean(nuke): old code

---
 pype/plugins/nuke/publish/collect_instances.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_instances.py b/pype/plugins/nuke/publish/collect_instances.py
index cbbef70e4a..57b4208ce4 100644
--- a/pype/plugins/nuke/publish/collect_instances.py
+++ b/pype/plugins/nuke/publish/collect_instances.py
@@ -89,8 +89,6 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
                     instance.append(i)
                 node.end()
 
-            family = avalon_knob_data["family"]
-            families = list()
             families_ak = avalon_knob_data.get("families")
 
             if families_ak:

From a4938110c2c7025f74fe4686a1c4de51706359e1 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:01:47 +0100
Subject: [PATCH 03/29] feat(nuke): accepting `prerender` family on instance

---
 pype/plugins/nuke/publish/collect_instances.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/pype/plugins/nuke/publish/collect_instances.py b/pype/plugins/nuke/publish/collect_instances.py
index 57b4208ce4..893c6db7e5 100644
--- a/pype/plugins/nuke/publish/collect_instances.py
+++ b/pype/plugins/nuke/publish/collect_instances.py
@@ -80,6 +80,8 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
                             add_family = "render.farm"
                             instance.data["transfer"] = False
                         families.append(add_family)
+                        if "prerender" in family:
+                            families.append("prerender")
                     else:
                         # add family into families
                         families.insert(0, family)

From 7bbc0e5677169091c34611cf9b657ce871de402e Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:02:24 +0100
Subject: [PATCH 04/29] clean(nuke): write family is more direct and lest
 anatomical

---
 pype/plugins/nuke/publish/collect_review.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pype/plugins/nuke/publish/collect_review.py b/pype/plugins/nuke/publish/collect_review.py
index c95c94541d..f02f22e053 100644
--- a/pype/plugins/nuke/publish/collect_review.py
+++ b/pype/plugins/nuke/publish/collect_review.py
@@ -9,7 +9,7 @@ class CollectReview(pyblish.api.InstancePlugin):
     order = pyblish.api.CollectorOrder + 0.3
     label = "Collect Review"
     hosts = ["nuke"]
-    families = ["render", "render.local", "render.farm"]
+    families = ["write", "prerender"]
 
     def process(self, instance):
 

From d9a2c7dad5dac122fe1194dc3bf0e02cd9ebac0c Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:02:46 +0100
Subject: [PATCH 05/29] clean(nuke): not used import

---
 pype/plugins/nuke/publish/collect_writes.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/pype/plugins/nuke/publish/collect_writes.py b/pype/plugins/nuke/publish/collect_writes.py
index 0dc7c81fae..a6fbdbab8b 100644
--- a/pype/plugins/nuke/publish/collect_writes.py
+++ b/pype/plugins/nuke/publish/collect_writes.py
@@ -1,7 +1,6 @@
 import os
 import nuke
 import pyblish.api
-import pype.api as pype
 
 
 @pyblish.api.log

From 3d32068c10b8e51f1e05d90fdb75485cde312ed2 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:06:13 +0100
Subject: [PATCH 06/29] feat(nuke): accepting prerender in img collection

---
 pype/plugins/nuke/publish/collect_writes.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/pype/plugins/nuke/publish/collect_writes.py b/pype/plugins/nuke/publish/collect_writes.py
index a6fbdbab8b..b1213199f5 100644
--- a/pype/plugins/nuke/publish/collect_writes.py
+++ b/pype/plugins/nuke/publish/collect_writes.py
@@ -67,6 +67,8 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
         )
 
         if 'render' in instance.data['families']:
+        if [fm for fm in instance.data['families']
+                if fm in ["render", "prerender"]]:
             if "representations" not in instance.data:
                 instance.data["representations"] = list()
 

From e2f51960452f29bb8e717324415c45aa638ba5bf Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:06:41 +0100
Subject: [PATCH 07/29] clean(nuke): removing wrong family definition

---
 pype/plugins/nuke/publish/validate_write_bounding_box.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pype/plugins/nuke/publish/validate_write_bounding_box.py b/pype/plugins/nuke/publish/validate_write_bounding_box.py
index e4b7c77a25..cedeea6d9f 100644
--- a/pype/plugins/nuke/publish/validate_write_bounding_box.py
+++ b/pype/plugins/nuke/publish/validate_write_bounding_box.py
@@ -57,7 +57,7 @@ class ValidateNukeWriteBoundingBox(pyblish.api.InstancePlugin):
 
     order = pyblish.api.ValidatorOrder
     optional = True
-    families = ["render", "render.local", "render.farm"]
+    families = ["write"]
     label = "Write Bounding Box"
     hosts = ["nuke"]
     actions = [RepairNukeBoundingBoxAction]

From 8c8a4e11ab9a17c2319569d64673f20cc15c9773 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:09:05 +0100
Subject: [PATCH 08/29] feat(nuke): prerender family clarifying

---
 pype/plugins/nuke/publish/collect_writes.py | 22 ++++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_writes.py b/pype/plugins/nuke/publish/collect_writes.py
index b1213199f5..aa5f825a98 100644
--- a/pype/plugins/nuke/publish/collect_writes.py
+++ b/pype/plugins/nuke/publish/collect_writes.py
@@ -66,7 +66,6 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             int(last_frame)
         )
 
-        if 'render' in instance.data['families']:
         if [fm for fm in instance.data['families']
                 if fm in ["render", "prerender"]]:
             if "representations" not in instance.data:
@@ -96,7 +95,8 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
                     # this will only run if slate frame is not already
                     # rendered from previews publishes
                     if "slate" in instance.data["families"] \
-                            and (frame_length == collected_frames_len):
+                            and (frame_length == collected_frames_len) \
+                            and ("prerender" not in instance.data["families"]):
                         frame_slate_str = "%0{}d".format(
                             len(str(last_frame))) % (first_frame - 1)
                         slate_frame = collected_frames[0].replace(
@@ -105,6 +105,8 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
 
                 representation['files'] = collected_frames
                 instance.data["representations"].append(representation)
+                if "render" not in instance.data["families"]:
+                    instance.data["families"].append("render")
             except Exception:
                 instance.data["representations"].append(representation)
                 self.log.debug("couldn't collect frames: {}".format(label))
@@ -144,5 +146,19 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             "deadlineChunkSize": deadlineChunkSize,
             "deadlinePriority": deadlinePriority
         })
-
+        self.log.debug("families: {}".format(families))
+        if "prerender" in families:
+            _families = list()
+            for fm in families:
+                if fm in _families:
+                    continue
+                if "render" in fm:
+                    if "prerender" in fm:
+                        continue
+                    _families.append(fm)
+            instance.data.update({
+                "family": "prerender",
+                "families": _families
+                })
+            self.log.debug("_families: {}".format(_families))
         self.log.debug("instance.data: {}".format(instance.data))

From 9248e4b8cf95c467560259f02126ef0d531ab3ac Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 18 Mar 2020 19:15:04 +0100
Subject: [PATCH 09/29] clear(nuke): cleaning wrong families

---
 pype/plugins/nuke/publish/extract_review_data_mov.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pype/plugins/nuke/publish/extract_review_data_mov.py b/pype/plugins/nuke/publish/extract_review_data_mov.py
index 8b204680a7..683da24fc8 100644
--- a/pype/plugins/nuke/publish/extract_review_data_mov.py
+++ b/pype/plugins/nuke/publish/extract_review_data_mov.py
@@ -15,7 +15,7 @@ class ExtractReviewDataMov(pype.api.Extractor):
     order = pyblish.api.ExtractorOrder + 0.01
     label = "Extract Review Data Mov"
 
-    families = ["review", "render", "render.local"]
+    families = ["review"]
     hosts = ["nuke"]
 
     def process(self, instance):

From 31f0ab63788037a11c1fecb7817bb621651159ad Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:35:23 +0100
Subject: [PATCH 10/29] feat(nuke): prerender node dont need `review` knob

---
 pype/nuke/lib.py                              | 38 +++++++++++++------
 .../nuke/create/create_write_prerender.py     |  4 +-
 2 files changed, 30 insertions(+), 12 deletions(-)

diff --git a/pype/nuke/lib.py b/pype/nuke/lib.py
index 8e241dad16..989cbf569f 100644
--- a/pype/nuke/lib.py
+++ b/pype/nuke/lib.py
@@ -215,14 +215,14 @@ def script_name():
 
 def add_button_write_to_read(node):
     name = "createReadNode"
-    label = "Create Read"
+    label = "[ Create Read ]"
     value = "import write_to_read;write_to_read.write_to_read(nuke.thisNode())"
     k = nuke.PyScript_Knob(name, label, value)
     k.setFlag(0x1000)
     node.addKnob(k)
 
 
-def create_write_node(name, data, input=None, prenodes=None):
+def create_write_node(name, data, input=None, prenodes=None, review=True):
     ''' Creating write node which is group node
 
     Arguments:
@@ -231,6 +231,7 @@ def create_write_node(name, data, input=None, prenodes=None):
         input (node): selected node to connect to
         prenodes (list, optional): list of lists, definitions for nodes
                                 to be created before write
+        review (bool): adding review knob
 
     Example:
         prenodes = [(
@@ -380,15 +381,8 @@ def create_write_node(name, data, input=None, prenodes=None):
 
     add_rendering_knobs(GN)
 
-    # adding write to read button
-    add_button_write_to_read(GN)
-
-    divider = nuke.Text_Knob('')
-    GN.addKnob(divider)
-
-    # set tile color
-    tile_color = _data.get("tile_color", "0xff0000ff")
-    GN["tile_color"].setValue(tile_color)
+    if review:
+        add_review_knob(GN)
 
     # add render button
     lnk = nuke.Link_Knob("Render")
@@ -396,9 +390,20 @@ def create_write_node(name, data, input=None, prenodes=None):
     lnk.setName("Render")
     GN.addKnob(lnk)
 
+    divider = nuke.Text_Knob('')
+    GN.addKnob(divider)
+
+    # adding write to read button
+    add_button_write_to_read(GN)
+
     # Deadline tab.
     add_deadline_tab(GN)
 
+
+    # set tile color
+    tile_color = _data.get("tile_color", "0xff0000ff")
+    GN["tile_color"].setValue(tile_color)
+
     return GN
 
 
@@ -420,6 +425,17 @@ def add_rendering_knobs(node):
         knob = nuke.Boolean_Knob("render_farm", "Render on Farm")
         knob.setValue(False)
         node.addKnob(knob)
+    return node
+
+def add_review_knob(node):
+    ''' Adds additional review knob to given node
+
+    Arguments:
+        node (obj): nuke node object to be fixed
+
+    Return:
+        node (obj): with added knob
+    '''
     if "review" not in node.knobs():
         knob = nuke.Boolean_Knob("review", "Review")
         knob.setValue(True)
diff --git a/pype/plugins/nuke/create/create_write_prerender.py b/pype/plugins/nuke/create/create_write_prerender.py
index f8210db9db..6e242f886c 100644
--- a/pype/plugins/nuke/create/create_write_prerender.py
+++ b/pype/plugins/nuke/create/create_write_prerender.py
@@ -87,7 +87,9 @@ class CreateWritePrerender(plugin.PypeCreator):
             self.data["subset"],
             write_data,
             input=selected_node,
-            prenodes=[])
+            prenodes=[],
+            review=False
+            )
 
         # relinking to collected connections
         for i, input in enumerate(inputs):

From 5c7bba0c0676a95a35b1de2e03c523989d7197b2 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:36:13 +0100
Subject: [PATCH 11/29] feat(nuke): setting up properly families

---
 pype/plugins/nuke/publish/collect_review.py              | 2 +-
 pype/plugins/nuke/publish/collect_slate_node.py          | 2 +-
 pype/plugins/nuke/publish/extract_render_local.py        | 2 +-
 pype/plugins/nuke/publish/submit_nuke_deadline.py        | 2 +-
 pype/plugins/nuke/publish/validate_write_bounding_box.py | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_review.py b/pype/plugins/nuke/publish/collect_review.py
index f02f22e053..c95c94541d 100644
--- a/pype/plugins/nuke/publish/collect_review.py
+++ b/pype/plugins/nuke/publish/collect_review.py
@@ -9,7 +9,7 @@ class CollectReview(pyblish.api.InstancePlugin):
     order = pyblish.api.CollectorOrder + 0.3
     label = "Collect Review"
     hosts = ["nuke"]
-    families = ["write", "prerender"]
+    families = ["render", "render.local", "render.farm"]
 
     def process(self, instance):
 
diff --git a/pype/plugins/nuke/publish/collect_slate_node.py b/pype/plugins/nuke/publish/collect_slate_node.py
index d8d6b50f05..9c7f1b5e95 100644
--- a/pype/plugins/nuke/publish/collect_slate_node.py
+++ b/pype/plugins/nuke/publish/collect_slate_node.py
@@ -8,7 +8,7 @@ class CollectSlate(pyblish.api.InstancePlugin):
     order = pyblish.api.CollectorOrder + 0.09
     label = "Collect Slate Node"
     hosts = ["nuke"]
-    families = ["write"]
+    families = ["render", "render.local", "render.farm"]
 
     def process(self, instance):
         node = instance[0]
diff --git a/pype/plugins/nuke/publish/extract_render_local.py b/pype/plugins/nuke/publish/extract_render_local.py
index 5467d239c2..1dad413ee5 100644
--- a/pype/plugins/nuke/publish/extract_render_local.py
+++ b/pype/plugins/nuke/publish/extract_render_local.py
@@ -17,7 +17,7 @@ class NukeRenderLocal(pype.api.Extractor):
     order = pyblish.api.ExtractorOrder
     label = "Render Local"
     hosts = ["nuke"]
-    families = ["render.local"]
+    families = ["render.local", "prerender.local"]
 
     def process(self, instance):
         node = None
diff --git a/pype/plugins/nuke/publish/submit_nuke_deadline.py b/pype/plugins/nuke/publish/submit_nuke_deadline.py
index 0a9ef33398..3da2e58e4d 100644
--- a/pype/plugins/nuke/publish/submit_nuke_deadline.py
+++ b/pype/plugins/nuke/publish/submit_nuke_deadline.py
@@ -19,7 +19,7 @@ class NukeSubmitDeadline(pyblish.api.InstancePlugin):
     label = "Submit to Deadline"
     order = pyblish.api.IntegratorOrder + 0.1
     hosts = ["nuke", "nukestudio"]
-    families = ["render.farm"]
+    families = ["render.farm", "prerender.farm"]
     optional = True
 
     deadline_priority = 50
diff --git a/pype/plugins/nuke/publish/validate_write_bounding_box.py b/pype/plugins/nuke/publish/validate_write_bounding_box.py
index cedeea6d9f..e4b7c77a25 100644
--- a/pype/plugins/nuke/publish/validate_write_bounding_box.py
+++ b/pype/plugins/nuke/publish/validate_write_bounding_box.py
@@ -57,7 +57,7 @@ class ValidateNukeWriteBoundingBox(pyblish.api.InstancePlugin):
 
     order = pyblish.api.ValidatorOrder
     optional = True
-    families = ["write"]
+    families = ["render", "render.local", "render.farm"]
     label = "Write Bounding Box"
     hosts = ["nuke"]
     actions = [RepairNukeBoundingBoxAction]

From 63eac17649a7c42b4ba4f8d93c06f0dfad1af346 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:36:46 +0100
Subject: [PATCH 12/29] clean(nuke): old code and better definition of families

---
 pype/plugins/nuke/publish/collect_writes.py | 18 ++----------------
 1 file changed, 2 insertions(+), 16 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_writes.py b/pype/plugins/nuke/publish/collect_writes.py
index aa5f825a98..f3f33b7a6d 100644
--- a/pype/plugins/nuke/publish/collect_writes.py
+++ b/pype/plugins/nuke/publish/collect_writes.py
@@ -105,8 +105,6 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
 
                 representation['files'] = collected_frames
                 instance.data["representations"].append(representation)
-                if "render" not in instance.data["families"]:
-                    instance.data["families"].append("render")
             except Exception:
                 instance.data["representations"].append(representation)
                 self.log.debug("couldn't collect frames: {}".format(label))
@@ -127,6 +125,7 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             deadlinePriority = group_node["deadlinePriority"].value()
 
         families = [f for f in instance.data["families"] if "write" not in f]
+
         instance.data.update({
             "versionData": version_data,
             "path": path,
@@ -147,18 +146,5 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             "deadlinePriority": deadlinePriority
         })
         self.log.debug("families: {}".format(families))
-        if "prerender" in families:
-            _families = list()
-            for fm in families:
-                if fm in _families:
-                    continue
-                if "render" in fm:
-                    if "prerender" in fm:
-                        continue
-                    _families.append(fm)
-            instance.data.update({
-                "family": "prerender",
-                "families": _families
-                })
-            self.log.debug("_families: {}".format(_families))
+
         self.log.debug("instance.data: {}".format(instance.data))

From 7fbb72b8a6eecdaf0d3799b59a248aa6cded0aa3 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:37:10 +0100
Subject: [PATCH 13/29] clean(nuke): improving code

---
 .../plugins/nuke/publish/collect_instances.py | 29 ++++---------------
 1 file changed, 5 insertions(+), 24 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_instances.py b/pype/plugins/nuke/publish/collect_instances.py
index 893c6db7e5..54891d189c 100644
--- a/pype/plugins/nuke/publish/collect_instances.py
+++ b/pype/plugins/nuke/publish/collect_instances.py
@@ -52,6 +52,7 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
 
             # establish families
             family = avalon_knob_data["family"]
+            families_ak = avalon_knob_data.get("families")
             families = list()
 
             # except disabled nodes but exclude backdrops in test
@@ -68,20 +69,16 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
             # Add all nodes in group instances.
             if node.Class() == "Group":
                 # only alter families for render family
-                if ("render" in family):
-                    # check if node is not disabled
-                    families.append(avalon_knob_data["families"])
+                if "write" in families_ak:
                     if node["render"].value():
                         self.log.info("flagged for render")
-                        add_family = "render.local"
+                        add_family = "{}.local".format(family)
                         # dealing with local/farm rendering
                         if node["render_farm"].value():
                             self.log.info("adding render farm family")
-                            add_family = "render.farm"
+                            add_family = "{}.farm".format(family)
                             instance.data["transfer"] = False
                         families.append(add_family)
-                        if "prerender" in family:
-                            families.append("prerender")
                     else:
                         # add family into families
                         families.insert(0, family)
@@ -91,7 +88,7 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
                     instance.append(i)
                 node.end()
 
-            families_ak = avalon_knob_data.get("families")
+            self.log.debug("__ families: `{}`".format(families))
 
             if families_ak:
                 families.append(families_ak)
@@ -104,22 +101,6 @@ class CollectNukeInstances(pyblish.api.ContextPlugin):
             resolution_height = format.height()
             pixel_aspect = format.pixelAspect()
 
-            if node.Class() not in "Read":
-                if "render" not in node.knobs().keys():
-                    pass
-                elif node["render"].value():
-                    self.log.info("flagged for render")
-                    add_family = "render.local"
-                    # dealing with local/farm rendering
-                    if node["render_farm"].value():
-                        self.log.info("adding render farm family")
-                        add_family = "render.farm"
-                        instance.data["transfer"] = False
-                    families.append(add_family)
-                else:
-                    # add family into families
-                    families.insert(0, family)
-
             instance.data.update({
                 "subset": subset,
                 "asset": os.environ["AVALON_ASSET"],

From b3702c49df14b2351cfc9c99e8987fb4e5685896 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:37:37 +0100
Subject: [PATCH 14/29] feat(global): explicit families for integrate new

---
 pype/plugins/global/publish/integrate_new.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/pype/plugins/global/publish/integrate_new.py b/pype/plugins/global/publish/integrate_new.py
index aa214f36cb..ddb40e321a 100644
--- a/pype/plugins/global/publish/integrate_new.py
+++ b/pype/plugins/global/publish/integrate_new.py
@@ -64,6 +64,9 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin):
                 "scene",
                 "vrayproxy",
                 "render",
+                "render.local",
+                "prerender",
+                "prerender.local",
                 "imagesequence",
                 "review",
                 "rendersetup",

From 9a1167c26d784a43bb6643b4fcd6514d2aebe7b0 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 20 Mar 2020 21:38:11 +0100
Subject: [PATCH 15/29] feat(nuke): dealing with `prerender` family when
 submitting to deadline

---
 pype/plugins/global/publish/submit_publish_job.py | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/pype/plugins/global/publish/submit_publish_job.py b/pype/plugins/global/publish/submit_publish_job.py
index dcf19ae32c..0b7a8473d4 100644
--- a/pype/plugins/global/publish/submit_publish_job.py
+++ b/pype/plugins/global/publish/submit_publish_job.py
@@ -141,7 +141,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
 
     hosts = ["fusion", "maya", "nuke"]
 
-    families = ["render.farm", "renderlayer", "imagesequence"]
+    families = ["render.farm", "prerender.farm", "renderlayer", "imagesequence"]
 
     aov_filter = {"maya": ["beauty"]}
 
@@ -583,6 +583,12 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             "resolutionHeight": data.get("resolutionHeight", 1080),
         }
 
+        if "prerender.farm" in instance.data["families"]:
+            instance_skeleton_data.update({
+                "family": "prerender",
+                "families": ["prerender"]
+            })
+
         # transfer specific families from original instance to new render
         for item in self.families_transfer:
             if item in instance.data.get("families", []):

From d1dd5023571b10934c0b81dc4d8cd31e6983998f Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Tue, 24 Mar 2020 16:35:37 +0100
Subject: [PATCH 16/29] fix(nuke): validator clam fps float number

---
 pype/plugins/nuke/publish/validate_script.py | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/pype/plugins/nuke/publish/validate_script.py b/pype/plugins/nuke/publish/validate_script.py
index f7dd84d714..36df228ed5 100644
--- a/pype/plugins/nuke/publish/validate_script.py
+++ b/pype/plugins/nuke/publish/validate_script.py
@@ -74,17 +74,14 @@ class ValidateScript(pyblish.api.InstancePlugin):
         if "handleEnd" in asset_attributes:
             handle_end = asset_attributes["handleEnd"]
 
-        # Set frame range with handles
-        # asset_attributes["frameStart"] -= handle_start
-        # asset_attributes["frameEnd"] += handle_end
-        if len(str(asset_attributes["fps"])) > 4:
-            asset_attributes["fps"] = float("{0:.8f}".format(asset_attributes["fps"]))
+        asset_attributes["fps"] = float("{0:.4f}".format(
+                asset_attributes["fps"]))
 
         # Get values from nukescript
         script_attributes = {
             "handleStart": ctx_data["handleStart"],
             "handleEnd": ctx_data["handleEnd"],
-            "fps": ctx_data["fps"],
+            "fps": float("{0:.4f}".format(ctx_data["fps"])),
             "frameStart": ctx_data["frameStart"],
             "frameEnd": ctx_data["frameEnd"],
             "resolutionWidth": ctx_data["resolutionWidth"],

From 695351cefcf25e3bc564f4f35fb87c6efeebfcb9 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Tue, 24 Mar 2020 18:27:11 +0100
Subject: [PATCH 17/29] feat(nks): improving way file's metadata are collected

---
 .../nukestudio/publish/collect_clips.py       | 23 ++++++++++++-------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/pype/plugins/nukestudio/publish/collect_clips.py b/pype/plugins/nukestudio/publish/collect_clips.py
index 6a1dad9a6d..746df67485 100644
--- a/pype/plugins/nukestudio/publish/collect_clips.py
+++ b/pype/plugins/nukestudio/publish/collect_clips.py
@@ -47,6 +47,16 @@ class CollectClips(api.ContextPlugin):
                 track = item.parent()
                 source = item.source().mediaSource()
                 source_path = source.firstpath()
+                file_head = source.filenameHead()
+                file_info = next((f for f in source.fileinfos()), None)
+                source_first_frame = file_info.startFrame()
+                is_sequence = False
+
+                if not source.singleFile():
+                    self.log.info("Single file")
+                    is_sequence = True
+                    source_path = file_info.filename()
+
                 effects = [f for f in item.linkedItems()
                            if f.isEnabled()
                            if isinstance(f, hiero.core.EffectTrackItem)]
@@ -78,12 +88,6 @@ class CollectClips(api.ContextPlugin):
                         )
                     )
 
-                try:
-                    head, padding, ext = os.path.basename(source_path).split(".")
-                    source_first_frame = int(padding)
-                except Exception:
-                    source_first_frame = 0
-
                 data.update({
                         "name": "{0}_{1}".format(track.name(), item.name()),
                         "item": item,
@@ -91,6 +95,8 @@ class CollectClips(api.ContextPlugin):
                         "timecodeStart": str(source.timecodeStart()),
                         "timelineTimecodeStart": str(sequence.timecodeStart()),
                         "sourcePath": source_path,
+                        "sourceFileHead": file_head,
+                        "isSequence": is_sequence,
                         "track": track.name(),
                         "trackIndex": track_index,
                         "sourceFirst": source_first_frame,
@@ -101,8 +107,9 @@ class CollectClips(api.ContextPlugin):
                             int(item.sourceIn())) + 1,
                         "clipIn": int(item.timelineIn()),
                         "clipOut": int(item.timelineOut()),
-                        "clipDuration": (int(item.timelineOut()) -
-                            int(item.timelineIn())) + 1,
+                        "clipDuration": (
+                            int(item.timelineOut()) - int(
+                                item.timelineIn())) + 1,
                         "asset": asset,
                         "family": "clip",
                         "families": [],

From c6c4c26a54386b0b05ec147942ecd5ced97fc4f6 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Tue, 24 Mar 2020 18:27:45 +0100
Subject: [PATCH 18/29] fix(nks): plate single file vs sequence treatment

---
 .../nukestudio/publish/collect_plates.py      | 31 +++++++------------
 1 file changed, 12 insertions(+), 19 deletions(-)

diff --git a/pype/plugins/nukestudio/publish/collect_plates.py b/pype/plugins/nukestudio/publish/collect_plates.py
index 4ed281f0ee..8a79354bbf 100644
--- a/pype/plugins/nukestudio/publish/collect_plates.py
+++ b/pype/plugins/nukestudio/publish/collect_plates.py
@@ -147,22 +147,15 @@ class CollectPlatesData(api.InstancePlugin):
                 "version": version
             })
 
+        source_first_frame = instance.data.get("sourceFirst")
+        source_file_head = instance.data.get("sourceFileHead")
 
-        try:
-            basename, ext = os.path.splitext(source_file)
-            head, padding = os.path.splitext(basename)
-            ext = ext[1:]
-            padding = padding[1:]
-            self.log.debug("_ padding: `{}`".format(padding))
-            # head, padding, ext = source_file.split('.')
-            source_first_frame = int(padding)
-            padding = len(padding)
-            file = "{head}.%0{padding}d.{ext}".format(
-                head=head,
-                padding=padding,
-                ext=ext
-            )
-
+        if instance.data.get("isSequence", False):
+            self.log.info("Is sequence of files")
+            file = os.path.basename(source_file)
+            ext = os.path.splitext(file)[-1][1:]
+            self.log.debug("source_file_head: `{}`".format(source_file_head))
+            head = source_file_head[:-1]
             start_frame = int(source_first_frame + instance.data["sourceInH"])
             duration = int(
                 instance.data["sourceOutH"] - instance.data["sourceInH"])
@@ -170,10 +163,10 @@ class CollectPlatesData(api.InstancePlugin):
             self.log.debug("start_frame: `{}`".format(start_frame))
             self.log.debug("end_frame: `{}`".format(end_frame))
             files = [file % i for i in range(start_frame, (end_frame + 1), 1)]
-        except Exception as e:
-            self.log.warning("Exception in file: {}".format(e))
-            head, ext = os.path.splitext(source_file)
-            ext = ext[1:]
+        else:
+            self.log.info("Is single file")
+            ext = os.path.splitext(source_file)[-1][1:]
+            head = source_file_head
             files = source_file
             start_frame = instance.data["sourceInH"]
             end_frame = instance.data["sourceOutH"]

From 461441ebf42ab734f741acd764739e51f2d805c8 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Tue, 24 Mar 2020 19:47:36 +0100
Subject: [PATCH 19/29] fix(nk): build first workfile get jpeg sequences

---
 pype/nuke/lib.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pype/nuke/lib.py b/pype/nuke/lib.py
index 6cd66407d6..cbec8b4300 100644
--- a/pype/nuke/lib.py
+++ b/pype/nuke/lib.py
@@ -1135,7 +1135,7 @@ class BuildWorkfile(WorkfileSettings):
                 regex_filter=None,
                 version=None,
                 representations=["exr", "dpx", "lutJson", "mov",
-                                 "preview", "png"]):
+                                 "preview", "png", "jpeg", "jpg"]):
         """
         A short description.
 

From 9577c236243d7caa42a4bdd1409e469d2f1cd0d2 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 18:07:12 +0100
Subject: [PATCH 20/29] fix(global): slate was on even if render on farm

---
 pype/plugins/global/publish/extract_review_slate.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/pype/plugins/global/publish/extract_review_slate.py b/pype/plugins/global/publish/extract_review_slate.py
index 8c33a0d853..da94c7714a 100644
--- a/pype/plugins/global/publish/extract_review_slate.py
+++ b/pype/plugins/global/publish/extract_review_slate.py
@@ -11,7 +11,9 @@ class ExtractReviewSlate(pype.api.Extractor):
 
     label = "Review with Slate frame"
     order = pyblish.api.ExtractorOrder + 0.031
-    families = ["slate"]
+    families = ["slate", "review"]
+    match = pyblish.api.Subset
+
     hosts = ["nuke", "maya", "shell"]
     optional = True
 

From 570269f0247331460ad66baee0a76728bba0242b Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 18:56:17 +0100
Subject: [PATCH 21/29] fix(global, nuke): setting families filter

---
 .../publish/integrate_ftrack_instances.py     |  1 +
 pype/plugins/global/publish/integrate_new.py  |  2 --
 pype/plugins/nuke/load/load_mov.py            |  1 +
 pype/plugins/nuke/load/load_sequence.py       |  4 ++--
 .../nuke/publish/increment_script_version.py  | 21 +++++--------------
 .../nuke/publish/validate_rendered_frames.py  |  2 +-
 6 files changed, 10 insertions(+), 21 deletions(-)

diff --git a/pype/plugins/ftrack/publish/integrate_ftrack_instances.py b/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
index db257e901a..59fb507788 100644
--- a/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
+++ b/pype/plugins/ftrack/publish/integrate_ftrack_instances.py
@@ -22,6 +22,7 @@ class IntegrateFtrackInstance(pyblish.api.InstancePlugin):
                       'setdress': 'setdress',
                       'pointcache': 'cache',
                       'render': 'render',
+                      'render2d': 'render',
                       'nukescript': 'comp',
                       'write': 'render',
                       'review': 'mov',
diff --git a/pype/plugins/global/publish/integrate_new.py b/pype/plugins/global/publish/integrate_new.py
index 127bff90f0..0ceac1f4a7 100644
--- a/pype/plugins/global/publish/integrate_new.py
+++ b/pype/plugins/global/publish/integrate_new.py
@@ -64,9 +64,7 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin):
                 "scene",
                 "vrayproxy",
                 "render",
-                "render.local",
                 "prerender",
-                "prerender.local",
                 "imagesequence",
                 "review",
                 "rendersetup",
diff --git a/pype/plugins/nuke/load/load_mov.py b/pype/plugins/nuke/load/load_mov.py
index 88e65156cb..5d15efcd3a 100644
--- a/pype/plugins/nuke/load/load_mov.py
+++ b/pype/plugins/nuke/load/load_mov.py
@@ -92,6 +92,7 @@ class LoadMov(api.Loader):
         "source",
         "plate",
         "render",
+        "prerender",
         "review"] + presets["families"]
 
     representations = [
diff --git a/pype/plugins/nuke/load/load_sequence.py b/pype/plugins/nuke/load/load_sequence.py
index 690f074c3f..083cc86474 100644
--- a/pype/plugins/nuke/load/load_sequence.py
+++ b/pype/plugins/nuke/load/load_sequence.py
@@ -70,7 +70,7 @@ def loader_shift(node, frame, relative=True):
 class LoadSequence(api.Loader):
     """Load image sequence into Nuke"""
 
-    families = ["render2d", "source", "plate", "render"]
+    families = ["render2d", "source", "plate", "render", "prerender"]
     representations = ["exr", "dpx", "jpg", "jpeg", "png"]
 
     label = "Load sequence"
@@ -87,7 +87,7 @@ class LoadSequence(api.Loader):
         version = context['version']
         version_data = version.get("data", {})
         repr_id = context["representation"]["_id"]
-        
+
         self.log.info("version_data: {}\n".format(version_data))
         self.log.debug(
             "Representation id `{}` ".format(repr_id))
diff --git a/pype/plugins/nuke/publish/increment_script_version.py b/pype/plugins/nuke/publish/increment_script_version.py
index 6e3ce08276..c76083eb1e 100644
--- a/pype/plugins/nuke/publish/increment_script_version.py
+++ b/pype/plugins/nuke/publish/increment_script_version.py
@@ -9,6 +9,7 @@ class IncrementScriptVersion(pyblish.api.ContextPlugin):
     order = pyblish.api.IntegratorOrder + 0.9
     label = "Increment Script Version"
     optional = True
+    families = ["workfile", "render", "render.local", "render.farm"]
     hosts = ['nuke']
 
     def process(self, context):
@@ -16,19 +17,7 @@ class IncrementScriptVersion(pyblish.api.ContextPlugin):
         assert all(result["success"] for result in context.data["results"]), (
             "Publishing not succesfull so version is not increased.")
 
-        instances = context[:]
-
-        prerender_check = list()
-        families_check = list()
-        for instance in instances:
-            if ("prerender" in str(instance)) and instance.data.get("families", None):
-                prerender_check.append(instance)
-            if instance.data.get("families", None):
-                families_check.append(True)
-
-
-        if len(prerender_check) != len(families_check):
-            from pype.lib import version_up
-            path = context.data["currentFile"]
-            nuke.scriptSaveAs(version_up(path))
-            self.log.info('Incrementing script version')
+        from pype.lib import version_up
+        path = context.data["currentFile"]
+        nuke.scriptSaveAs(version_up(path))
+        self.log.info('Incrementing script version')
diff --git a/pype/plugins/nuke/publish/validate_rendered_frames.py b/pype/plugins/nuke/publish/validate_rendered_frames.py
index 6e9b91dd72..425789f18a 100644
--- a/pype/plugins/nuke/publish/validate_rendered_frames.py
+++ b/pype/plugins/nuke/publish/validate_rendered_frames.py
@@ -28,7 +28,7 @@ class ValidateRenderedFrames(pyblish.api.InstancePlugin):
     """ Validates file output. """
 
     order = pyblish.api.ValidatorOrder + 0.1
-    families = ["render"]
+    families = ["render", "prerender"]
 
     label = "Validate rendered frame"
     hosts = ["nuke", "nukestudio"]

From f25dea58a808e80876acf5115c41a7fd2c2b3272 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 18:57:20 +0100
Subject: [PATCH 22/29] fix(nuke, global): shuffling families after plugins
 finish processing

---
 pype/plugins/nuke/publish/collect_writes.py   | 24 ++++++++++++++-----
 .../nuke/publish/extract_render_local.py      | 18 ++++++++++----
 .../nuke/publish/submit_nuke_deadline.py      | 10 ++++++++
 3 files changed, 42 insertions(+), 10 deletions(-)

diff --git a/pype/plugins/nuke/publish/collect_writes.py b/pype/plugins/nuke/publish/collect_writes.py
index f3f33b7a6d..6379a1db87 100644
--- a/pype/plugins/nuke/publish/collect_writes.py
+++ b/pype/plugins/nuke/publish/collect_writes.py
@@ -12,9 +12,11 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
     hosts = ["nuke", "nukeassist"]
     families = ["write"]
 
+    # preset attributes
+    sync_workfile_version = True
+
     def process(self, instance):
-        # adding 2d focused rendering
-        instance.data["families"].append("render2d")
+        families = instance.data["families"]
 
         node = None
         for x in instance:
@@ -52,10 +54,13 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
         output_dir = os.path.dirname(path)
         self.log.debug('output dir: {}'.format(output_dir))
 
-        # get version to instance for integration
-        instance.data['version'] = instance.context.data["version"]
+        if not next((f for f in families
+                     if "prerender" in f),
+                    None) and self.sync_workfile_version:
+            # get version to instance for integration
+            instance.data['version'] = instance.context.data["version"]
 
-        self.log.debug('Write Version: %s' % instance.data('version'))
+            self.log.debug('Write Version: %s' % instance.data('version'))
 
         # create label
         name = node.name()
@@ -66,7 +71,7 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             int(last_frame)
         )
 
-        if [fm for fm in instance.data['families']
+        if [fm for fm in families
                 if fm in ["render", "prerender"]]:
             if "representations" not in instance.data:
                 instance.data["representations"] = list()
@@ -145,6 +150,13 @@ class CollectNukeWrites(pyblish.api.InstancePlugin):
             "deadlineChunkSize": deadlineChunkSize,
             "deadlinePriority": deadlinePriority
         })
+
+        if "prerender" in families:
+            instance.data.update({
+                "family": "prerender",
+                "families": []
+            })
+
         self.log.debug("families: {}".format(families))
 
         self.log.debug("instance.data: {}".format(instance.data))
diff --git a/pype/plugins/nuke/publish/extract_render_local.py b/pype/plugins/nuke/publish/extract_render_local.py
index 1dad413ee5..b7aa59a457 100644
--- a/pype/plugins/nuke/publish/extract_render_local.py
+++ b/pype/plugins/nuke/publish/extract_render_local.py
@@ -20,6 +20,8 @@ class NukeRenderLocal(pype.api.Extractor):
     families = ["render.local", "prerender.local"]
 
     def process(self, instance):
+        families = instance.data["families"]
+
         node = None
         for x in instance:
             if x.Class() == "Write":
@@ -30,7 +32,7 @@ class NukeRenderLocal(pype.api.Extractor):
         first_frame = instance.data.get("frameStartHandle", None)
 
         # exception for slate workflow
-        if "slate" in instance.data["families"]:
+        if "slate" in families:
             first_frame -= 1
 
         last_frame = instance.data.get("frameEndHandle", None)
@@ -53,7 +55,7 @@ class NukeRenderLocal(pype.api.Extractor):
         )
 
         # exception for slate workflow
-        if "slate" in instance.data["families"]:
+        if "slate" in families:
             first_frame += 1
 
         path = node['file'].value()
@@ -79,8 +81,16 @@ class NukeRenderLocal(pype.api.Extractor):
             out_dir
         ))
 
-        instance.data['family'] = 'render'
-        instance.data['families'].append('render')
+        # redefinition of families
+        if "render.local" in families:
+            instance.data['family'] = 'render2d'
+            families.remove('render.local')
+            families.insert(0, "render")
+        elif "prerender.local" in families:
+            instance.data['family'] = 'prerender'
+            families.remove('prerender.local')
+            families.insert(0, "prerender")
+        instance.data["families"] = families
 
         collections, remainder = clique.assemble(collected_frames)
         self.log.info('collections: {}'.format(str(collections)))
diff --git a/pype/plugins/nuke/publish/submit_nuke_deadline.py b/pype/plugins/nuke/publish/submit_nuke_deadline.py
index 3da2e58e4d..7990c20112 100644
--- a/pype/plugins/nuke/publish/submit_nuke_deadline.py
+++ b/pype/plugins/nuke/publish/submit_nuke_deadline.py
@@ -28,6 +28,7 @@ class NukeSubmitDeadline(pyblish.api.InstancePlugin):
     deadline_chunk_size = 1
 
     def process(self, instance):
+        families = instance.data["families"]
 
         node = instance[0]
         context = instance.context
@@ -82,6 +83,15 @@ class NukeSubmitDeadline(pyblish.api.InstancePlugin):
             instance.data["deadlineSubmissionJob"] = resp.json()
             instance.data["publishJobState"] = "Suspended"
 
+        # redefinition of families
+        if "render.farm" in families:
+            instance.data['family'] = 'write'
+            families.insert(0, "render2d")
+        elif "prerender.farm" in families:
+            instance.data['family'] = 'write'
+            families.insert(0, "prerender")
+        instance.data["families"] = families
+
     def payload_submit(self,
                        instance,
                        script_path,

From 93761e3010743d139f272022a578af3d90e4acfc Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 18:57:52 +0100
Subject: [PATCH 23/29] fix(global): add correct frame number to repre.data

---
 pype/plugins/global/publish/integrate_new.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/pype/plugins/global/publish/integrate_new.py b/pype/plugins/global/publish/integrate_new.py
index 0ceac1f4a7..44f91a343c 100644
--- a/pype/plugins/global/publish/integrate_new.py
+++ b/pype/plugins/global/publish/integrate_new.py
@@ -383,6 +383,8 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin):
                     if not dst_start_frame:
                         dst_start_frame = dst_padding
 
+                template_data["frame"] = dst_start_frame
+
                 dst = "{0}{1}{2}".format(
                     dst_head,
                     dst_start_frame,

From 5f616f14f1a97db551b713a1ab73415d9ab6ecd4 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 18:58:18 +0100
Subject: [PATCH 24/29] fix(nuke): submitting with correct family

---
 .../global/publish/submit_publish_job.py      | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/pype/plugins/global/publish/submit_publish_job.py b/pype/plugins/global/publish/submit_publish_job.py
index 144a0822a2..134f8e9098 100644
--- a/pype/plugins/global/publish/submit_publish_job.py
+++ b/pype/plugins/global/publish/submit_publish_job.py
@@ -141,7 +141,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
 
     hosts = ["fusion", "maya", "nuke"]
 
-    families = ["render.farm", "prerender.farm", "renderlayer", "imagesequence"]
+    families = ["render.farm", "prerener", "renderlayer", "imagesequence"]
 
     aov_filter = {"maya": ["beauty"]}
 
@@ -168,9 +168,9 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
     instance_transfer = {
         "slate": ["slateFrame"],
         "review": ["lutPath"],
-        "render.farm": ["bakeScriptPath", "bakeRenderPath",
-                        "bakeWriteNodeName", "version"]
-    }
+        "render2d": ["bakeScriptPath", "bakeRenderPath",
+                     "bakeWriteNodeName", "version"]
+        }
 
     # list of family names to transfer to new family if present
     families_transfer = ["render3d", "render2d", "ftrack", "slate"]
@@ -586,17 +586,24 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
             "resolutionHeight": data.get("resolutionHeight", 1080),
         }
 
-        if "prerender.farm" in instance.data["families"]:
+        if "prerender" in instance.data["families"]:
             instance_skeleton_data.update({
                 "family": "prerender",
-                "families": ["prerender"]
-            })
+                "families": []
+                })
 
         # transfer specific families from original instance to new render
         for item in self.families_transfer:
             if item in instance.data.get("families", []):
                 instance_skeleton_data["families"] += [item]
 
+        if "render.farm" in instance.data["families"]:
+            instance_skeleton_data.update({
+                "family": "render2d",
+                "families": ["render"] + [f for f in instance.data["families"]
+                                          if "render.farm" not in f]
+            })
+
         # transfer specific properties from original instance based on
         # mapping dictionary `instance_transfer`
         for key, values in self.instance_transfer.items():

From 129eb5db6931832c8a3592295e352dbdd4700ea9 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 19:04:03 +0100
Subject: [PATCH 25/29] fix(nuke):  hound pep8

---
 pype/plugins/nuke/create/create_write_prerender.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/pype/plugins/nuke/create/create_write_prerender.py b/pype/plugins/nuke/create/create_write_prerender.py
index 6e242f886c..210c84e0cd 100644
--- a/pype/plugins/nuke/create/create_write_prerender.py
+++ b/pype/plugins/nuke/create/create_write_prerender.py
@@ -1,8 +1,7 @@
 from collections import OrderedDict
 from pype.nuke import (
     plugin,
-    lib as pnlib
-    )
+    lib as pnlib)
 import nuke
 
 
@@ -81,15 +80,15 @@ class CreateWritePrerender(plugin.PypeCreator):
         else:
             self.log.info("Adding template path from plugin")
             write_data.update({
-                "fpath_template": "{work}/prerenders/nuke/{subset}/{subset}.{frame}.{ext}"})
+                "fpath_template": ("{work}/prerenders/nuke/{subset}"
+                                   "/{subset}.{frame}.{ext}")})
 
         write_node = pnlib.create_write_node(
             self.data["subset"],
             write_data,
             input=selected_node,
             prenodes=[],
-            review=False
-            )
+            review=False)
 
         # relinking to collected connections
         for i, input in enumerate(inputs):

From 8d9d965d6d4f8b6fa8a217e45415f3eee611f866 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 19:06:19 +0100
Subject: [PATCH 26/29] fix(nuke): hound pep8

---
 pype/plugins/nuke/create/create_write_render.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/pype/plugins/nuke/create/create_write_render.py b/pype/plugins/nuke/create/create_write_render.py
index c3b60ba2b0..06ef237305 100644
--- a/pype/plugins/nuke/create/create_write_render.py
+++ b/pype/plugins/nuke/create/create_write_render.py
@@ -1,8 +1,7 @@
 from collections import OrderedDict
 from pype.nuke import (
     plugin,
-    lib as pnlib
-    )
+    lib as pnlib)
 import nuke
 
 
@@ -82,7 +81,8 @@ class CreateWriteRender(plugin.PypeCreator):
         else:
             self.log.info("Adding template path from plugin")
             write_data.update({
-                "fpath_template": "{work}/renders/nuke/{subset}/{subset}.{frame}.{ext}"})
+                "fpath_template": ("{work}/renders/nuke/{subset}"
+                                   "/{subset}.{frame}.{ext}")})
 
         write_node = pnlib.create_write_node(
             self.data["subset"],

From 48d7489621f36fe6743207747f093df43f8512f9 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Wed, 25 Mar 2020 19:12:34 +0100
Subject: [PATCH 27/29] fix(global): hound pep8

---
 .../global/publish/extract_review_slate.py        | 15 +++++++++------
 pype/plugins/global/publish/submit_publish_job.py |  5 ++---
 .../nuke/publish/extract_review_data_mov.py       |  2 +-
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/pype/plugins/global/publish/extract_review_slate.py b/pype/plugins/global/publish/extract_review_slate.py
index da94c7714a..aaa67bde68 100644
--- a/pype/plugins/global/publish/extract_review_slate.py
+++ b/pype/plugins/global/publish/extract_review_slate.py
@@ -36,7 +36,8 @@ class ExtractReviewSlate(pype.api.Extractor):
         fps = inst_data.get("fps")
 
         # defining image ratios
-        resolution_ratio = (float(resolution_width) * pixel_aspect) /  resolution_height
+        resolution_ratio = ((float(resolution_width) * pixel_aspect) /
+                            resolution_height)
         delivery_ratio = float(to_width) / float(to_height)
         self.log.debug("__ resolution_ratio: `{}`".format(resolution_ratio))
         self.log.debug("__ delivery_ratio: `{}`".format(delivery_ratio))
@@ -91,7 +92,7 @@ class ExtractReviewSlate(pype.api.Extractor):
             input_args.extend([
                 "-r {}".format(fps),
                 "-t 0.04"]
-                )
+            )
 
             # output args
             codec_args = repre["_profile"].get('codec', [])
@@ -113,7 +114,7 @@ class ExtractReviewSlate(pype.api.Extractor):
                     self.log.debug("lower then delivery")
                     width_scale = int(to_width * scale_factor)
                     width_half_pad = int((
-                        to_width - width_scale)/2)
+                        to_width - width_scale) / 2)
                     height_scale = to_height
                     height_half_pad = 0
                 else:
@@ -126,7 +127,7 @@ class ExtractReviewSlate(pype.api.Extractor):
                     height_scale = int(
                         resolution_height * scale_factor)
                     height_half_pad = int(
-                        (to_height - height_scale)/2)
+                        (to_height - height_scale) / 2)
 
                 self.log.debug(
                     "__ width_scale: `{}`".format(width_scale))
@@ -137,8 +138,10 @@ class ExtractReviewSlate(pype.api.Extractor):
                 self.log.debug(
                     "__ height_half_pad: `{}`".format(height_half_pad))
 
-                scaling_arg = "scale={0}x{1}:flags=lanczos,pad={2}:{3}:{4}:{5}:black,setsar=1".format(
-                    width_scale, height_scale, to_width, to_height, width_half_pad, height_half_pad
+                scaling_arg = ("scale={0}x{1}:flags=lanczos,"
+                               "pad={2}:{3}:{4}:{5}:black,setsar=1").format(
+                    width_scale, height_scale, to_width, to_height,
+                    width_half_pad, height_half_pad
                 )
 
                 vf_back = self.add_video_filter_args(
diff --git a/pype/plugins/global/publish/submit_publish_job.py b/pype/plugins/global/publish/submit_publish_job.py
index 134f8e9098..af16b11db6 100644
--- a/pype/plugins/global/publish/submit_publish_job.py
+++ b/pype/plugins/global/publish/submit_publish_job.py
@@ -170,7 +170,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
         "review": ["lutPath"],
         "render2d": ["bakeScriptPath", "bakeRenderPath",
                      "bakeWriteNodeName", "version"]
-        }
+    }
 
     # list of family names to transfer to new family if present
     families_transfer = ["render3d", "render2d", "ftrack", "slate"]
@@ -589,8 +589,7 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin):
         if "prerender" in instance.data["families"]:
             instance_skeleton_data.update({
                 "family": "prerender",
-                "families": []
-                })
+                "families": []})
 
         # transfer specific families from original instance to new render
         for item in self.families_transfer:
diff --git a/pype/plugins/nuke/publish/extract_review_data_mov.py b/pype/plugins/nuke/publish/extract_review_data_mov.py
index 0bd5394548..7c56dc8b92 100644
--- a/pype/plugins/nuke/publish/extract_review_data_mov.py
+++ b/pype/plugins/nuke/publish/extract_review_data_mov.py
@@ -3,7 +3,7 @@ import pyblish.api
 from avalon.nuke import lib as anlib
 from pype.nuke import lib as pnlib
 import pype
-reload(pnlib)
+
 
 class ExtractReviewDataMov(pype.api.Extractor):
     """Extracts movie and thumbnail with baked in luts

From f152b9b6a7c691d962344519be36cf1d12b84b6f Mon Sep 17 00:00:00 2001
From: Ondrej Samohel <ondrej@samohel.cz>
Date: Thu, 26 Mar 2020 16:19:08 +0100
Subject: [PATCH 28/29] do not allow <RenderPass> token in Redshift

---
 .../maya/publish/validate_rendersettings.py   | 21 +++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/pype/plugins/maya/publish/validate_rendersettings.py b/pype/plugins/maya/publish/validate_rendersettings.py
index c98f0f8cdc..67239d4790 100644
--- a/pype/plugins/maya/publish/validate_rendersettings.py
+++ b/pype/plugins/maya/publish/validate_rendersettings.py
@@ -13,13 +13,17 @@ class ValidateRenderSettings(pyblish.api.InstancePlugin):
     """Validates the global render settings
 
     * File Name Prefix must start with: `maya/<Scene>`
-        all other token are customizable but sane values are:
+        all other token are customizable but sane values for Arnold are:
 
         `maya/<Scene>/<RenderLayer>/<RenderLayer>_<RenderPass>`
 
-        <Camera> token is supported also, usefull for multiple renderable
+        <Camera> token is supported also, useful for multiple renderable
         cameras per render layer.
 
+        For Redshift omit <RenderPass> token. Redshift will append it
+        automatically if AOVs are enabled and if you user Multipart EXR
+        it doesn't make much sense.
+
     * Frame Padding must be:
         * default: 4
 
@@ -127,8 +131,13 @@ class ValidateRenderSettings(pyblish.api.InstancePlugin):
             # no vray checks implemented yet
             pass
         elif renderer == "redshift":
-            # no redshift check implemented yet
-            pass
+            if re.search(cls.R_AOV_TOKEN, prefix):
+                invalid = True
+                cls.log.error("Do not use AOV token [ {} ] - "
+                              "Redshift automatically append AOV name and "
+                              "it doesn't make much sense with "
+                              "Multipart EXR".format(prefix))
+
         elif renderer == "renderman":
             file_prefix = cmds.getAttr("rmanGlobals.imageFileFormat")
             dir_prefix = cmds.getAttr("rmanGlobals.imageOutputDir")
@@ -143,8 +152,8 @@ class ValidateRenderSettings(pyblish.api.InstancePlugin):
                     dir_prefix))
 
         else:
-            multichannel = cmds.getAttr("defaultArnoldDriver.mergeAOVs")
-            if multichannel:
+            multipart = cmds.getAttr("defaultArnoldDriver.mergeAOVs")
+            if multipart:
                 if re.search(cls.R_AOV_TOKEN, prefix):
                     invalid = True
                     cls.log.error("Wrong image prefix [ {} ] - "

From 41e6eb05caf8bf053bc2d8183a484be278436f20 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Thu, 26 Mar 2020 17:06:00 +0100
Subject: [PATCH 29/29] fix(nuke): build first workfile was not accepting jpeg
 sequences

---
 pype/nuke/lib.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pype/nuke/lib.py b/pype/nuke/lib.py
index ad2d576da3..3ff9c6d397 100644
--- a/pype/nuke/lib.py
+++ b/pype/nuke/lib.py
@@ -1135,7 +1135,7 @@ class BuildWorkfile(WorkfileSettings):
                 regex_filter=None,
                 version=None,
                 representations=["exr", "dpx", "lutJson", "mov",
-                                 "preview", "png"]):
+                                 "preview", "png", "jpeg", "jpg"]):
         """
         A short description.