From fbee0a8b3c295dc55cd64a3037f517f88de150fa Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Fri, 12 May 2023 11:07:50 +0200
Subject: [PATCH] pr comments

---
 .../fusion/plugins/publish/collect_render.py  |  3 ---
 .../validate_expected_frames_existence.py     | 23 +++++++++----------
 .../plugins/publish/submit_fusion_deadline.py |  3 ---
 3 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/openpype/hosts/fusion/plugins/publish/collect_render.py b/openpype/hosts/fusion/plugins/publish/collect_render.py
index 5adb8a13f0..4898226f03 100644
--- a/openpype/hosts/fusion/plugins/publish/collect_render.py
+++ b/openpype/hosts/fusion/plugins/publish/collect_render.py
@@ -1,5 +1,4 @@
 import os
-from pprint import pformat
 import attr
 import pyblish.api
 
@@ -92,7 +91,6 @@ class CollectFusionRender(
             )
 
             render_target = inst.data["creator_attributes"]["render_target"]
-            self.log.debug("render_target: '{}'".format(render_target))
 
             if render_target == "local":
                 # for local renders
@@ -125,7 +123,6 @@ class CollectFusionRender(
         for instance in self._context:
             if "render.frames" in instance.data.get("families", []):
                 self._update_for_frames(instance)
-            self.log.debug(pformat(instance.data))
 
     def get_expected_files(self, render_instance):
         """
diff --git a/openpype/hosts/fusion/plugins/publish/validate_expected_frames_existence.py b/openpype/hosts/fusion/plugins/publish/validate_expected_frames_existence.py
index befaae13be..aa89799867 100644
--- a/openpype/hosts/fusion/plugins/publish/validate_expected_frames_existence.py
+++ b/openpype/hosts/fusion/plugins/publish/validate_expected_frames_existence.py
@@ -23,21 +23,20 @@ class ValidateLocalFramesExistence(pyblish.api.InstancePlugin):
         if non_existing_frames is None:
             non_existing_frames = []
 
-        if "render.frames" in instance.data.get("families", []):
-            tool = instance.data["toolSaver"]
+        tool = instance.data["toolSaver"]
 
-            expected_files = instance.data["expectedFiles"]
+        expected_files = instance.data["expectedFiles"]
 
-            for file in expected_files:
-                if not os.path.exists(file):
-                    cls.log.error(
-                        f"Missing file: {file}"
-                    )
-                    non_existing_frames.append(file)
+        for file in expected_files:
+            if not os.path.exists(file):
+                cls.log.error(
+                    f"Missing file: {file}"
+                )
+                non_existing_frames.append(file)
 
-            if len(non_existing_frames) > 0:
-                cls.log.error(f"Some of {tool.Name}'s files does not exist")
-                return [tool]
+        if len(non_existing_frames) > 0:
+            cls.log.error(f"Some of {tool.Name}'s files does not exist")
+            return [tool]
 
     def process(self, instance):
         non_existing_frames = []
diff --git a/openpype/modules/deadline/plugins/publish/submit_fusion_deadline.py b/openpype/modules/deadline/plugins/publish/submit_fusion_deadline.py
index 891bfde6c5..af4bd37302 100644
--- a/openpype/modules/deadline/plugins/publish/submit_fusion_deadline.py
+++ b/openpype/modules/deadline/plugins/publish/submit_fusion_deadline.py
@@ -1,7 +1,6 @@
 import os
 import json
 import getpass
-from pprint import pformat
 
 import requests
 
@@ -90,8 +89,6 @@ class FusionSubmitDeadline(
         attribute_values = self.get_attr_values_from_data(
             instance.data)
 
-        self.log.debug(pformat(attribute_values))
-
         # add suspend_publish attributeValue to instance data
         instance.data["suspend_publish"] = attribute_values[
             "suspend_publish"]