From 5806abcd6d3e5d8ce77bb7cfb430b0b81439af68 Mon Sep 17 00:00:00 2001 From: Ondrej Samohel Date: Tue, 1 Jun 2021 15:27:08 +0200 Subject: [PATCH] fix hound issues --- .../plugins/publish/submit_publish_job.py | 15 +-------------- openpype/plugins/publish/extract_review.py | 6 ++---- openpype/plugins/publish/integrate_new.py | 4 ++-- 3 files changed, 5 insertions(+), 20 deletions(-) diff --git a/openpype/modules/deadline/plugins/publish/submit_publish_job.py b/openpype/modules/deadline/plugins/publish/submit_publish_job.py index 0703e7feda..52ed9bba76 100644 --- a/openpype/modules/deadline/plugins/publish/submit_publish_job.py +++ b/openpype/modules/deadline/plugins/publish/submit_publish_job.py @@ -433,18 +433,10 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin): app = os.environ.get("AVALON_APP", "") preview = False - self.log.info("ffffff") - self.log.info(app) if app in self.aov_filter.keys(): - self.log.info("in") for aov_pattern in self.aov_filter[app]: - self.log.info(aov_pattern) - self.log.info(aov) - if re.match(aov_pattern, - aov - ): + if re.match(aov_pattern, aov): preview = True - self.log.info("{}:{}:{}".format(aov, app, instance_data)) break new_instance = copy(instance_data) @@ -511,23 +503,18 @@ class ProcessSubmittedJobOnFarm(pyblish.api.InstancePlugin): bake_render_path = instance.get("bakeRenderPath", []) # create representation for every collected sequence - self.log.debug("-------") for collection in collections: - self.log.debug(collection) ext = collection.tail.lstrip(".") preview = False # if filtered aov name is found in filename, toggle it for # preview video rendering for app in self.aov_filter.keys(): - self.log.debug(app) if os.environ.get("AVALON_APP", "") == app: for aov in self.aov_filter[app]: - self.log.debug(aov) if re.match( aov, list(collection)[0] ): - self.log.info("{}:{}:{}".format(aov, app, instance)) preview = True break diff --git a/openpype/plugins/publish/extract_review.py b/openpype/plugins/publish/extract_review.py index 6aea4b1800..e1e24af3ea 100644 --- a/openpype/plugins/publish/extract_review.py +++ b/openpype/plugins/publish/extract_review.py @@ -191,9 +191,9 @@ class ExtractReview(pyblish.api.InstancePlugin): "New representation tags: `{}`".format(new_repre["tags"]) ) - temp_data = self.prepare_temp_data(instance, repre, output_def) + temp_data = self.prepare_temp_data( + instance, repre, output_def) files_to_clean = [] - self.log.info("Is sequence: {}".format(temp_data["input_is_sequence"])) if temp_data["input_is_sequence"]: self.log.info("Filling gaps in sequence.") files_to_clean = self.fill_sequence_gaps( @@ -641,8 +641,6 @@ class ExtractReview(pyblish.api.InstancePlugin): AssertionError: if more then one collection is obtained. """ - from pprint import pprint - collections = clique.assemble(files)[0] assert len(collections) == 1, "Multiple collections found." col = collections[0] diff --git a/openpype/plugins/publish/integrate_new.py b/openpype/plugins/publish/integrate_new.py index 2b9b02001d..e1c1a56d5c 100644 --- a/openpype/plugins/publish/integrate_new.py +++ b/openpype/plugins/publish/integrate_new.py @@ -418,7 +418,6 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin): dst_padding_exp = src_padding_exp dst_start_frame = None collection_start = list(src_collection.indexes)[0] - index_frame = index_frame_start for i in src_collection.indexes: # TODO 1.) do not count padding in each index iteration # 2.) do not count dst_padding from src_padding before @@ -433,7 +432,8 @@ class IntegrateAssetNew(pyblish.api.InstancePlugin): if index_frame_start is not None: dst_padding_exp = "%0{}d".format(frame_start_padding) - dst_padding = dst_padding_exp % (index_frame_start + frame_number) + dst_padding = dst_padding_exp % ( + index_frame_start + frame_number) dst = "{0}{1}{2}".format( dst_head,