From 492c42d13f97fa5b0f37ceb2001c7d29a4716ff4 Mon Sep 17 00:00:00 2001 From: Jakub Jezek Date: Tue, 28 Apr 2020 16:09:09 +0200 Subject: [PATCH] clean(ppro): hound recommendations --- pype/avalon_apps/rest_api.py | 3 ++- pype/hooks/premiere/prelaunch.py | 3 --- .../adobecommunicator/publish/collect_context.py | 3 ++- pype/plugins/global/publish/integrate_new.py | 10 ++++++---- pype/plugins/premiere/publish/collect_resolution.py | 2 +- pype/premiere/__init__.py | 4 +--- 6 files changed, 12 insertions(+), 13 deletions(-) diff --git a/pype/avalon_apps/rest_api.py b/pype/avalon_apps/rest_api.py index aba67abd7e6..1267ee39928 100644 --- a/pype/avalon_apps/rest_api.py +++ b/pype/avalon_apps/rest_api.py @@ -70,7 +70,8 @@ def get_assets(self, request): _asset, identificator, _project_name )) - @RestApi.route("/publish/", url_prefix="/premiere", methods="GET") + @RestApi.route("/publish/", + url_prefix="/premiere", methods="GET") def publish(self, request): """ http://localhost:8021/premiere/publish/shot021?json_in=this/path/file_in.json&json_out=this/path/file_out.json diff --git a/pype/hooks/premiere/prelaunch.py b/pype/hooks/premiere/prelaunch.py index 26bfff28f44..f2edb60195a 100644 --- a/pype/hooks/premiere/prelaunch.py +++ b/pype/hooks/premiere/prelaunch.py @@ -2,9 +2,6 @@ import traceback from pype.lib import PypeHook from pypeapp import Logger -import importlib -import avalon.api -import pype.premiere from pype.premiere import lib as prlib diff --git a/pype/plugins/adobecommunicator/publish/collect_context.py b/pype/plugins/adobecommunicator/publish/collect_context.py index 7759fbd6dff..139dd86480e 100644 --- a/pype/plugins/adobecommunicator/publish/collect_context.py +++ b/pype/plugins/adobecommunicator/publish/collect_context.py @@ -24,7 +24,8 @@ class CollectContextDataFromAport(pyblish.api.ContextPlugin): order = pyblish.api.CollectorOrder - 0.49 def process(self, context): - self.log.info("registred_hosts: `{}`".format(pyblish.api.registered_hosts())) + self.log.info( + "registred_hosts: `{}`".format(pyblish.api.registered_hosts())) io.install() # get json paths from data input_json_path = os.environ.get("AC_PUBLISH_INPATH") diff --git a/pype/plugins/global/publish/integrate_new.py b/pype/plugins/global/publish/integrate_new.py index d31cf5bf545..9ea3d0bda9a 100644 --- a/pype/plugins/global/publish/integrate_new.py +++ b/pype/plugins/global/publish/integrate_new.py @@ -610,7 +610,7 @@ def get_subset(self, asset, instance): "name": subset_name, "data": { "families": instance.data.get('families') - }, + }, "parent": asset["_id"] }).inserted_id @@ -664,15 +664,17 @@ def create_version_data(self, context, instance): families += current_families self.log.debug("Registered root: {}".format(api.registered_root())) - self.log.debug("PYPE_STUDIO_PROJECTS_MOUNT: {}".format(os.getenv("PYPE_STUDIO_PROJECTS_MOUNT"))) + # create relative source path for DB try: source = instance.data['source'] except KeyError: source = context.data["currentFile"] self.log.debug("source: {}".format(source)) - source = str(source).replace(os.getenv("PYPE_STUDIO_PROJECTS_MOUNT"), - api.registered_root()) + source = str(source).replace( + os.getenv("PYPE_STUDIO_PROJECTS_MOUNT"), + api.registered_root() + ) relative_path = os.path.relpath(source, api.registered_root()) source = os.path.join("{root}", relative_path).replace("\\", "/") diff --git a/pype/plugins/premiere/publish/collect_resolution.py b/pype/plugins/premiere/publish/collect_resolution.py index 6c5a735a5d7..d93dbf0b66e 100644 --- a/pype/plugins/premiere/publish/collect_resolution.py +++ b/pype/plugins/premiere/publish/collect_resolution.py @@ -22,7 +22,7 @@ def process(self, instance): pixel_aspect = float(metadata['ppro.format.pixelaspect']) res_width = metadata['ppro.format.width'] res_height = metadata['ppro.format.height'] - + instance.data['pixelAspect'] = pixel_aspect instance.data['resolutionWidth'] = res_width instance.data['resolutionHeight'] = res_height diff --git a/pype/premiere/__init__.py b/pype/premiere/__init__.py index 6e178e704a7..9dfe2ec3842 100644 --- a/pype/premiere/__init__.py +++ b/pype/premiere/__init__.py @@ -9,10 +9,8 @@ reload_pipeline, ls, LOAD_PATH, - INVENTORY_PATH, CREATE_PATH, - PUBLISH_PATH, - PLUGINS_DIR + PUBLISH_PATH ) __all__ = [