From 0fbc2aeceaf67e0db5c95608dfa68af43d76833b Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Thu, 23 Jan 2020 12:17:46 +0100
Subject: [PATCH 1/3] feat(ftrack): disable ftrack comment integration

---
 pype/plugins/ftrack/publish/integrate_ftrack_comments.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/pype/plugins/ftrack/publish/integrate_ftrack_comments.py b/pype/plugins/ftrack/publish/integrate_ftrack_comments.py
index 9d0b7b3ab9..4f7afb4346 100644
--- a/pype/plugins/ftrack/publish/integrate_ftrack_comments.py
+++ b/pype/plugins/ftrack/publish/integrate_ftrack_comments.py
@@ -9,6 +9,7 @@ class IntegrateFtrackComments(pyblish.api.InstancePlugin):
     order = pyblish.api.IntegratorOrder
     label = "Integrate Comments to Ftrack."
     families = ["shot"]
+    enabled = False
 
     def process(self, instance):
         session = instance.context.data["ftrackSession"]

From da990057f45124d52463c51e21983b7e36d933bc Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 24 Jan 2020 14:26:46 +0100
Subject: [PATCH 2/3] feat(nuke): adding validator for output resolution

---
 .../publish/validate_output_resolution.py     | 78 +++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644 pype/plugins/nuke/publish/validate_output_resolution.py

diff --git a/pype/plugins/nuke/publish/validate_output_resolution.py b/pype/plugins/nuke/publish/validate_output_resolution.py
new file mode 100644
index 0000000000..2563ee929f
--- /dev/null
+++ b/pype/plugins/nuke/publish/validate_output_resolution.py
@@ -0,0 +1,78 @@
+import nuke
+
+import pyblish.api
+
+
+class RepairWriteResolutionDifference(pyblish.api.Action):
+
+    label = "Repair"
+    icon = "wrench"
+    on = "failed"
+
+    def process(self, context, plugin):
+
+        # Get the errored instances
+        failed = []
+        for result in context.data["results"]:
+            if (result["error"] is not None and result["instance"] is not None
+               and result["instance"] not in failed):
+                failed.append(result["instance"])
+
+        # Apply pyblish.logic to get the instances for the plug-in
+        instances = pyblish.api.instances_by_plugin(failed, plugin)
+
+        for instance in instances:
+            reformat = instance[0].dependencies()[0]
+            if reformat.Class() != "Reformat":
+                reformat = nuke.nodes.Reformat(inputs=[instance[0].input(0)])
+
+                xpos = instance[0].xpos()
+                ypos = instance[0].ypos() - 26
+
+                dependent_ypos = instance[0].dependencies()[0].ypos()
+                if (instance[0].ypos() - dependent_ypos) <= 51:
+                    xpos += 110
+
+                reformat.setXYpos(xpos, ypos)
+
+                instance[0].setInput(0, reformat)
+
+            reformat["resize"].setValue("none")
+
+
+class ValidateOutputResolution(pyblish.api.InstancePlugin):
+    """Validates Output Resolution.
+
+    It is making sure the resolution of write's input is the same as
+    Format definition of script in Root node.
+    """
+
+    order = pyblish.api.ValidatorOrder
+    optional = True
+    families = ["render", "render.local", "render.farm"]
+    label = "Write Resolution"
+    hosts = ["nuke"]
+    actions = [RepairWriteResolutionDifference]
+
+    def process(self, instance):
+
+        # Skip bounding box check if a crop node exists.
+        if instance[0].dependencies()[0].Class() == "Crop":
+            return
+
+        msg = "Bounding box is outside the format."
+        assert self.check_resolution(instance), msg
+
+    def check_resolution(self, instance):
+        node = instance[0]
+
+        root_width = instance.data["resolutionWidth"]
+        root_height = instance.data["resolutionHeight"]
+
+        write_width = node.format().width()
+        write_height = node.format().height()
+
+        if (root_width != write_width) or (root_height != write_height):
+            return None
+        else:
+            return True

From 675da4d1359898ea4e0cc6311b90882373de44f8 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakub@pype.club>
Date: Fri, 24 Jan 2020 14:27:16 +0100
Subject: [PATCH 3/3] fix(nuke): making sure validator will run on all render
 families

---
 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 417d4ab004..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 = ["render"]
+    families = ["render", "render.local", "render.farm"]
     label = "Write Bounding Box"
     hosts = ["nuke"]
     actions = [RepairNukeBoundingBoxAction]