From 54e2687afa13b4963b86ecbc4b99786e1a57ced2 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Tue, 18 Jul 2023 10:21:21 +0300
Subject: [PATCH 01/49] create review validator

---
 .../publish/validate_review_colorspace.py     | 143 ++++++++++++++++++
 1 file changed, 143 insertions(+)
 create mode 100644 openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
new file mode 100644
index 0000000000..18ca39234c
--- /dev/null
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -0,0 +1,143 @@
+# -*- coding: utf-8 -*-
+import pyblish.api
+
+from collections import defaultdict
+from openpype.pipeline import PublishValidationError
+
+
+class ValidateReviewColorspace(pyblish.api.InstancePlugin):
+    """Validate Review Colorspace parameters.
+
+
+    """
+
+    order = pyblish.api.ValidatorOrder + 0.1
+    families = ["review"]
+    hosts = ["houdini"]
+    label = "Validate Review Colorspace"
+
+    def process(self, instance):
+        invalid = self.get_invalid(instance)
+        if invalid:
+            raise PublishValidationError(
+                ("Colorspace parameter is not valid."),
+                title=self.label
+            )
+
+    @classmethod
+    def get_invalid(cls, instance):
+        import hou  # noqa
+        output_node = instance.data.get("output_node")
+        rop_node = hou.node(instance.data["instance_node"])
+        if output_node is None:
+            cls.log.error(
+                "SOP Output node in '%s' does not exist. "
+                "Ensure a valid SOP output path is set." % rop_node.path()
+            )
+
+            return [rop_node.path()]
+
+        pattern = rop_node.parm("prim_to_detail_pattern").eval().strip()
+        if not pattern:
+            cls.log.debug(
+                "Alembic ROP has no 'Primitive to Detail' pattern. "
+                "Validation is ignored.."
+            )
+            return
+
+        build_from_path = rop_node.parm("build_from_path").eval()
+        if not build_from_path:
+            cls.log.debug(
+                "Alembic ROP has 'Build from Path' disabled. "
+                "Validation is ignored.."
+            )
+            return
+
+        path_attr = rop_node.parm("path_attrib").eval()
+        if not path_attr:
+            cls.log.error(
+                "The Alembic ROP node has no Path Attribute"
+                "value set, but 'Build Hierarchy from Attribute'"
+                "is enabled."
+            )
+            return [rop_node.path()]
+
+        # Let's assume each attribute is explicitly named for now and has no
+        # wildcards for Primitive to Detail. This simplifies the check.
+        cls.log.debug("Checking Primitive to Detail pattern: %s" % pattern)
+        cls.log.debug("Checking with path attribute: %s" % path_attr)
+
+        if not hasattr(output_node, "geometry"):
+            # In the case someone has explicitly set an Object
+            # node instead of a SOP node in Geometry context
+            # then for now we ignore - this allows us to also
+            # export object transforms.
+            cls.log.warning("No geometry output node found, skipping check..")
+            return
+
+        # Check if the primitive attribute exists
+        frame = instance.data.get("frameStart", 0)
+        geo = output_node.geometryAtFrame(frame)
+
+        # If there are no primitives on the start frame then it might be
+        # something that is emitted over time. As such we can't actually
+        # validate whether the attributes exist, because they won't exist
+        # yet. In that case, just warn the user and allow it.
+        if len(geo.iterPrims()) == 0:
+            cls.log.warning(
+                "No primitives found on current frame. Validation"
+                " for Primitive to Detail will be skipped."
+            )
+            return
+
+        attrib = geo.findPrimAttrib(path_attr)
+        if not attrib:
+            cls.log.info(
+                "Geometry Primitives are missing "
+                "path attribute: `%s`" % path_attr
+            )
+            return [output_node.path()]
+
+        # Ensure at least a single string value is present
+        if not attrib.strings():
+            cls.log.info(
+                "Primitive path attribute has no "
+                "string values: %s" % path_attr
+            )
+            return [output_node.path()]
+
+        paths = None
+        for attr in pattern.split(" "):
+            if not attr.strip():
+                # Ignore empty values
+                continue
+
+            # Check if the primitive attribute exists
+            attrib = geo.findPrimAttrib(attr)
+            if not attrib:
+                # It is allowed to not have the attribute at all
+                continue
+
+            # The issue can only happen if at least one string attribute is
+            # present. So we ignore cases with no values whatsoever.
+            if not attrib.strings():
+                continue
+
+            check = defaultdict(set)
+            values = geo.primStringAttribValues(attr)
+            if paths is None:
+                paths = geo.primStringAttribValues(path_attr)
+
+            for path, value in zip(paths, values):
+                check[path].add(value)
+
+            for path, values in check.items():
+                # Whenever a single path has multiple values for the
+                # Primitive to Detail attribute then we consider it
+                # inconsistent and invalidate the ROP node's content.
+                if len(values) > 1:
+                    cls.log.warning(
+                        "Path has multiple values: %s (path: %s)"
+                        % (list(values), path)
+                    )
+                    return [output_node.path()]

From 94ec68ab21182bbb01258a9e73ccb01b10a5a1e8 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 19 Jul 2023 00:13:51 +0300
Subject: [PATCH 02/49] update review validator

---
 .../publish/validate_review_colorspace.py     | 153 ++++++------------
 1 file changed, 49 insertions(+), 104 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 18ca39234c..47c1e886d1 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -1,143 +1,88 @@
 # -*- coding: utf-8 -*-
 import pyblish.api
-
-from collections import defaultdict
 from openpype.pipeline import PublishValidationError
+from openpype.pipeline.publish import RepairAction
+from openpype.hosts.houdini.api.action import SelectROPAction
+
+
+class SetDefaultViewSpaceAction(RepairAction):
+    label = "Set default view space"
+    icon = "mdi.monitor"
 
 
 class ValidateReviewColorspace(pyblish.api.InstancePlugin):
     """Validate Review Colorspace parameters.
 
-
+    It checks if 'OCIO Colorspace' parameter was set to valid value.
     """
 
     order = pyblish.api.ValidatorOrder + 0.1
     families = ["review"]
     hosts = ["houdini"]
     label = "Validate Review Colorspace"
+    actions = [SetDefaultViewSpaceAction, SelectROPAction]
 
     def process(self, instance):
         invalid = self.get_invalid(instance)
         if invalid:
             raise PublishValidationError(
-                ("Colorspace parameter is not valid."),
+                ("'OCIO Colorspace' parameter is not valid."),
                 title=self.label
             )
 
     @classmethod
     def get_invalid(cls, instance):
         import hou  # noqa
-        output_node = instance.data.get("output_node")
+
         rop_node = hou.node(instance.data["instance_node"])
-        if output_node is None:
-            cls.log.error(
-                "SOP Output node in '%s' does not exist. "
-                "Ensure a valid SOP output path is set." % rop_node.path()
-            )
-
-            return [rop_node.path()]
-
-        pattern = rop_node.parm("prim_to_detail_pattern").eval().strip()
-        if not pattern:
-            cls.log.debug(
-                "Alembic ROP has no 'Primitive to Detail' pattern. "
-                "Validation is ignored.."
-            )
-            return
-
-        build_from_path = rop_node.parm("build_from_path").eval()
-        if not build_from_path:
-            cls.log.debug(
-                "Alembic ROP has 'Build from Path' disabled. "
-                "Validation is ignored.."
-            )
-            return
-
-        path_attr = rop_node.parm("path_attrib").eval()
-        if not path_attr:
-            cls.log.error(
-                "The Alembic ROP node has no Path Attribute"
-                "value set, but 'Build Hierarchy from Attribute'"
-                "is enabled."
-            )
-            return [rop_node.path()]
-
-        # Let's assume each attribute is explicitly named for now and has no
-        # wildcards for Primitive to Detail. This simplifies the check.
-        cls.log.debug("Checking Primitive to Detail pattern: %s" % pattern)
-        cls.log.debug("Checking with path attribute: %s" % path_attr)
-
-        if not hasattr(output_node, "geometry"):
-            # In the case someone has explicitly set an Object
-            # node instead of a SOP node in Geometry context
-            # then for now we ignore - this allows us to also
-            # export object transforms.
-            cls.log.warning("No geometry output node found, skipping check..")
-            return
-
-        # Check if the primitive attribute exists
-        frame = instance.data.get("frameStart", 0)
-        geo = output_node.geometryAtFrame(frame)
-
-        # If there are no primitives on the start frame then it might be
-        # something that is emitted over time. As such we can't actually
-        # validate whether the attributes exist, because they won't exist
-        # yet. In that case, just warn the user and allow it.
-        if len(geo.iterPrims()) == 0:
+        if hou.Color.ocio_defaultDisplay() == "default":
             cls.log.warning(
-                "No primitives found on current frame. Validation"
-                " for Primitive to Detail will be skipped."
+                "Default Houdini colorspace is used, "
+                " skipping check.."
             )
             return
 
-        attrib = geo.findPrimAttrib(path_attr)
-        if not attrib:
-            cls.log.info(
-                "Geometry Primitives are missing "
-                "path attribute: `%s`" % path_attr
+        if rop_node.evalParm("colorcorrect") != 2:
+            # any colorspace settings other than default requires
+            # 'Color Correct' parm to be set to 'OpenColorIO'
+            rop_node.setParms({"colorcorrect": 2})
+            cls.log.debug(
+                "'Color Correct' parm on '%s' has been set to"
+                " 'OpenColorIO'", rop_node
             )
-            return [output_node.path()]
 
-        # Ensure at least a single string value is present
-        if not attrib.strings():
-            cls.log.info(
-                "Primitive path attribute has no "
-                "string values: %s" % path_attr
+        if rop_node.evalParm("ociocolorspace") not in \
+                hou.Color.ocio_spaces():
+
+            cls.log.error(
+                "'OCIO Colorspace' value on '%s' is not valid, "
+                "select a valid option from the dropdown menu.",
+                rop_node
             )
-            return [output_node.path()]
+            return rop_node
 
-        paths = None
-        for attr in pattern.split(" "):
-            if not attr.strip():
-                # Ignore empty values
-                continue
+    @classmethod
+    def repair(cls, instance):
+        """Set Default View Space Action.
 
-            # Check if the primitive attribute exists
-            attrib = geo.findPrimAttrib(attr)
-            if not attrib:
-                # It is allowed to not have the attribute at all
-                continue
+        It is a helper action more than a repair action,
+        used to set colorspace on opengl node to the default view.
+        """
 
-            # The issue can only happen if at least one string attribute is
-            # present. So we ignore cases with no values whatsoever.
-            if not attrib.strings():
-                continue
+        import hou
+        import PyOpenColorIO as OCIO
 
-            check = defaultdict(set)
-            values = geo.primStringAttribValues(attr)
-            if paths is None:
-                paths = geo.primStringAttribValues(path_attr)
+        rop_node = hou.node(instance.data["instance_node"])
 
-            for path, value in zip(paths, values):
-                check[path].add(value)
+        config = OCIO.GetCurrentConfig()
+        display = hou.Color.ocio_defaultDisplay()
+        view = hou.Color.ocio_defaultView()
 
-            for path, values in check.items():
-                # Whenever a single path has multiple values for the
-                # Primitive to Detail attribute then we consider it
-                # inconsistent and invalidate the ROP node's content.
-                if len(values) > 1:
-                    cls.log.warning(
-                        "Path has multiple values: %s (path: %s)"
-                        % (list(values), path)
-                    )
-                    return [output_node.path()]
+        default_view_space = config.getDisplayColorSpaceName(
+            display, view)
+
+        rop_node.setParms({"ociocolorspace" : default_view_space})
+        cls.log.debug(
+            "'OCIO Colorspace' parm on '%s' has been set to '%s'",
+            default_view_space, rop_node
+        )

From 4d79320cf4f1abdd05dc3ce6fc46d6ccb30b2aca Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 19 Jul 2023 00:50:43 +0300
Subject: [PATCH 03/49] make hound happy

---
 .../hosts/houdini/plugins/publish/validate_review_colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 47c1e886d1..02284dc641 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -81,7 +81,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         default_view_space = config.getDisplayColorSpaceName(
             display, view)
 
-        rop_node.setParms({"ociocolorspace" : default_view_space})
+        rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(
             "'OCIO Colorspace' parm on '%s' has been set to '%s'",
             default_view_space, rop_node

From f4a5858edbed465f11908d18697b5dc0c6414b76 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 19 Jul 2023 19:43:39 +0300
Subject: [PATCH 04/49] update validator

---
 .../houdini/plugins/publish/validate_review_colorspace.py  | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 02284dc641..8f3799cde4 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -33,9 +33,10 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
     @classmethod
     def get_invalid(cls, instance):
         import hou  # noqa
+        import os
 
         rop_node = hou.node(instance.data["instance_node"])
-        if hou.Color.ocio_defaultDisplay() == "default":
+        if os.getenv("OCIO") is None:
             cls.log.warning(
                 "Default Houdini colorspace is used, "
                 " skipping check.."
@@ -78,8 +79,8 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         display = hou.Color.ocio_defaultDisplay()
         view = hou.Color.ocio_defaultView()
 
-        default_view_space = config.getDisplayColorSpaceName(
-            display, view)
+        default_view_space = config.getDisplayViewColorSpaceName(
+            display, view) # works with PyOpenColorIO 2.2.1
 
         rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(

From 00d4afd1178d4ee95ee8d09a4a8c8348f3948606 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 19 Jul 2023 19:44:45 +0300
Subject: [PATCH 05/49] make hound happy

---
 .../hosts/houdini/plugins/publish/validate_review_colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 8f3799cde4..addfa05bf1 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -80,7 +80,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         view = hou.Color.ocio_defaultView()
 
         default_view_space = config.getDisplayViewColorSpaceName(
-            display, view) # works with PyOpenColorIO 2.2.1
+            display, view)  # works with PyOpenColorIO 2.2.1
 
         rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(

From d90d45c56df70e876d02acbf9d4e09aa1b6f5746 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Thu, 20 Jul 2023 13:38:09 +0300
Subject: [PATCH 06/49] move get function to colorspace.py

---
 .../publish/validate_review_colorspace.py     | 19 +++++---
 openpype/pipeline/colorspace.py               | 36 ++++++++++++++++
 openpype/scripts/ocio_wrapper.py              | 43 +++++++++++++++++++
 3 files changed, 91 insertions(+), 7 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index addfa05bf1..cfc5a5d71d 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -71,19 +71,24 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         """
 
         import hou
-        import PyOpenColorIO as OCIO
+        from openpype.pipeline.colorspace import get_display_view_colorspace_name
+        from openpype.hosts.houdini.api.lib import get_color_management_preferences #noqa
 
         rop_node = hou.node(instance.data["instance_node"])
 
-        config = OCIO.GetCurrentConfig()
-        display = hou.Color.ocio_defaultDisplay()
-        view = hou.Color.ocio_defaultView()
+        data =  get_color_management_preferences()
+        config_path = data.get("config")
+        display =  data.get("display")
+        view =  data.get("view")
 
-        default_view_space = config.getDisplayViewColorSpaceName(
-            display, view)  # works with PyOpenColorIO 2.2.1
+        cls.log.debug("Get default view colorspace name..")
+
+        default_view_space = get_display_view_colorspace_name(config_path,
+                                                              display, view)
 
         rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(
-            "'OCIO Colorspace' parm on '%s' has been set to '%s'",
+            "'OCIO Colorspace' parm on '%s' has been set to "
+            "the default view color space '%s'",
             default_view_space, rop_node
         )
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 3f2d4891c1..a1d86b2fec 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -589,3 +589,39 @@ def _get_imageio_settings(project_settings, host_name):
     imageio_host = project_settings.get(host_name, {}).get("imageio", {})
 
     return imageio_global, imageio_host
+
+def get_display_view_colorspace_name(config_path, display, view):
+
+    if not compatibility_check():
+        # python environment is not compatible with PyOpenColorIO
+        # needs to be run in subprocess
+        return get_display_view_colorspace_subprocess(config_path,
+                                                      display, view)
+
+    from openpype.scripts.ocio_wrapper import _get_display_view_colorspace_name #noqa
+
+    return _get_display_view_colorspace_name(config_path, display, view)
+
+def get_display_view_colorspace_subprocess(config_path, display, view):
+    with _make_temp_json_file() as tmp_json_path:
+        # Prepare subprocess arguments
+        args = [
+            "run", get_ocio_config_script_path(),
+            "config", "get_display_view_colorspace_name",
+            "--in_path", config_path,
+            "--out_path", tmp_json_path,
+            "--display", display,
+            "--view", view
+
+        ]
+        log.info("Executing: {}".format(" ".join(args)))
+
+        process_kwargs = {
+            "logger": log
+        }
+
+        run_openpype_process(*args, **process_kwargs)
+
+        # return all colorspaces
+        return_json_data = open(tmp_json_path).read()
+        return json.loads(return_json_data)
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index 16558642c6..ca703fd65c 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -173,6 +173,49 @@ def _get_views_data(config_path):
 
     return data
 
+def _get_display_view_colorspace_name(config_path, display, view):
+    config_path = Path(config_path)
+
+    if not config_path.is_file():
+        raise IOError("Input path should be `config.ocio` file")
+
+    config = ocio.Config().CreateFromFile(str(config_path))
+    colorspace = config.getDisplayViewColorSpaceName(display, view)
+
+    return colorspace
+
+@config.command(
+    name="get_display_view_colorspace_name",
+    help=(
+        "return default view colorspace name "
+        "for the given display and view "
+        "--path input arg is required"
+    )
+)
+@click.option("--in_path", required=True,
+              help="path where to read ocio config file",
+              type=click.Path(exists=True))
+@click.option("--out_path", required=True,
+              help="path where to write output json file",
+              type=click.Path())
+@click.option("--display", required=True,
+              help="display",
+              type=click.STRING)
+@click.option("--view", required=True,
+              help="view",
+              type=click.STRING)
+def get_display_view_colorspace_name(in_path, out_path,
+                                     display, view):
+
+    json_path = Path(out_path)
+
+    out_data = _get_display_view_colorspace_name(in_path,
+                                                 display, view)
+
+    with open(json_path, "w") as f:
+        json.dump(out_data, f)
+
+    print(f"Viewer data are saved to '{json_path}'")
 
 if __name__ == '__main__':
     main()

From 2bb11d956ca2ec629f61d5d14e0b2ef4130a0363 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Thu, 20 Jul 2023 13:54:46 +0300
Subject: [PATCH 07/49] add doc strings

---
 .../publish/validate_review_colorspace.py     |  2 +-
 openpype/pipeline/colorspace.py               | 22 ++++++++++++
 openpype/scripts/ocio_wrapper.py              | 34 +++++++++++++++++--
 3 files changed, 55 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index cfc5a5d71d..67e29e0ee2 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -71,7 +71,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         """
 
         import hou
-        from openpype.pipeline.colorspace import get_display_view_colorspace_name
+        from openpype.pipeline.colorspace import get_display_view_colorspace_name #noqa
         from openpype.hosts.houdini.api.lib import get_color_management_preferences #noqa
 
         rop_node = hou.node(instance.data["instance_node"])
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index a1d86b2fec..22e8175a7e 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -591,6 +591,16 @@ def _get_imageio_settings(project_settings, host_name):
     return imageio_global, imageio_host
 
 def get_display_view_colorspace_name(config_path, display, view):
+    """get view colorspace name for the given display and view.
+
+    Args:
+        config_path (str): path string leading to config.ocio
+        display (str): display name e.g. "ACES"
+        view (str): view name e.g. "sRGB"
+
+    Returns:
+        view color space name (str) e.g. "Output - sRGB"
+    """
 
     if not compatibility_check():
         # python environment is not compatible with PyOpenColorIO
@@ -603,6 +613,18 @@ def get_display_view_colorspace_name(config_path, display, view):
     return _get_display_view_colorspace_name(config_path, display, view)
 
 def get_display_view_colorspace_subprocess(config_path, display, view):
+    """get view colorspace name for the given display and view
+        via subprocess.
+
+    Args:
+        config_path (str): path string leading to config.ocio
+        display (str): display name e.g. "ACES"
+        view (str): view name e.g. "sRGB"
+
+    Returns:
+        view color space name (str) e.g. "Output - sRGB"
+    """
+
     with _make_temp_json_file() as tmp_json_path:
         # Prepare subprocess arguments
         args = [
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index ca703fd65c..f94faabe11 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -174,6 +174,21 @@ def _get_views_data(config_path):
     return data
 
 def _get_display_view_colorspace_name(config_path, display, view):
+    """get view colorspace name for the given display and view.
+
+    Args:
+        config_path (str): path string leading to config.ocio
+        display (str): display name e.g. "ACES"
+        view (str): view name e.g. "sRGB"
+
+
+    Raises:
+        IOError: Input config does not exist.
+
+    Returns:
+        view color space name (str) e.g. "Output - sRGB"
+    """
+
     config_path = Path(config_path)
 
     if not config_path.is_file():
@@ -199,13 +214,28 @@ def _get_display_view_colorspace_name(config_path, display, view):
               help="path where to write output json file",
               type=click.Path())
 @click.option("--display", required=True,
-              help="display",
+              help="display name",
               type=click.STRING)
 @click.option("--view", required=True,
-              help="view",
+              help="view name",
               type=click.STRING)
 def get_display_view_colorspace_name(in_path, out_path,
                                      display, view):
+    """Aggregate view colorspace name to file.
+
+    Python 2 wrapped console command
+
+    Args:
+        in_path (str): config file path string
+        out_path (str): temp json file path string
+        display (str): display name e.g. "ACES"
+        view (str): view name e.g. "sRGB"
+
+    Example of use:
+    > pyton.exe ./ocio_wrapper.py config \
+        get_display_view_colorspace_name --in_path=<path> \
+        --out_path=<path> --display=<display> --view=<view>
+    """
 
     json_path = Path(out_path)
 

From c8e66fd632323416a63c0c9d1bf248e516f2c2be Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 9 Aug 2023 20:34:05 +0300
Subject: [PATCH 08/49] resolve some conversations

---
 .../publish/validate_review_colorspace.py        | 16 ++++++++--------
 openpype/pipeline/colorspace.py                  |  4 +++-
 openpype/scripts/ocio_wrapper.py                 |  4 +++-
 3 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 67e29e0ee2..e493349946 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -4,6 +4,9 @@ from openpype.pipeline import PublishValidationError
 from openpype.pipeline.publish import RepairAction
 from openpype.hosts.houdini.api.action import SelectROPAction
 
+import os
+import hou
+
 
 class SetDefaultViewSpaceAction(RepairAction):
     label = "Set default view space"
@@ -32,8 +35,6 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
 
     @classmethod
     def get_invalid(cls, instance):
-        import hou  # noqa
-        import os
 
         rop_node = hou.node(instance.data["instance_node"])
         if os.getenv("OCIO") is None:
@@ -70,16 +71,15 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         used to set colorspace on opengl node to the default view.
         """
 
-        import hou
-        from openpype.pipeline.colorspace import get_display_view_colorspace_name #noqa
-        from openpype.hosts.houdini.api.lib import get_color_management_preferences #noqa
+        from openpype.pipeline.colorspace import get_display_view_colorspace_name  # noqa
+        from openpype.hosts.houdini.api.lib import get_color_management_preferences  # noqa
 
         rop_node = hou.node(instance.data["instance_node"])
 
-        data =  get_color_management_preferences()
+        data = get_color_management_preferences()
         config_path = data.get("config")
-        display =  data.get("display")
-        view =  data.get("view")
+        display = data.get("display")
+        view = data.get("view")
 
         cls.log.debug("Get default view colorspace name..")
 
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 22e8175a7e..d84424270c 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -590,6 +590,7 @@ def _get_imageio_settings(project_settings, host_name):
 
     return imageio_global, imageio_host
 
+
 def get_display_view_colorspace_name(config_path, display, view):
     """get view colorspace name for the given display and view.
 
@@ -608,10 +609,11 @@ def get_display_view_colorspace_name(config_path, display, view):
         return get_display_view_colorspace_subprocess(config_path,
                                                       display, view)
 
-    from openpype.scripts.ocio_wrapper import _get_display_view_colorspace_name #noqa
+    from openpype.scripts.ocio_wrapper import _get_display_view_colorspace_name  # noqa
 
     return _get_display_view_colorspace_name(config_path, display, view)
 
+
 def get_display_view_colorspace_subprocess(config_path, display, view):
     """get view colorspace name for the given display and view
         via subprocess.
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index f94faabe11..556568ce20 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -173,6 +173,7 @@ def _get_views_data(config_path):
 
     return data
 
+
 def _get_display_view_colorspace_name(config_path, display, view):
     """get view colorspace name for the given display and view.
 
@@ -199,6 +200,7 @@ def _get_display_view_colorspace_name(config_path, display, view):
 
     return colorspace
 
+
 @config.command(
     name="get_display_view_colorspace_name",
     help=(
@@ -223,7 +225,7 @@ def get_display_view_colorspace_name(in_path, out_path,
                                      display, view):
     """Aggregate view colorspace name to file.
 
-    Python 2 wrapped console command
+    Wrapper command for processes without acces to OpenColorIO
 
     Args:
         in_path (str): config file path string

From 877facc5b89317a0fc541d2f06a9d9611c9e1acb Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 9 Aug 2023 21:56:42 +0300
Subject: [PATCH 09/49] set default colorspace on creation if there's OCIO

---
 .../houdini/plugins/create/create_review.py   | 51 ++++++++++++++++++-
 1 file changed, 50 insertions(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/create/create_review.py b/openpype/hosts/houdini/plugins/create/create_review.py
index ab06b30c35..797116aaca 100644
--- a/openpype/hosts/houdini/plugins/create/create_review.py
+++ b/openpype/hosts/houdini/plugins/create/create_review.py
@@ -3,6 +3,9 @@
 from openpype.hosts.houdini.api import plugin
 from openpype.lib import EnumDef, BoolDef, NumberDef
 
+import os
+import hou
+
 
 class CreateReview(plugin.HoudiniCreator):
     """Review with OpenGL ROP"""
@@ -13,7 +16,6 @@ class CreateReview(plugin.HoudiniCreator):
     icon = "video-camera"
 
     def create(self, subset_name, instance_data, pre_create_data):
-        import hou
 
         instance_data.pop("active", None)
         instance_data.update({"node_type": "opengl"})
@@ -82,6 +84,10 @@ class CreateReview(plugin.HoudiniCreator):
 
         instance_node.setParms(parms)
 
+        # Set OCIO Colorspace to the default output colorspace
+        #  if there's OCIO
+        self.set_colorcorrect_to_default_view_space(instance_node)
+
         to_lock = ["id", "family"]
 
         self.lock_parameters(instance_node, to_lock)
@@ -123,3 +129,46 @@ class CreateReview(plugin.HoudiniCreator):
                       minimum=0.0001,
                       decimals=3)
         ]
+
+    def set_colorcorrect_to_default_view_space(self,
+                                               instance_node):
+        """Set ociocolorspace to the default output space."""
+
+        if os.getenv("OCIO") is None:
+            # No OCIO, skip setting ociocolorspace
+            return
+
+        # if there's OCIO then set Color Correction parameter
+        # to OpenColorIO
+        instance_node.setParms({"colorcorrect": 2})
+
+        self.log.debug("Get default view colorspace name..")
+
+        default_view_space = self.get_default_view_space()
+        instance_node.setParms(
+            {"ociocolorspace": default_view_space}
+        )
+
+        self.log.debug(
+            "'OCIO Colorspace' parm on '{}' has been set to "
+            "the default view color space '{}'"
+            .format(instance_node, default_view_space)
+        )
+
+        return default_view_space
+
+    def get_default_view_space(self):
+        """Get default view space for ociocolorspace parm."""
+
+        from openpype.pipeline.colorspace import get_display_view_colorspace_name  # noqa
+        from openpype.hosts.houdini.api.lib import get_color_management_preferences  # noqa
+
+        data = get_color_management_preferences()
+        config_path = data.get("config")
+        display = data.get("display")
+        view = data.get("view")
+
+        default_view_space = get_display_view_colorspace_name(config_path,
+                                                              display, view)
+
+        return default_view_space

From cff92425676acef62fc6b8489517e66286072925 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 9 Aug 2023 21:57:57 +0300
Subject: [PATCH 10/49] use .format instead of %s

---
 .../publish/validate_review_colorspace.py       | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index e493349946..5390b6b52f 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -49,17 +49,17 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
             # 'Color Correct' parm to be set to 'OpenColorIO'
             rop_node.setParms({"colorcorrect": 2})
             cls.log.debug(
-                "'Color Correct' parm on '%s' has been set to"
-                " 'OpenColorIO'", rop_node
+                "'Color Correct' parm on '{}' has been set to"
+                " 'OpenColorIO'".format(rop_node)
             )
 
         if rop_node.evalParm("ociocolorspace") not in \
                 hou.Color.ocio_spaces():
 
             cls.log.error(
-                "'OCIO Colorspace' value on '%s' is not valid, "
-                "select a valid option from the dropdown menu.",
-                rop_node
+                "'OCIO Colorspace' value on '{}' is not valid, "
+                "select a valid option from the dropdown menu."
+                .format(rop_node)
             )
             return rop_node
 
@@ -88,7 +88,8 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
 
         rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(
-            "'OCIO Colorspace' parm on '%s' has been set to "
-            "the default view color space '%s'",
-            default_view_space, rop_node
+            "'OCIO Colorspace' parm on '{}' has been set to "
+            "the default view color space '{}'"
+            .formate(rop_node, default_view_space)
+
         )

From 256ffee407e8718ca6099c5f7185d0ec88a0ace7 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Mon, 14 Aug 2023 15:54:14 +0300
Subject: [PATCH 11/49] resolve some conversations

---
 openpype/pipeline/colorspace.py  | 2 +-
 openpype/scripts/ocio_wrapper.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 392089237b..a0efb5e18c 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -604,7 +604,7 @@ def _get_imageio_settings(project_settings, host_name):
 
 
 def get_display_view_colorspace_name(config_path, display, view):
-    """get view colorspace name for the given display and view.
+    """Return colorspace name for the given display and view.
 
     Args:
         config_path (str): path string leading to config.ocio
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index 556568ce20..e491206ebb 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -247,7 +247,7 @@ def get_display_view_colorspace_name(in_path, out_path,
     with open(json_path, "w") as f:
         json.dump(out_data, f)
 
-    print(f"Viewer data are saved to '{json_path}'")
+    print(f"Display view colorspace saved to '{json_path}'")
 
 if __name__ == '__main__':
     main()

From 499b4623a30a0b4d7d900683d119394f27304540 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Thu, 24 Aug 2023 15:41:21 +0200
Subject: [PATCH 12/49] adding abstraction of publishing related functions

---
 openpype/pipeline/colorspace.py | 153 ++++++++++++++++++++++++++++++--
 1 file changed, 144 insertions(+), 9 deletions(-)

diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 731132911a..649d355f62 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -13,12 +13,17 @@ from openpype.lib import (
     Logger
 )
 from openpype.pipeline import Anatomy
+from openpype.lib.transcoding import VIDEO_EXTENSIONS, IMAGE_EXTENSIONS
+
 
 log = Logger.get_logger(__name__)
 
 
-class CashedData:
+class CachedData:
     remapping = None
+    allowed_exts = {
+        ext.lstrip(".") for ext in IMAGE_EXTENSIONS.union(VIDEO_EXTENSIONS)
+    }
 
 
 @contextlib.contextmanager
@@ -546,15 +551,15 @@ def get_remapped_colorspace_to_native(
         Union[str, None]: native colorspace name defined in remapping or None
     """
 
-    CashedData.remapping.setdefault(host_name, {})
-    if CashedData.remapping[host_name].get("to_native") is None:
+    CachedData.remapping.setdefault(host_name, {})
+    if CachedData.remapping[host_name].get("to_native") is None:
         remapping_rules = imageio_host_settings["remapping"]["rules"]
-        CashedData.remapping[host_name]["to_native"] = {
+        CachedData.remapping[host_name]["to_native"] = {
             rule["ocio_name"]: rule["host_native_name"]
             for rule in remapping_rules
         }
 
-    return CashedData.remapping[host_name]["to_native"].get(
+    return CachedData.remapping[host_name]["to_native"].get(
         ocio_colorspace_name)
 
 
@@ -572,15 +577,15 @@ def get_remapped_colorspace_from_native(
         Union[str, None]: Ocio colorspace name defined in remapping or None.
     """
 
-    CashedData.remapping.setdefault(host_name, {})
-    if CashedData.remapping[host_name].get("from_native") is None:
+    CachedData.remapping.setdefault(host_name, {})
+    if CachedData.remapping[host_name].get("from_native") is None:
         remapping_rules = imageio_host_settings["remapping"]["rules"]
-        CashedData.remapping[host_name]["from_native"] = {
+        CachedData.remapping[host_name]["from_native"] = {
             rule["host_native_name"]: rule["ocio_name"]
             for rule in remapping_rules
         }
 
-    return CashedData.remapping[host_name]["from_native"].get(
+    return CachedData.remapping[host_name]["from_native"].get(
         host_native_colorspace_name)
 
 
@@ -601,3 +606,133 @@ def _get_imageio_settings(project_settings, host_name):
     imageio_host = project_settings.get(host_name, {}).get("imageio", {})
 
     return imageio_global, imageio_host
+
+
+def get_colorspace_settings_from_publish_context(context_data):
+    """Returns solved settings for the host context.
+
+    Args:
+        context_data (publish.Context.data): publishing context data
+
+    Returns:
+        tuple | bool: config, file rules or None
+    """
+    if "imageioSettings" in context_data:
+        return context_data["imageioSettings"]
+
+    project_name = context_data["projectName"]
+    host_name = context_data["hostName"]
+    anatomy_data = context_data["anatomyData"]
+    project_settings_ = context_data["project_settings"]
+
+    config_data = get_imageio_config(
+        project_name, host_name,
+        project_settings=project_settings_,
+        anatomy_data=anatomy_data
+    )
+
+    # in case host color management is not enabled
+    if not config_data:
+        return None
+
+    file_rules = get_imageio_file_rules(
+        project_name, host_name,
+        project_settings=project_settings_
+    )
+
+    # caching settings for future instance processing
+    context_data["imageioSettings"] = (config_data, file_rules)
+
+    return config_data, file_rules
+
+
+def set_colorspace_data_to_representation(
+    representation, context_data,
+    colorspace=None,
+    colorspace_settings=None,
+    log=None
+):
+    """Sets colorspace data to representation.
+
+    Args:
+        representation (dict): publishing representation
+        context_data (publish.Context.data): publishing context data
+        config_data (dict): host resolved config data
+        file_rules (dict): host resolved file rules data
+        colorspace (str, optional): colorspace name. Defaults to None.
+        colorspace_settings (tuple[dict, dict], optional):
+            Settings for config_data and file_rules.
+            Defaults to None.
+        log (logging.Logger, optional): logger instance. Defaults to None.
+
+    Example:
+        ```
+        {
+            # for other publish plugins and loaders
+            "colorspace": "linear",
+            "config": {
+                # for future references in case need
+                "path": "/abs/path/to/config.ocio",
+                # for other plugins within remote publish cases
+                "template": "{project[root]}/path/to/config.ocio"
+            }
+        }
+        ```
+
+    """
+    log = log or Logger.get_logger(__name__)
+
+    file_ext = representation["ext"]
+
+    # check if `file_ext` in lower case is in CachedData.allowed_exts
+    if file_ext.lstrip(".").lower() not in CachedData.allowed_exts:
+        log.debug(
+            "Extension '{}' is not in allowed extensions.".format(file_ext)
+        )
+        return
+
+    if colorspace_settings is None:
+        colorspace_settings = get_colorspace_settings_from_publish_context(
+            context_data)
+
+    # in case host color management is not enabled
+    if not colorspace_settings:
+        log.warning("Host's colorspace management is disabled.")
+        return
+
+    # unpack colorspace settings
+    config_data, file_rules = colorspace_settings
+
+    if not config_data:
+        # warn in case no colorspace path was defined
+        log.warning("No colorspace management was defined")
+        return
+
+    log.debug("Config data is: `{}`".format(config_data))
+
+    project_name = context_data["projectName"]
+    host_name = context_data["hostName"]
+    project_settings = context_data["project_settings"]
+
+    # get one filename
+    filename = representation["files"]
+    if isinstance(filename, list):
+        filename = filename[0]
+
+    # get matching colorspace from rules
+    colorspace = colorspace or get_imageio_colorspace_from_filepath(
+        filename, host_name, project_name,
+        config_data=config_data,
+        file_rules=file_rules,
+        project_settings=project_settings
+    )
+
+    # infuse data to representation
+    if colorspace:
+        colorspace_data = {
+            "colorspace": colorspace,
+            "config": config_data
+        }
+
+        # update data key
+        representation["colorspaceData"] = colorspace_data

From 8260eb36bdd3e1d4cb8590f59853099b11dd75a8 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Thu, 24 Aug 2023 16:39:12 +0200
Subject: [PATCH 13/49] implementing abstarctions from colorspace

---
 openpype/pipeline/publish/publish_plugins.py | 106 ++-----------------
 1 file changed, 10 insertions(+), 96 deletions(-)

diff --git a/openpype/pipeline/publish/publish_plugins.py b/openpype/pipeline/publish/publish_plugins.py
index ba3be6397e..17ede069cb 100644
--- a/openpype/pipeline/publish/publish_plugins.py
+++ b/openpype/pipeline/publish/publish_plugins.py
@@ -1,6 +1,5 @@
 import inspect
 from abc import ABCMeta
-from pprint import pformat
 import pyblish.api
 from pyblish.plugin import MetaPlugin, ExplicitMetaPlugin
 from openpype.lib.transcoding import VIDEO_EXTENSIONS, IMAGE_EXTENSIONS
@@ -14,9 +13,8 @@ from .lib import (
 )
 
 from openpype.pipeline.colorspace import (
-    get_imageio_colorspace_from_filepath,
-    get_imageio_config,
-    get_imageio_file_rules
+    get_colorspace_settings_from_publish_context,
+    set_colorspace_data_to_representation
 )
 
 
@@ -306,12 +304,8 @@ class ColormanagedPyblishPluginMixin(object):
     matching colorspace from rules. Finally, it infuses this
     data into the representation.
     """
-    allowed_ext = set(
-        ext.lstrip(".") for ext in IMAGE_EXTENSIONS.union(VIDEO_EXTENSIONS)
-    )
 
-    @staticmethod
-    def get_colorspace_settings(context):
+    def get_colorspace_settings(self, context):
         """Returns solved settings for the host context.
 
         Args:
@@ -320,33 +314,7 @@ class ColormanagedPyblishPluginMixin(object):
         Returns:
             tuple | bool: config, file rules or None
         """
-        if "imageioSettings" in context.data:
-            return context.data["imageioSettings"]
-
-        project_name = context.data["projectName"]
-        host_name = context.data["hostName"]
-        anatomy_data = context.data["anatomyData"]
-        project_settings_ = context.data["project_settings"]
-
-        config_data = get_imageio_config(
-            project_name, host_name,
-            project_settings=project_settings_,
-            anatomy_data=anatomy_data
-        )
-
-        # in case host color management is not enabled
-        if not config_data:
-            return None
-
-        file_rules = get_imageio_file_rules(
-            project_name, host_name,
-            project_settings=project_settings_
-        )
-
-        # caching settings for future instance processing
-        context.data["imageioSettings"] = (config_data, file_rules)
-
-        return config_data, file_rules
+        return get_colorspace_settings_from_publish_context(context.data)
 
     def set_representation_colorspace(
         self, representation, context,
@@ -380,64 +348,10 @@ class ColormanagedPyblishPluginMixin(object):
             ```
 
         """
-        ext = representation["ext"]
-        # check extension
-        self.log.debug("__ ext: `{}`".format(ext))
-
-        # check if ext in lower case is in self.allowed_ext
-        if ext.lstrip(".").lower() not in self.allowed_ext:
-            self.log.debug(
-                "Extension '{}' is not in allowed extensions.".format(ext)
-            )
-            return
-
-        if colorspace_settings is None:
-            colorspace_settings = self.get_colorspace_settings(context)
-
-        # in case host color management is not enabled
-        if not colorspace_settings:
-            self.log.warning("Host's colorspace management is disabled.")
-            return
-
-        # unpack colorspace settings
-        config_data, file_rules = colorspace_settings
-
-        if not config_data:
-            # warn in case no colorspace path was defined
-            self.log.warning("No colorspace management was defined")
-            return
-
-        self.log.debug("Config data is: `{}`".format(config_data))
-
-        project_name = context.data["projectName"]
-        host_name = context.data["hostName"]
-        project_settings = context.data["project_settings"]
-
-        # get one filename
-        filename = representation["files"]
-        if isinstance(filename, list):
-            filename = filename[0]
-
-        self.log.debug("__ filename: `{}`".format(filename))
-
-        # get matching colorspace from rules
-        colorspace = colorspace or get_imageio_colorspace_from_filepath(
-            filename, host_name, project_name,
-            config_data=config_data,
-            file_rules=file_rules,
-            project_settings=project_settings
+        # using cached settings if available
+        set_colorspace_data_to_representation(
+            representation, context.data,
+            colorspace,
+            colorspace_settings,
+            log=self.log
         )
-        self.log.debug("__ colorspace: `{}`".format(colorspace))
-
-        # infuse data to representation
-        if colorspace:
-            colorspace_data = {
-                "colorspace": colorspace,
-                "config": config_data
-            }
-
-            # update data key
-            representation["colorspaceData"] = colorspace_data
-
-            self.log.debug("__ colorspace_data: `{}`".format(
-                pformat(colorspace_data)))

From 0f904cb32a0b7283cafd344f0b33fd7b95e6bda7 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Mon, 28 Aug 2023 17:23:19 +0200
Subject: [PATCH 14/49] adding default sequence frame data

---
 openpype/plugins/publish/collect_sequence_frame_data.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/openpype/plugins/publish/collect_sequence_frame_data.py b/openpype/plugins/publish/collect_sequence_frame_data.py
index c200b245e9..241e7b9011 100644
--- a/openpype/plugins/publish/collect_sequence_frame_data.py
+++ b/openpype/plugins/publish/collect_sequence_frame_data.py
@@ -50,4 +50,7 @@ class CollectSequenceFrameData(pyblish.api.InstancePlugin):
             return {
                 "frameStart": repres_frames[0],
                 "frameEnd": repres_frames[-1],
+                "handleStart": 0,
+                "handleEnd": 0,
+                "fps": instance.context.data["projectEntity"]["data"]["fps"]
             }

From 9ff5c071b3f7e0cc1bf7f002719565d192030e3a Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Tue, 29 Aug 2023 21:05:43 +0300
Subject: [PATCH 15/49] resolve conversations

---
 openpype/hosts/houdini/api/colorspace.py      | 18 +++++++++++++-
 .../houdini/plugins/create/create_review.py   | 24 +++----------------
 .../publish/validate_review_colorspace.py     | 19 ++++-----------
 openpype/pipeline/colorspace.py               |  6 ++---
 4 files changed, 28 insertions(+), 39 deletions(-)

diff --git a/openpype/hosts/houdini/api/colorspace.py b/openpype/hosts/houdini/api/colorspace.py
index 7047644225..5c3c605cd1 100644
--- a/openpype/hosts/houdini/api/colorspace.py
+++ b/openpype/hosts/houdini/api/colorspace.py
@@ -1,7 +1,7 @@
 import attr
 import hou
 from openpype.hosts.houdini.api.lib import get_color_management_preferences
-
+from openpype.pipeline.colorspace import get_display_view_colorspace_name
 
 @attr.s
 class LayerMetadata(object):
@@ -54,3 +54,19 @@ class ARenderProduct(object):
             )
         ]
         return colorspace_data
+
+
+def get_default_display_view_colorspace():
+    """Get default display view colorspace.
+
+    It's used for 'ociocolorspace' parm in OpneGL Node."""
+
+    data = get_color_management_preferences()
+    config_path = data.get("config")
+    display = data.get("display")
+    view = data.get("view")
+
+    default_view_space = get_display_view_colorspace_name(config_path,
+                                                            display,
+                                                            view)
+    return default_view_space
diff --git a/openpype/hosts/houdini/plugins/create/create_review.py b/openpype/hosts/houdini/plugins/create/create_review.py
index 797116aaca..75a92e5e77 100644
--- a/openpype/hosts/houdini/plugins/create/create_review.py
+++ b/openpype/hosts/houdini/plugins/create/create_review.py
@@ -2,6 +2,7 @@
 """Creator plugin for creating openGL reviews."""
 from openpype.hosts.houdini.api import plugin
 from openpype.lib import EnumDef, BoolDef, NumberDef
+from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace
 
 import os
 import hou
@@ -142,9 +143,8 @@ class CreateReview(plugin.HoudiniCreator):
         # to OpenColorIO
         instance_node.setParms({"colorcorrect": 2})
 
-        self.log.debug("Get default view colorspace name..")
-
-        default_view_space = self.get_default_view_space()
+        # Get default view space for ociocolorspace parm.
+        default_view_space = get_default_display_view_colorspace()
         instance_node.setParms(
             {"ociocolorspace": default_view_space}
         )
@@ -154,21 +154,3 @@ class CreateReview(plugin.HoudiniCreator):
             "the default view color space '{}'"
             .format(instance_node, default_view_space)
         )
-
-        return default_view_space
-
-    def get_default_view_space(self):
-        """Get default view space for ociocolorspace parm."""
-
-        from openpype.pipeline.colorspace import get_display_view_colorspace_name  # noqa
-        from openpype.hosts.houdini.api.lib import get_color_management_preferences  # noqa
-
-        data = get_color_management_preferences()
-        config_path = data.get("config")
-        display = data.get("display")
-        view = data.get("view")
-
-        default_view_space = get_display_view_colorspace_name(config_path,
-                                                              display, view)
-
-        return default_view_space
diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 5390b6b52f..09e4a489d2 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -3,6 +3,7 @@ import pyblish.api
 from openpype.pipeline import PublishValidationError
 from openpype.pipeline.publish import RepairAction
 from openpype.hosts.houdini.api.action import SelectROPAction
+from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace
 
 import os
 import hou
@@ -38,7 +39,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
 
         rop_node = hou.node(instance.data["instance_node"])
         if os.getenv("OCIO") is None:
-            cls.log.warning(
+            cls.log.debug(
                 "Default Houdini colorspace is used, "
                 " skipping check.."
             )
@@ -71,25 +72,15 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
         used to set colorspace on opengl node to the default view.
         """
 
-        from openpype.pipeline.colorspace import get_display_view_colorspace_name  # noqa
-        from openpype.hosts.houdini.api.lib import get_color_management_preferences  # noqa
-
         rop_node = hou.node(instance.data["instance_node"])
 
-        data = get_color_management_preferences()
-        config_path = data.get("config")
-        display = data.get("display")
-        view = data.get("view")
-
-        cls.log.debug("Get default view colorspace name..")
-
-        default_view_space = get_display_view_colorspace_name(config_path,
-                                                              display, view)
+        # Get default view colorspace name
+        default_view_space = get_default_display_view_colorspace()
 
         rop_node.setParms({"ociocolorspace": default_view_space})
         cls.log.debug(
             "'OCIO Colorspace' parm on '{}' has been set to "
             "the default view color space '{}'"
-            .formate(rop_node, default_view_space)
+            .format(rop_node, default_view_space)
 
         )
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index a0efb5e18c..37974f4a0b 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -658,6 +658,6 @@ def get_display_view_colorspace_subprocess(config_path, display, view):
 
         run_openpype_process(*args, **process_kwargs)
 
-        # return all colorspaces
-        return_json_data = open(tmp_json_path).read()
-        return json.loads(return_json_data)
+        # return default view colorspace name
+        with open(tmp_json_path, "r") as f:
+            return json.load(f)

From c799ae42eab8be32051ee373bdb71176002a20b8 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Tue, 29 Aug 2023 21:06:36 +0300
Subject: [PATCH 16/49] add spaces

---
 openpype/hosts/houdini/api/colorspace.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/openpype/hosts/houdini/api/colorspace.py b/openpype/hosts/houdini/api/colorspace.py
index 5c3c605cd1..2662a968e2 100644
--- a/openpype/hosts/houdini/api/colorspace.py
+++ b/openpype/hosts/houdini/api/colorspace.py
@@ -67,6 +67,6 @@ def get_default_display_view_colorspace():
     view = data.get("view")
 
     default_view_space = get_display_view_colorspace_name(config_path,
-                                                            display,
-                                                            view)
+                                                          display,
+                                                          view)
     return default_view_space

From 75673151a6374591067c5e8f356ee3bb93706961 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Tue, 29 Aug 2023 21:18:27 +0300
Subject: [PATCH 17/49] resolve hound conversations

---
 openpype/hosts/houdini/plugins/create/create_review.py          | 2 +-
 .../hosts/houdini/plugins/publish/validate_review_colorspace.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/create/create_review.py b/openpype/hosts/houdini/plugins/create/create_review.py
index 75a92e5e77..c087c54f6c 100644
--- a/openpype/hosts/houdini/plugins/create/create_review.py
+++ b/openpype/hosts/houdini/plugins/create/create_review.py
@@ -2,7 +2,7 @@
 """Creator plugin for creating openGL reviews."""
 from openpype.hosts.houdini.api import plugin
 from openpype.lib import EnumDef, BoolDef, NumberDef
-from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace
+from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
 import os
 import hou
diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 09e4a489d2..2c7420bf48 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -3,7 +3,7 @@ import pyblish.api
 from openpype.pipeline import PublishValidationError
 from openpype.pipeline.publish import RepairAction
 from openpype.hosts.houdini.api.action import SelectROPAction
-from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace
+from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
 import os
 import hou

From 668bcb2d40d5708fc40c1e1a8465e36c94ffdef8 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 30 Aug 2023 22:28:43 +0300
Subject: [PATCH 18/49] BigRoy's Comments

---
 openpype/hosts/houdini/api/colorspace.py | 19 +++++++------------
 openpype/pipeline/colorspace.py          |  2 +-
 2 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/openpype/hosts/houdini/api/colorspace.py b/openpype/hosts/houdini/api/colorspace.py
index 2662a968e2..fb0a724eb9 100644
--- a/openpype/hosts/houdini/api/colorspace.py
+++ b/openpype/hosts/houdini/api/colorspace.py
@@ -57,16 +57,11 @@ class ARenderProduct(object):
 
 
 def get_default_display_view_colorspace():
-    """Get default display view colorspace.
+    """Get default display view colorspace name. """
 
-    It's used for 'ociocolorspace' parm in OpneGL Node."""
-
-    data = get_color_management_preferences()
-    config_path = data.get("config")
-    display = data.get("display")
-    view = data.get("view")
-
-    default_view_space = get_display_view_colorspace_name(config_path,
-                                                          display,
-                                                          view)
-    return default_view_space
+    prefs = get_color_management_preferences()
+    return get_display_view_colorspace_name(
+        config_path=prefs["config"],
+        display=prefs["display"],
+        view=prefs["view"]
+    )
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 37974f4a0b..3bb258e8f2 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -650,7 +650,7 @@ def get_display_view_colorspace_subprocess(config_path, display, view):
             "--view", view
 
         ]
-        log.info("Executing: {}".format(" ".join(args)))
+        log.debug("Executing: {}".format(" ".join(args)))
 
         process_kwargs = {
             "logger": log

From 41babcaa85880b957a6409bf29d02da2291f91d2 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 30 Aug 2023 22:43:52 +0300
Subject: [PATCH 19/49] update doc strings

---
 openpype/hosts/houdini/api/colorspace.py | 4 +++-
 openpype/pipeline/colorspace.py          | 4 ++--
 openpype/scripts/ocio_wrapper.py         | 2 +-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/openpype/hosts/houdini/api/colorspace.py b/openpype/hosts/houdini/api/colorspace.py
index fb0a724eb9..b1a4d5dcd5 100644
--- a/openpype/hosts/houdini/api/colorspace.py
+++ b/openpype/hosts/houdini/api/colorspace.py
@@ -57,7 +57,9 @@ class ARenderProduct(object):
 
 
 def get_default_display_view_colorspace():
-    """Get default display view colorspace name. """
+    """Returns the colorspace attribute of the default (display, view) pair.
+
+    """
 
     prefs = get_color_management_preferences()
     return get_display_view_colorspace_name(
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 3bb258e8f2..3dd33d0425 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -604,7 +604,7 @@ def _get_imageio_settings(project_settings, host_name):
 
 
 def get_display_view_colorspace_name(config_path, display, view):
-    """Return colorspace name for the given display and view.
+    """Returns the colorspace attribute of the (display, view) pair.
 
     Args:
         config_path (str): path string leading to config.ocio
@@ -627,7 +627,7 @@ def get_display_view_colorspace_name(config_path, display, view):
 
 
 def get_display_view_colorspace_subprocess(config_path, display, view):
-    """get view colorspace name for the given display and view
+    """Returns the colorspace attribute of the (display, view) pair
         via subprocess.
 
     Args:
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index e491206ebb..cae6e6975b 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -175,7 +175,7 @@ def _get_views_data(config_path):
 
 
 def _get_display_view_colorspace_name(config_path, display, view):
-    """get view colorspace name for the given display and view.
+    """Returns the colorspace attribute of the (display, view) pair.
 
     Args:
         config_path (str): path string leading to config.ocio

From 6c5039f7075ce59b2911a680d41ffae5a59dca09 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Wed, 30 Aug 2023 22:46:07 +0300
Subject: [PATCH 20/49] BigRoy's Comment

---
 openpype/hosts/houdini/api/colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/api/colorspace.py b/openpype/hosts/houdini/api/colorspace.py
index b1a4d5dcd5..cc40b9df1c 100644
--- a/openpype/hosts/houdini/api/colorspace.py
+++ b/openpype/hosts/houdini/api/colorspace.py
@@ -59,7 +59,7 @@ class ARenderProduct(object):
 def get_default_display_view_colorspace():
     """Returns the colorspace attribute of the default (display, view) pair.
 
-    """
+    It's used for 'ociocolorspace' parm in OpenGL Node."""
 
     prefs = get_color_management_preferences()
     return get_display_view_colorspace_name(

From 1a3c8ad77252ffa9217c6af98ba27dcdd56c366a Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Thu, 31 Aug 2023 17:34:42 +0300
Subject: [PATCH 21/49] make validateReviewColorspace optional

---
 .../plugins/publish/validate_review_colorspace.py  | 10 ++++++++--
 .../defaults/project_settings/houdini.json         |  5 +++++
 .../schemas/schema_houdini_publish.json            |  6 +++++-
 .../houdini/server/settings/publish_plugins.py     | 14 +++++++++++---
 server_addon/houdini/server/version.py             |  2 +-
 5 files changed, 30 insertions(+), 7 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 2c7420bf48..f680f62142 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -1,6 +1,9 @@
 # -*- coding: utf-8 -*-
 import pyblish.api
-from openpype.pipeline import PublishValidationError
+from openpype.pipeline import (
+    PublishValidationError,
+    OptionalPyblishPluginMixin
+)
 from openpype.pipeline.publish import RepairAction
 from openpype.hosts.houdini.api.action import SelectROPAction
 from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
@@ -14,7 +17,8 @@ class SetDefaultViewSpaceAction(RepairAction):
     icon = "mdi.monitor"
 
 
-class ValidateReviewColorspace(pyblish.api.InstancePlugin):
+class ValidateReviewColorspace(pyblish.api.InstancePlugin,
+                               OptionalPyblishPluginMixin):
     """Validate Review Colorspace parameters.
 
     It checks if 'OCIO Colorspace' parameter was set to valid value.
@@ -26,6 +30,8 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin):
     label = "Validate Review Colorspace"
     actions = [SetDefaultViewSpaceAction, SelectROPAction]
 
+    optional = True
+
     def process(self, instance):
         invalid = self.get_invalid(instance)
         if invalid:
diff --git a/openpype/settings/defaults/project_settings/houdini.json b/openpype/settings/defaults/project_settings/houdini.json
index 9d047c28bd..93d5c50d5e 100644
--- a/openpype/settings/defaults/project_settings/houdini.json
+++ b/openpype/settings/defaults/project_settings/houdini.json
@@ -93,6 +93,11 @@
                 "$JOB"
             ]
         },
+        "ValidateReviewColorspace": {
+            "enabled": true,
+            "optional": true,
+            "active": true
+        },
         "ValidateContainers": {
             "enabled": true,
             "optional": true,
diff --git a/openpype/settings/entities/schemas/projects_schema/schemas/schema_houdini_publish.json b/openpype/settings/entities/schemas/projects_schema/schemas/schema_houdini_publish.json
index aa6eaf5164..b57089007e 100644
--- a/openpype/settings/entities/schemas/projects_schema/schemas/schema_houdini_publish.json
+++ b/openpype/settings/entities/schemas/projects_schema/schemas/schema_houdini_publish.json
@@ -40,6 +40,10 @@
       "type": "schema_template",
       "name": "template_publish_plugin",
       "template_data": [
+          {
+              "key": "ValidateReviewColorspace",
+              "label": "Validate Review Colorspace"
+          },
           {
               "key": "ValidateContainers",
               "label": "ValidateContainers"
@@ -47,4 +51,4 @@
       ]
     }
   ]
-}
\ No newline at end of file
+}
diff --git a/server_addon/houdini/server/settings/publish_plugins.py b/server_addon/houdini/server/settings/publish_plugins.py
index 7d35d7e634..4534d8d0d9 100644
--- a/server_addon/houdini/server/settings/publish_plugins.py
+++ b/server_addon/houdini/server/settings/publish_plugins.py
@@ -120,7 +120,7 @@ class ValidateWorkfilePathsModel(BaseSettingsModel):
     )
 
 
-class ValidateContainersModel(BaseSettingsModel):
+class BasicValidateModel(BaseSettingsModel):
     enabled: bool = Field(title="Enabled")
     optional: bool = Field(title="Optional")
     active: bool = Field(title="Active")
@@ -130,8 +130,11 @@ class PublishPluginsModel(BaseSettingsModel):
     ValidateWorkfilePaths: ValidateWorkfilePathsModel = Field(
         default_factory=ValidateWorkfilePathsModel,
         title="Validate workfile paths settings.")
-    ValidateContainers: ValidateContainersModel = Field(
-        default_factory=ValidateContainersModel,
+    ValidateReviewColorspace: BasicValidateModel = Field(
+        default_factory=BasicValidateModel,
+        title="Validate Review Colorspace.")
+    ValidateContainers: BasicValidateModel = Field(
+        default_factory=BasicValidateModel,
         title="Validate Latest Containers.")
 
 
@@ -148,6 +151,11 @@ DEFAULT_HOUDINI_PUBLISH_SETTINGS = {
             "$JOB"
         ]
     },
+    "ValidateReviewColorspace": {
+        "enabled": True,
+        "optional": True,
+        "active": True
+    },
     "ValidateContainers": {
         "enabled": True,
         "optional": True,
diff --git a/server_addon/houdini/server/version.py b/server_addon/houdini/server/version.py
index 485f44ac21..b3f4756216 100644
--- a/server_addon/houdini/server/version.py
+++ b/server_addon/houdini/server/version.py
@@ -1 +1 @@
-__version__ = "0.1.1"
+__version__ = "0.1.2"

From 1a936c155985e804f0ddc01e14a4573c56bbb8e7 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Thu, 31 Aug 2023 22:42:13 +0300
Subject: [PATCH 22/49] BigRoy's comments

---
 .../houdini/plugins/publish/validate_review_colorspace.py   | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index f680f62142..3f5e5bc354 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -13,7 +13,7 @@ import hou
 
 
 class SetDefaultViewSpaceAction(RepairAction):
-    label = "Set default view space"
+    label = "Set default view colorspace"
     icon = "mdi.monitor"
 
 
@@ -33,6 +33,10 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
     optional = True
 
     def process(self, instance):
+
+        if not self.is_active(instance.data):
+            return
+
         invalid = self.get_invalid(instance)
         if invalid:
             raise PublishValidationError(

From d4393bad6b466438a839786903152fc36e293cae Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Thu, 31 Aug 2023 23:18:52 +0200
Subject: [PATCH 23/49] changing signature since we only need context.data for
 input.

---
 openpype/pipeline/colorspace.py              | 39 ++++++--------------
 openpype/pipeline/publish/publish_plugins.py |  8 +---
 2 files changed, 13 insertions(+), 34 deletions(-)

diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 649d355f62..b7728936b0 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -617,7 +617,7 @@ def get_colorspace_settings_from_publish_context(context_data):
     Returns:
         tuple | bool: config, file rules or None
     """
-    if "imageioSettings" in context_data:
+    if "imageioSettings" in context_data and context_data["imageioSettings"]:
         return context_data["imageioSettings"]
 
     project_name = context_data["projectName"]
@@ -631,14 +631,13 @@ def get_colorspace_settings_from_publish_context(context_data):
         anatomy_data=anatomy_data
     )
 
-    # in case host color management is not enabled
-    if not config_data:
-        return None
-
-    file_rules = get_imageio_file_rules(
-        project_name, host_name,
-        project_settings=project_settings_
-    )
+    # caching invalid state, so it's not recalculated all the time
+    file_rules = None
+    if config_data:
+        file_rules = get_imageio_file_rules(
+            project_name, host_name,
+            project_settings=project_settings_
+        )
 
     # caching settings for future instance processing
     context_data["imageioSettings"] = (config_data, file_rules)
@@ -649,7 +648,6 @@ def get_colorspace_settings_from_publish_context(context_data):
 def set_colorspace_data_to_representation(
     representation, context_data,
     colorspace=None,
-    colorspace_settings=None,
     log=None
 ):
     """Sets colorspace data to representation.
@@ -657,12 +655,7 @@ def set_colorspace_data_to_representation(
     Args:
         representation (dict): publishing representation
         context_data (publish.Context.data): publishing context data
-        config_data (dict): host resolved config data
-        file_rules (dict): host resolved file rules data
         colorspace (str, optional): colorspace name. Defaults to None.
-        colorspace_settings (tuple[dict, dict], optional):
-            Settings for config_data and file_rules.
-            Defaults to None.
         log (logging.Logger, optional): logger instance. Defaults to None.
 
     Example:
@@ -691,21 +684,13 @@ def set_colorspace_data_to_representation(
         )
         return
 
-    if colorspace_settings is None:
-        colorspace_settings = get_colorspace_settings_from_publish_context(
-            context_data)
+    # get colorspace settings
+    config_data, file_rules = get_colorspace_settings_from_publish_context(
+        context_data)
 
     # in case host color management is not enabled
-    if not colorspace_settings:
-        log.warning("Host's colorspace management is disabled.")
-        return
-
-    # unpack colorspace settings
-    config_data, file_rules = colorspace_settings
-
     if not config_data:
-        # warn in case no colorspace path was defined
-        log.warning("No colorspace management was defined")
+        log.warning("Host's colorspace management is disabled.")
         return
 
     log.debug("Config data is: `{}`".format(config_data))
diff --git a/openpype/pipeline/publish/publish_plugins.py b/openpype/pipeline/publish/publish_plugins.py
index 17ede069cb..ae6cbc42d1 100644
--- a/openpype/pipeline/publish/publish_plugins.py
+++ b/openpype/pipeline/publish/publish_plugins.py
@@ -319,19 +319,13 @@ class ColormanagedPyblishPluginMixin(object):
     def set_representation_colorspace(
         self, representation, context,
         colorspace=None,
-        colorspace_settings=None
     ):
         """Sets colorspace data to representation.
 
         Args:
             representation (dict): publishing representation
             context (publish.Context): publishing context
-            config_data (dict): host resolved config data
-            file_rules (dict): host resolved file rules data
             colorspace (str, optional): colorspace name. Defaults to None.
-            colorspace_settings (tuple[dict, dict], optional):
-                Settings for config_data and file_rules.
-                Defaults to None.
 
         Example:
             ```
@@ -348,10 +342,10 @@ class ColormanagedPyblishPluginMixin(object):
             ```
 
         """
+
         # using cached settings if available
         set_colorspace_data_to_representation(
             representation, context.data,
             colorspace,
-            colorspace_settings,
             log=self.log
         )

From 1f057525dd6dd2f3d03fbb97f68ee8504ce9aae5 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Thu, 31 Aug 2023 23:28:12 +0200
Subject: [PATCH 24/49] testing: fixing zip file ID

---
 tests/unit/openpype/pipeline/test_colorspace.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/unit/openpype/pipeline/test_colorspace.py b/tests/unit/openpype/pipeline/test_colorspace.py
index c22acee2d4..ac35a28303 100644
--- a/tests/unit/openpype/pipeline/test_colorspace.py
+++ b/tests/unit/openpype/pipeline/test_colorspace.py
@@ -28,10 +28,9 @@ class TestPipelineColorspace(TestPipeline):
         cd to OpenPype repo root dir
         poetry run python ./start.py runtests ../tests/unit/openpype/pipeline
     """
-
     TEST_FILES = [
         (
-            "1Lf-mFxev7xiwZCWfImlRcw7Fj8XgNQMh",
+            "1csqimz8bbNcNgxtEXklLz6GRv91D3KgA",
             "test_pipeline_colorspace.zip",
             ""
         )

From 869c6277ff1d89599086169926dfb48673c088c0 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Fri, 1 Sep 2023 21:03:09 +0300
Subject: [PATCH 25/49] BigRoy's comment

---
 .../publish/validate_review_colorspace.py     | 40 ++++++++++++-------
 1 file changed, 26 insertions(+), 14 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 3f5e5bc354..e5d4756556 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -37,15 +37,15 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
         if not self.is_active(instance.data):
             return
 
-        invalid = self.get_invalid(instance)
-        if invalid:
+        invalid_nodes, message = self.get_invalid_with_message(instance)
+        if invalid_nodes:
             raise PublishValidationError(
-                ("'OCIO Colorspace' parameter is not valid."),
+                message,
                 title=self.label
             )
 
     @classmethod
-    def get_invalid(cls, instance):
+    def get_invalid_with_message(cls, instance):
 
         rop_node = hou.node(instance.data["instance_node"])
         if os.getenv("OCIO") is None:
@@ -53,26 +53,31 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
                 "Default Houdini colorspace is used, "
                 " skipping check.."
             )
-            return
+            return None, None
 
         if rop_node.evalParm("colorcorrect") != 2:
             # any colorspace settings other than default requires
             # 'Color Correct' parm to be set to 'OpenColorIO'
-            rop_node.setParms({"colorcorrect": 2})
-            cls.log.debug(
-                "'Color Correct' parm on '{}' has been set to"
-                " 'OpenColorIO'".format(rop_node)
+            error = (
+                "'Color Correction' parm on '{}' ROP must be set to"
+                " 'OpenColorIO'".format(rop_node.path())
             )
+            return rop_node , error
 
         if rop_node.evalParm("ociocolorspace") not in \
                 hou.Color.ocio_spaces():
 
-            cls.log.error(
-                "'OCIO Colorspace' value on '{}' is not valid, "
-                "select a valid option from the dropdown menu."
-                .format(rop_node)
+            error = (
+                "Invalid value: Colorspace name doesn't exist.\n"
+                "Check 'OCIO Colorspace' parameter on '{}' ROP"
+                .format(rop_node.path())
             )
-            return rop_node
+            return rop_node, error
+
+    @classmethod
+    def get_invalid(cls, instance):
+        nodes, _ = cls.get_invalid_with_message(instance)
+        return nodes
 
     @classmethod
     def repair(cls, instance):
@@ -84,6 +89,13 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
 
         rop_node = hou.node(instance.data["instance_node"])
 
+        if rop_node.evalParm("colorcorrect") != 2:
+            rop_node.setParms({"colorcorrect": 2})
+            cls.log.debug(
+                "'Color Correction' parm on '{}' has been set to"
+                " 'OpenColorIO'".format(rop_node.path())
+            )
+
         # Get default view colorspace name
         default_view_space = get_default_display_view_colorspace()
 

From 087aca6d8b9377c054e623bc2f12f6b418a69aa0 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Fri, 1 Sep 2023 21:04:59 +0300
Subject: [PATCH 26/49] resolve hound

---
 .../hosts/houdini/plugins/publish/validate_review_colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index e5d4756556..47370678d0 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -62,7 +62,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
                 "'Color Correction' parm on '{}' ROP must be set to"
                 " 'OpenColorIO'".format(rop_node.path())
             )
-            return rop_node , error
+            return rop_node, error
 
         if rop_node.evalParm("ociocolorspace") not in \
                 hou.Color.ocio_spaces():

From c2b948d35fea1f0cf2f7146bb0085fc3bd2dfeef Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Mon, 4 Sep 2023 11:09:47 +0300
Subject: [PATCH 27/49] BigRoy's comments

---
 .../publish/validate_review_colorspace.py     | 27 ++++---------------
 openpype/pipeline/colorspace.py               |  7 +----
 openpype/scripts/ocio_wrapper.py              | 11 ++++----
 3 files changed, 11 insertions(+), 34 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 47370678d0..d457a295c5 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -37,47 +37,30 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
         if not self.is_active(instance.data):
             return
 
-        invalid_nodes, message = self.get_invalid_with_message(instance)
-        if invalid_nodes:
-            raise PublishValidationError(
-                message,
-                title=self.label
-            )
-
-    @classmethod
-    def get_invalid_with_message(cls, instance):
-
-        rop_node = hou.node(instance.data["instance_node"])
         if os.getenv("OCIO") is None:
-            cls.log.debug(
+            self.log.debug(
                 "Default Houdini colorspace is used, "
                 " skipping check.."
             )
-            return None, None
+            return
 
+        rop_node = hou.node(instance.data["instance_node"])
         if rop_node.evalParm("colorcorrect") != 2:
             # any colorspace settings other than default requires
             # 'Color Correct' parm to be set to 'OpenColorIO'
-            error = (
+            raise PublishValidationError(
                 "'Color Correction' parm on '{}' ROP must be set to"
                 " 'OpenColorIO'".format(rop_node.path())
             )
-            return rop_node, error
 
         if rop_node.evalParm("ociocolorspace") not in \
                 hou.Color.ocio_spaces():
 
-            error = (
+            raise PublishValidationError(
                 "Invalid value: Colorspace name doesn't exist.\n"
                 "Check 'OCIO Colorspace' parameter on '{}' ROP"
                 .format(rop_node.path())
             )
-            return rop_node, error
-
-    @classmethod
-    def get_invalid(cls, instance):
-        nodes, _ = cls.get_invalid_with_message(instance)
-        return nodes
 
     @classmethod
     def repair(cls, instance):
diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index 3dd33d0425..e167e18cfb 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -648,15 +648,10 @@ def get_display_view_colorspace_subprocess(config_path, display, view):
             "--out_path", tmp_json_path,
             "--display", display,
             "--view", view
-
         ]
         log.debug("Executing: {}".format(" ".join(args)))
 
-        process_kwargs = {
-            "logger": log
-        }
-
-        run_openpype_process(*args, **process_kwargs)
+        run_openpype_process(*args, logger=log)
 
         # return default view colorspace name
         with open(tmp_json_path, "r") as f:
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index cae6e6975b..2c11bb7eeb 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -225,7 +225,7 @@ def get_display_view_colorspace_name(in_path, out_path,
                                      display, view):
     """Aggregate view colorspace name to file.
 
-    Wrapper command for processes without acces to OpenColorIO
+    Wrapper command for processes without access to OpenColorIO
 
     Args:
         in_path (str): config file path string
@@ -239,15 +239,14 @@ def get_display_view_colorspace_name(in_path, out_path,
         --out_path=<path> --display=<display> --view=<view>
     """
 
-    json_path = Path(out_path)
-
     out_data = _get_display_view_colorspace_name(in_path,
-                                                 display, view)
+                                                 display,
+                                                 view)
 
-    with open(json_path, "w") as f:
+    with open(out_path, "w") as f:
         json.dump(out_data, f)
 
-    print(f"Display view colorspace saved to '{json_path}'")
+    print(f"Display view colorspace saved to '{out_path}'")
 
 if __name__ == '__main__':
     main()

From 4ed278c0c885e22fdcf97c9a1b242428fee3ff05 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Mon, 4 Sep 2023 12:18:03 +0300
Subject: [PATCH 28/49] BigRoy's comment

---
 .../houdini/plugins/publish/validate_review_colorspace.py      | 3 +--
 openpype/scripts/ocio_wrapper.py                               | 2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index d457a295c5..545d7b16b1 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -83,9 +83,8 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
         default_view_space = get_default_display_view_colorspace()
 
         rop_node.setParms({"ociocolorspace": default_view_space})
-        cls.log.debug(
+        cls.log.info(
             "'OCIO Colorspace' parm on '{}' has been set to "
             "the default view color space '{}'"
             .format(rop_node, default_view_space)
-
         )
diff --git a/openpype/scripts/ocio_wrapper.py b/openpype/scripts/ocio_wrapper.py
index 2c11bb7eeb..40553d30f2 100644
--- a/openpype/scripts/ocio_wrapper.py
+++ b/openpype/scripts/ocio_wrapper.py
@@ -195,7 +195,7 @@ def _get_display_view_colorspace_name(config_path, display, view):
     if not config_path.is_file():
         raise IOError("Input path should be `config.ocio` file")
 
-    config = ocio.Config().CreateFromFile(str(config_path))
+    config = ocio.Config.CreateFromFile(str(config_path))
     colorspace = config.getDisplayViewColorSpaceName(display, view)
 
     return colorspace

From df2466b714f86336fad823bd9699aced271e5457 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Mon, 4 Sep 2023 12:58:31 +0200
Subject: [PATCH 29/49] default fps for sequence from asset instead of project

---
 openpype/plugins/publish/collect_sequence_frame_data.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/plugins/publish/collect_sequence_frame_data.py b/openpype/plugins/publish/collect_sequence_frame_data.py
index 241e7b9011..6c2bfbf358 100644
--- a/openpype/plugins/publish/collect_sequence_frame_data.py
+++ b/openpype/plugins/publish/collect_sequence_frame_data.py
@@ -52,5 +52,5 @@ class CollectSequenceFrameData(pyblish.api.InstancePlugin):
                 "frameEnd": repres_frames[-1],
                 "handleStart": 0,
                 "handleEnd": 0,
-                "fps": instance.context.data["projectEntity"]["data"]["fps"]
+                "fps": instance.context.data["assetEntity"]["data"]["fps"]
             }

From 378ec74136eba325cba019348cdc69cb90c7d6b6 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Mon, 4 Sep 2023 12:59:06 +0200
Subject: [PATCH 30/49] validator for frame range should include Plate family

---
 .../traypublisher/plugins/publish/validate_frame_ranges.py      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/traypublisher/plugins/publish/validate_frame_ranges.py b/openpype/hosts/traypublisher/plugins/publish/validate_frame_ranges.py
index b962ea464a..09de2d8db2 100644
--- a/openpype/hosts/traypublisher/plugins/publish/validate_frame_ranges.py
+++ b/openpype/hosts/traypublisher/plugins/publish/validate_frame_ranges.py
@@ -15,7 +15,7 @@ class ValidateFrameRange(OptionalPyblishPluginMixin,
 
     label = "Validate Frame Range"
     hosts = ["traypublisher"]
-    families = ["render"]
+    families = ["render", "plate"]
     order = ValidateContentsOrder
 
     optional = True

From ab019e312441af63b447f74d8e6119bce0ebb06e Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Mon, 4 Sep 2023 13:01:29 +0200
Subject: [PATCH 31/49] name of plugin should be more explicit

---
 ....py => collect_missing_frame_range_asset_entity.py} | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)
 rename openpype/hosts/traypublisher/plugins/publish/{collect_frame_range_asset_entity.py => collect_missing_frame_range_asset_entity.py} (83%)

diff --git a/openpype/hosts/traypublisher/plugins/publish/collect_frame_range_asset_entity.py b/openpype/hosts/traypublisher/plugins/publish/collect_missing_frame_range_asset_entity.py
similarity index 83%
rename from openpype/hosts/traypublisher/plugins/publish/collect_frame_range_asset_entity.py
rename to openpype/hosts/traypublisher/plugins/publish/collect_missing_frame_range_asset_entity.py
index c18e10e438..72379ea4e1 100644
--- a/openpype/hosts/traypublisher/plugins/publish/collect_frame_range_asset_entity.py
+++ b/openpype/hosts/traypublisher/plugins/publish/collect_missing_frame_range_asset_entity.py
@@ -2,16 +2,18 @@ import pyblish.api
 from openpype.pipeline import OptionalPyblishPluginMixin
 
 
-class CollectFrameDataFromAssetEntity(pyblish.api.InstancePlugin,
-                                      OptionalPyblishPluginMixin):
-    """Collect Frame Range data From Asset Entity
+class CollectMissingFrameDataFromAssetEntity(
+    pyblish.api.InstancePlugin,
+    OptionalPyblishPluginMixin
+):
+    """Collect Missing Frame Range data From Asset Entity
 
     Frame range data will only be collected if the keys
     are not yet collected for the instance.
     """
 
     order = pyblish.api.CollectorOrder + 0.491
-    label = "Collect Frame Data From Asset Entity"
+    label = "Collect Missing Frame Data From Asset Entity"
     families = ["plate", "pointcache",
                 "vdbcache", "online",
                 "render"]

From 90bfd0a79a8ef5fcee474340594949e44f657ab2 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 4 Sep 2023 13:26:36 +0200
Subject: [PATCH 32/49] Yeti Cache: Include viewport preview settings

---
 .../plugins/publish/collect_yeti_cache.py     | 36 ++++++++++++-------
 1 file changed, 23 insertions(+), 13 deletions(-)

diff --git a/openpype/hosts/maya/plugins/publish/collect_yeti_cache.py b/openpype/hosts/maya/plugins/publish/collect_yeti_cache.py
index e6b5ca4260..4dcda29050 100644
--- a/openpype/hosts/maya/plugins/publish/collect_yeti_cache.py
+++ b/openpype/hosts/maya/plugins/publish/collect_yeti_cache.py
@@ -4,12 +4,23 @@ import pyblish.api
 
 from openpype.hosts.maya.api import lib
 
-SETTINGS = {"renderDensity",
-            "renderWidth",
-            "renderLength",
-            "increaseRenderBounds",
-            "imageSearchPath",
-            "cbId"}
+
+SETTINGS = {
+    # Preview
+    "displayOutput",
+    "colorR", "colorG", "colorB",
+    "viewportDensity",
+    "viewportWidth",
+    "viewportLength",
+    # Render attributes
+    "renderDensity",
+    "renderWidth",
+    "renderLength",
+    "increaseRenderBounds",
+    "imageSearchPath",
+    # Pipeline specific
+    "cbId"
+}
 
 
 class CollectYetiCache(pyblish.api.InstancePlugin):
@@ -39,10 +50,6 @@ class CollectYetiCache(pyblish.api.InstancePlugin):
         # Get yeti nodes and their transforms
         yeti_shapes = cmds.ls(instance, type="pgYetiMaya")
         for shape in yeti_shapes:
-            shape_data = {"transform": None,
-                          "name": shape,
-                          "cbId": lib.get_id(shape),
-                          "attrs": None}
 
             # Get specific node attributes
             attr_data = {}
@@ -58,9 +65,12 @@ class CollectYetiCache(pyblish.api.InstancePlugin):
             parent = cmds.listRelatives(shape, parent=True)[0]
             transform_data = {"name": parent, "cbId": lib.get_id(parent)}
 
-            # Store collected data
-            shape_data["attrs"] = attr_data
-            shape_data["transform"] = transform_data
+            shape_data = {
+                "transform": transform_data,
+                "name": shape,
+                "cbId": lib.get_id(shape),
+                "attrs": attr_data,
+            }
 
             settings["nodes"].append(shape_data)
 

From 26ef5812b7428f792e81de4a2bfa61867e73f834 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 4 Sep 2023 13:42:01 +0200
Subject: [PATCH 33/49] Skip viewport attributes on 'update' but preserve what
 artist tweaked after initial load

---
 .../hosts/maya/plugins/load/load_yeti_cache.py   | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/openpype/hosts/maya/plugins/load/load_yeti_cache.py b/openpype/hosts/maya/plugins/load/load_yeti_cache.py
index 5cded13d4e..4a11ea9a2c 100644
--- a/openpype/hosts/maya/plugins/load/load_yeti_cache.py
+++ b/openpype/hosts/maya/plugins/load/load_yeti_cache.py
@@ -15,6 +15,16 @@ from openpype.hosts.maya.api import lib
 from openpype.hosts.maya.api.pipeline import containerise
 
 
+# Do not reset these values on update but only apply on first load
+# to preserve any potential local overrides
+SKIP_UPDATE_ATTRS = {
+    "displayOutput",
+    "viewportDensity",
+    "viewportWidth",
+    "viewportLength",
+}
+
+
 def set_attribute(node, attr, value):
     """Wrapper of set attribute which ignores None values"""
     if value is None:
@@ -205,6 +215,8 @@ class YetiCacheLoader(load.LoaderPlugin):
                     yeti_node = yeti_nodes[0]
 
                     for attr, value in node_settings["attrs"].items():
+                        if attr in SKIP_UPDATE_ATTRS:
+                            continue
                         set_attribute(attr, value, yeti_node)
 
         cmds.setAttr("{}.representation".format(container_node),
@@ -311,7 +323,6 @@ class YetiCacheLoader(load.LoaderPlugin):
         # Update attributes with defaults
         attributes = node_settings["attrs"]
         attributes.update({
-            "viewportDensity": 0.1,
             "verbosity": 2,
             "fileMode": 1,
 
@@ -321,6 +332,9 @@ class YetiCacheLoader(load.LoaderPlugin):
             "visibleInRefractions": True
         })
 
+        if "viewportDensity" not in attributes:
+            attributes["viewportDensity"] = 0.1
+
         # Apply attributes to pgYetiMaya node
         for attr, value in attributes.items():
             set_attribute(attr, value, yeti_node)

From 6ef67e3ff928fca53f4f10ea3166474d5b59c8c9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jakub=20Je=C5=BEek?= <jakubjezek001@gmail.com>
Date: Mon, 4 Sep 2023 17:17:15 +0200
Subject: [PATCH 34/49] Update openpype/pipeline/colorspace.py

Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
---
 openpype/pipeline/colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/pipeline/colorspace.py b/openpype/pipeline/colorspace.py
index b7728936b0..ce0835dcc6 100644
--- a/openpype/pipeline/colorspace.py
+++ b/openpype/pipeline/colorspace.py
@@ -20,7 +20,7 @@ log = Logger.get_logger(__name__)
 
 
 class CachedData:
-    remapping = None
+    remapping = {}
     allowed_exts = {
         ext.lstrip(".") for ext in IMAGE_EXTENSIONS.union(VIDEO_EXTENSIONS)
     }

From 2fdaadcdc151918163477f3275b646a7044f43e6 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Mon, 4 Sep 2023 19:50:00 +0300
Subject: [PATCH 35/49] Minikiu comment

---
 .../hosts/houdini/plugins/publish/validate_review_colorspace.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 545d7b16b1..61c3a755d0 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -39,7 +39,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
 
         if os.getenv("OCIO") is None:
             self.log.debug(
-                "Default Houdini colorspace is used, "
+                "Using Houdini's Default Color Management, "
                 " skipping check.."
             )
             return

From 9f8cd773bbc07e07db0536430735f97ae668fcf7 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Wed, 6 Sep 2023 10:13:22 +0200
Subject: [PATCH 36/49] use start value instead of current value

---
 openpype/tools/publisher/widgets/screenshot_widget.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/tools/publisher/widgets/screenshot_widget.py b/openpype/tools/publisher/widgets/screenshot_widget.py
index 4ccf920571..170345f170 100644
--- a/openpype/tools/publisher/widgets/screenshot_widget.py
+++ b/openpype/tools/publisher/widgets/screenshot_widget.py
@@ -46,7 +46,7 @@ class ScreenMarquee(QtWidgets.QDialog):
         for screen in QtWidgets.QApplication.screens():
             screen.geometryChanged.connect(self._fit_screen_geometry)
 
-        self._opacity = fade_anim.currentValue()
+        self._opacity = fade_anim.startValue()
         self._click_pos = None
         self._capture_rect = None
 

From 2315ef84f1325a157f55bf85797d3e9719373392 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Wed, 6 Sep 2023 10:13:35 +0200
Subject: [PATCH 37/49] do not start animation on init

---
 openpype/tools/publisher/widgets/screenshot_widget.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/openpype/tools/publisher/widgets/screenshot_widget.py b/openpype/tools/publisher/widgets/screenshot_widget.py
index 170345f170..64cccece6c 100644
--- a/openpype/tools/publisher/widgets/screenshot_widget.py
+++ b/openpype/tools/publisher/widgets/screenshot_widget.py
@@ -31,7 +31,6 @@ class ScreenMarquee(QtWidgets.QDialog):
         fade_anim.setEndValue(50)
         fade_anim.setDuration(200)
         fade_anim.setEasingCurve(QtCore.QEasingCurve.OutCubic)
-        fade_anim.start(QtCore.QAbstractAnimation.DeleteWhenStopped)
 
         fade_anim.valueChanged.connect(self._on_fade_anim)
 

From 135cb285120495c250a8cbbace25a8581445fb1e Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 22:17:21 +0200
Subject: [PATCH 38/49] Add deprecation warning to usage of `fname` on Loader
 plugins

---
 openpype/pipeline/load/plugins.py | 13 +++++++++++++
 openpype/pipeline/load/utils.py   |  3 ++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/openpype/pipeline/load/plugins.py b/openpype/pipeline/load/plugins.py
index f87fb3312d..8acfcfdb6c 100644
--- a/openpype/pipeline/load/plugins.py
+++ b/openpype/pipeline/load/plugins.py
@@ -234,6 +234,19 @@ class LoaderPlugin(list):
         """
         return cls.options or []
 
+    @property
+    def fname(self):
+        """Backwards compatibility with deprecation warning"""
+
+        self.log.warning((
+            "DEPRECATION WARNING: Source - Loader plugin {}."
+            " The 'fname' property on the Loader plugin will be removed in"
+            " future versions of OpenPype. Planned version to drop the support"
+            " is 3.16.6 or 3.17.0."
+        ).format(self.__class__.__name__))
+        if hasattr(self, "_fname"):
+            return self._fname
+
 
 class SubsetLoaderPlugin(LoaderPlugin):
     """Load subset into host application
diff --git a/openpype/pipeline/load/utils.py b/openpype/pipeline/load/utils.py
index 42418be40e..b10d6032b3 100644
--- a/openpype/pipeline/load/utils.py
+++ b/openpype/pipeline/load/utils.py
@@ -318,7 +318,8 @@ def load_with_repre_context(
 
     # Backwards compatibility: Originally the loader's __init__ required the
     # representation context to set `fname` attribute to the filename to load
-    loader.fname = get_representation_path_from_context(repre_context)
+    # Deprecated - to be removed in OpenPype 3.16.6 or 3.17.0.
+    loader._fname = get_representation_path_from_context(repre_context)
 
     return loader.load(repre_context, name, namespace, options)
 

From f954c877023f902a7ab28d2d86401829883734c0 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 22:17:56 +0200
Subject: [PATCH 39/49] Refactor usage of deprecated `self.fname` to new style

---
 openpype/hosts/blender/plugins/load/load_blend.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/openpype/hosts/blender/plugins/load/load_blend.py b/openpype/hosts/blender/plugins/load/load_blend.py
index 99f291a5a7..fa41f4374b 100644
--- a/openpype/hosts/blender/plugins/load/load_blend.py
+++ b/openpype/hosts/blender/plugins/load/load_blend.py
@@ -119,7 +119,7 @@ class BlendLoader(plugin.AssetLoader):
             context: Full parenthood of representation to load
             options: Additional settings dictionary
         """
-        libpath = self.fname
+        libpath = self.filepath_from_context(context)
         asset = context["asset"]["name"]
         subset = context["subset"]["name"]
 

From ba6dfc5eadd1a4d48f92924a2e1bcf3c7d5524f6 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 23:28:39 +0200
Subject: [PATCH 40/49] Remove unused variables + tweak logs

---
 .../hosts/blender/plugins/load/load_camera_abc.py  |  2 +-
 .../hosts/blender/plugins/load/load_camera_fbx.py  |  2 +-
 .../hosts/blender/plugins/publish/extract_abc.py   |  2 --
 .../plugins/publish/extract_abc_animation.py       |  2 --
 .../blender/plugins/publish/extract_camera_abc.py  |  5 -----
 openpype/hosts/flame/plugins/load/load_clip.py     |  1 -
 .../hosts/flame/plugins/load/load_clip_batch.py    |  1 -
 .../plugins/publish/collect_timeline_instances.py  |  1 -
 .../hosts/harmony/plugins/load/load_template.py    |  1 -
 openpype/hosts/hiero/api/plugin.py                 | 14 --------------
 .../hiero/plugins/publish/collect_clip_effects.py  |  2 --
 openpype/hosts/houdini/plugins/load/load_bgeo.py   |  1 -
 openpype/hosts/max/plugins/create/create_render.py |  1 -
 .../hosts/max/plugins/publish/collect_render.py    |  1 -
 .../max/plugins/publish/extract_camera_abc.py      |  2 --
 .../max/plugins/publish/extract_camera_fbx.py      |  3 +--
 .../max/plugins/publish/extract_max_scene_raw.py   |  3 +--
 .../hosts/max/plugins/publish/extract_model.py     |  4 +---
 .../hosts/max/plugins/publish/extract_model_fbx.py |  5 +----
 .../hosts/max/plugins/publish/extract_model_obj.py |  4 +---
 .../max/plugins/publish/extract_pointcache.py      |  2 --
 .../max/plugins/publish/extract_redshift_proxy.py  |  3 +--
 openpype/hosts/maya/api/plugin.py                  |  1 -
 .../maya/plugins/inventory/import_reference.py     |  1 -
 .../hosts/maya/plugins/load/load_multiverse_usd.py |  2 --
 openpype/hosts/maya/plugins/load/load_reference.py |  4 ++--
 openpype/hosts/maya/plugins/load/load_xgen.py      |  2 --
 .../plugins/publish/collect_multiverse_look.py     |  1 -
 .../hosts/maya/tools/mayalookassigner/widgets.py   |  4 +---
 .../hosts/nuke/plugins/load/load_camera_abc.py     |  2 --
 .../plugins/publish/extract_review_data_lut.py     |  1 -
 .../nuke/plugins/publish/extract_thumbnail.py      |  2 --
 .../plugins/publish/collect_auto_image.py          |  1 -
 openpype/hosts/resolve/api/plugin.py               | 10 ----------
 .../plugins/publish/extract_thumbnail.py           |  2 --
 .../tvpaint/plugins/load/load_reference_image.py   |  2 +-
 .../tvpaint/plugins/publish/extract_sequence.py    |  3 ---
 .../hosts/unreal/plugins/publish/extract_uasset.py |  3 +--
 38 files changed, 13 insertions(+), 90 deletions(-)

diff --git a/openpype/hosts/blender/plugins/load/load_camera_abc.py b/openpype/hosts/blender/plugins/load/load_camera_abc.py
index e5afecff66..05d3fb764d 100644
--- a/openpype/hosts/blender/plugins/load/load_camera_abc.py
+++ b/openpype/hosts/blender/plugins/load/load_camera_abc.py
@@ -100,7 +100,7 @@ class AbcCameraLoader(plugin.AssetLoader):
         asset_group = bpy.data.objects.new(group_name, object_data=None)
         avalon_container.objects.link(asset_group)
 
-        objects = self._process(libpath, asset_group, group_name)
+        self._process(libpath, asset_group, group_name)
 
         objects = []
         nodes = list(asset_group.children)
diff --git a/openpype/hosts/blender/plugins/load/load_camera_fbx.py b/openpype/hosts/blender/plugins/load/load_camera_fbx.py
index b9d05dda0a..3cca6e7fd3 100644
--- a/openpype/hosts/blender/plugins/load/load_camera_fbx.py
+++ b/openpype/hosts/blender/plugins/load/load_camera_fbx.py
@@ -103,7 +103,7 @@ class FbxCameraLoader(plugin.AssetLoader):
         asset_group = bpy.data.objects.new(group_name, object_data=None)
         avalon_container.objects.link(asset_group)
 
-        objects = self._process(libpath, asset_group, group_name)
+        self._process(libpath, asset_group, group_name)
 
         objects = []
         nodes = list(asset_group.children)
diff --git a/openpype/hosts/blender/plugins/publish/extract_abc.py b/openpype/hosts/blender/plugins/publish/extract_abc.py
index f4babc94d3..87159e53f0 100644
--- a/openpype/hosts/blender/plugins/publish/extract_abc.py
+++ b/openpype/hosts/blender/plugins/publish/extract_abc.py
@@ -21,8 +21,6 @@ class ExtractABC(publish.Extractor):
         filename = f"{instance.name}.abc"
         filepath = os.path.join(stagingdir, filename)
 
-        context = bpy.context
-
         # Perform extraction
         self.log.info("Performing extraction..")
 
diff --git a/openpype/hosts/blender/plugins/publish/extract_abc_animation.py b/openpype/hosts/blender/plugins/publish/extract_abc_animation.py
index e141ccaa44..44b2ba3761 100644
--- a/openpype/hosts/blender/plugins/publish/extract_abc_animation.py
+++ b/openpype/hosts/blender/plugins/publish/extract_abc_animation.py
@@ -20,8 +20,6 @@ class ExtractAnimationABC(publish.Extractor):
         filename = f"{instance.name}.abc"
         filepath = os.path.join(stagingdir, filename)
 
-        context = bpy.context
-
         # Perform extraction
         self.log.info("Performing extraction..")
 
diff --git a/openpype/hosts/blender/plugins/publish/extract_camera_abc.py b/openpype/hosts/blender/plugins/publish/extract_camera_abc.py
index a21a59b151..036be7bf3c 100644
--- a/openpype/hosts/blender/plugins/publish/extract_camera_abc.py
+++ b/openpype/hosts/blender/plugins/publish/extract_camera_abc.py
@@ -21,16 +21,11 @@ class ExtractCameraABC(publish.Extractor):
         filename = f"{instance.name}.abc"
         filepath = os.path.join(stagingdir, filename)
 
-        context = bpy.context
-
         # Perform extraction
         self.log.info("Performing extraction..")
 
         plugin.deselect_all()
 
-        selected = []
-        active = None
-
         asset_group = None
         for obj in instance:
             if obj.get(AVALON_PROPERTY):
diff --git a/openpype/hosts/flame/plugins/load/load_clip.py b/openpype/hosts/flame/plugins/load/load_clip.py
index 338833b449..ca4eab0f63 100644
--- a/openpype/hosts/flame/plugins/load/load_clip.py
+++ b/openpype/hosts/flame/plugins/load/load_clip.py
@@ -48,7 +48,6 @@ class LoadClip(opfapi.ClipLoader):
         self.fpd = fproject.current_workspace.desktop
 
         # load clip to timeline and get main variables
-        namespace = namespace
         version = context['version']
         version_data = version.get("data", {})
         version_name = version.get("name", None)
diff --git a/openpype/hosts/flame/plugins/load/load_clip_batch.py b/openpype/hosts/flame/plugins/load/load_clip_batch.py
index ca43b94ee9..1f3a017d72 100644
--- a/openpype/hosts/flame/plugins/load/load_clip_batch.py
+++ b/openpype/hosts/flame/plugins/load/load_clip_batch.py
@@ -45,7 +45,6 @@ class LoadClipBatch(opfapi.ClipLoader):
         self.batch = options.get("batch") or flame.batch
 
         # load clip to timeline and get main variables
-        namespace = namespace
         version = context['version']
         version_data = version.get("data", {})
         version_name = version.get("name", None)
diff --git a/openpype/hosts/flame/plugins/publish/collect_timeline_instances.py b/openpype/hosts/flame/plugins/publish/collect_timeline_instances.py
index 23fdf5e785..e14f960a2b 100644
--- a/openpype/hosts/flame/plugins/publish/collect_timeline_instances.py
+++ b/openpype/hosts/flame/plugins/publish/collect_timeline_instances.py
@@ -325,7 +325,6 @@ class CollectTimelineInstances(pyblish.api.ContextPlugin):
     def _create_shot_instance(self, context, clip_name, **data):
         master_layer = data.get("heroTrack")
         hierarchy_data = data.get("hierarchyData")
-        asset = data.get("asset")
 
         if not master_layer:
             return
diff --git a/openpype/hosts/harmony/plugins/load/load_template.py b/openpype/hosts/harmony/plugins/load/load_template.py
index f3c69a9104..a78a1bf1ec 100644
--- a/openpype/hosts/harmony/plugins/load/load_template.py
+++ b/openpype/hosts/harmony/plugins/load/load_template.py
@@ -82,7 +82,6 @@ class TemplateLoader(load.LoaderPlugin):
         node = harmony.find_node_by_name(node_name, "GROUP")
         self_name = self.__class__.__name__
 
-        update_and_replace = False
         if is_representation_from_latest(representation):
             self._set_green(node)
         else:
diff --git a/openpype/hosts/hiero/api/plugin.py b/openpype/hosts/hiero/api/plugin.py
index 65a4009756..52f96261b2 100644
--- a/openpype/hosts/hiero/api/plugin.py
+++ b/openpype/hosts/hiero/api/plugin.py
@@ -317,20 +317,6 @@ class Spacer(QtWidgets.QWidget):
         self.setLayout(layout)
 
 
-def get_reference_node_parents(ref):
-    """Return all parent reference nodes of reference node
-
-    Args:
-        ref (str): reference node.
-
-    Returns:
-        list: The upstream parent reference nodes.
-
-    """
-    parents = []
-    return parents
-
-
 class SequenceLoader(LoaderPlugin):
     """A basic SequenceLoader for Resolve
 
diff --git a/openpype/hosts/hiero/plugins/publish/collect_clip_effects.py b/openpype/hosts/hiero/plugins/publish/collect_clip_effects.py
index d455ad4a4e..fcb1ab27a0 100644
--- a/openpype/hosts/hiero/plugins/publish/collect_clip_effects.py
+++ b/openpype/hosts/hiero/plugins/publish/collect_clip_effects.py
@@ -43,7 +43,6 @@ class CollectClipEffects(pyblish.api.InstancePlugin):
             if review and review_track_index == _track_index:
                 continue
             for sitem in sub_track_items:
-                effect = None
                 # make sure this subtrack item is relative of track item
                 if ((track_item not in sitem.linkedItems())
                         and (len(sitem.linkedItems()) > 0)):
@@ -53,7 +52,6 @@ class CollectClipEffects(pyblish.api.InstancePlugin):
                     continue
 
                 effect = self.add_effect(_track_index, sitem)
-
                 if effect:
                     effects.update(effect)
 
diff --git a/openpype/hosts/houdini/plugins/load/load_bgeo.py b/openpype/hosts/houdini/plugins/load/load_bgeo.py
index 22680178c0..489bf944ed 100644
--- a/openpype/hosts/houdini/plugins/load/load_bgeo.py
+++ b/openpype/hosts/houdini/plugins/load/load_bgeo.py
@@ -34,7 +34,6 @@ class BgeoLoader(load.LoaderPlugin):
 
         # Create a new geo node
         container = obj.createNode("geo", node_name=node_name)
-        is_sequence = bool(context["representation"]["context"].get("frame"))
 
         # Remove the file node, it only loads static meshes
         # Houdini 17 has removed the file node from the geo node
diff --git a/openpype/hosts/max/plugins/create/create_render.py b/openpype/hosts/max/plugins/create/create_render.py
index 235046684e..9cc3c8da8a 100644
--- a/openpype/hosts/max/plugins/create/create_render.py
+++ b/openpype/hosts/max/plugins/create/create_render.py
@@ -14,7 +14,6 @@ class CreateRender(plugin.MaxCreator):
 
     def create(self, subset_name, instance_data, pre_create_data):
         from pymxs import runtime as rt
-        sel_obj = list(rt.selection)
         file = rt.maxFileName
         filename, _ = os.path.splitext(file)
         instance_data["AssetName"] = filename
diff --git a/openpype/hosts/max/plugins/publish/collect_render.py b/openpype/hosts/max/plugins/publish/collect_render.py
index 8ee2f43103..2dfa1520a9 100644
--- a/openpype/hosts/max/plugins/publish/collect_render.py
+++ b/openpype/hosts/max/plugins/publish/collect_render.py
@@ -30,7 +30,6 @@ class CollectRender(pyblish.api.InstancePlugin):
         asset = get_current_asset_name()
 
         files_by_aov = RenderProducts().get_beauty(instance.name)
-        folder = folder.replace("\\", "/")
         aovs = RenderProducts().get_aovs(instance.name)
         files_by_aov.update(aovs)
 
diff --git a/openpype/hosts/max/plugins/publish/extract_camera_abc.py b/openpype/hosts/max/plugins/publish/extract_camera_abc.py
index b42732e70d..b1918c53e0 100644
--- a/openpype/hosts/max/plugins/publish/extract_camera_abc.py
+++ b/openpype/hosts/max/plugins/publish/extract_camera_abc.py
@@ -22,8 +22,6 @@ class ExtractCameraAlembic(publish.Extractor, OptionalPyblishPluginMixin):
         start = float(instance.data.get("frameStartHandle", 1))
         end = float(instance.data.get("frameEndHandle", 1))
 
-        container = instance.data["instance_node"]
-
         self.log.info("Extracting Camera ...")
 
         stagingdir = self.staging_dir(instance)
diff --git a/openpype/hosts/max/plugins/publish/extract_camera_fbx.py b/openpype/hosts/max/plugins/publish/extract_camera_fbx.py
index 06ac3da093..537c88eb4d 100644
--- a/openpype/hosts/max/plugins/publish/extract_camera_fbx.py
+++ b/openpype/hosts/max/plugins/publish/extract_camera_fbx.py
@@ -19,9 +19,8 @@ class ExtractCameraFbx(publish.Extractor, OptionalPyblishPluginMixin):
     def process(self, instance):
         if not self.is_active(instance.data):
             return
-        container = instance.data["instance_node"]
 
-        self.log.info("Extracting Camera ...")
+        self.log.debug("Extracting Camera ...")
         stagingdir = self.staging_dir(instance)
         filename = "{name}.fbx".format(**instance.data)
 
diff --git a/openpype/hosts/max/plugins/publish/extract_max_scene_raw.py b/openpype/hosts/max/plugins/publish/extract_max_scene_raw.py
index de5db9ab56..a7a889c587 100644
--- a/openpype/hosts/max/plugins/publish/extract_max_scene_raw.py
+++ b/openpype/hosts/max/plugins/publish/extract_max_scene_raw.py
@@ -18,10 +18,9 @@ class ExtractMaxSceneRaw(publish.Extractor, OptionalPyblishPluginMixin):
     def process(self, instance):
         if not self.is_active(instance.data):
             return
-        container = instance.data["instance_node"]
 
         # publish the raw scene for camera
-        self.log.info("Extracting Raw Max Scene ...")
+        self.log.debug("Extracting Raw Max Scene ...")
 
         stagingdir = self.staging_dir(instance)
         filename = "{name}.max".format(**instance.data)
diff --git a/openpype/hosts/max/plugins/publish/extract_model.py b/openpype/hosts/max/plugins/publish/extract_model.py
index c7ecf7efc9..38f4848c5e 100644
--- a/openpype/hosts/max/plugins/publish/extract_model.py
+++ b/openpype/hosts/max/plugins/publish/extract_model.py
@@ -20,9 +20,7 @@ class ExtractModel(publish.Extractor, OptionalPyblishPluginMixin):
         if not self.is_active(instance.data):
             return
 
-        container = instance.data["instance_node"]
-
-        self.log.info("Extracting Geometry ...")
+        self.log.debug("Extracting Geometry ...")
 
         stagingdir = self.staging_dir(instance)
         filename = "{name}.abc".format(**instance.data)
diff --git a/openpype/hosts/max/plugins/publish/extract_model_fbx.py b/openpype/hosts/max/plugins/publish/extract_model_fbx.py
index 56c2cadd94..fd48ed5007 100644
--- a/openpype/hosts/max/plugins/publish/extract_model_fbx.py
+++ b/openpype/hosts/max/plugins/publish/extract_model_fbx.py
@@ -20,10 +20,7 @@ class ExtractModelFbx(publish.Extractor, OptionalPyblishPluginMixin):
         if not self.is_active(instance.data):
             return
 
-        container = instance.data["instance_node"]
-
-
-        self.log.info("Extracting Geometry ...")
+        self.log.debug("Extracting Geometry ...")
 
         stagingdir = self.staging_dir(instance)
         filename = "{name}.fbx".format(**instance.data)
diff --git a/openpype/hosts/max/plugins/publish/extract_model_obj.py b/openpype/hosts/max/plugins/publish/extract_model_obj.py
index 4fde65cf22..e522b1e7a1 100644
--- a/openpype/hosts/max/plugins/publish/extract_model_obj.py
+++ b/openpype/hosts/max/plugins/publish/extract_model_obj.py
@@ -20,9 +20,7 @@ class ExtractModelObj(publish.Extractor, OptionalPyblishPluginMixin):
         if not self.is_active(instance.data):
             return
 
-        container = instance.data["instance_node"]
-
-        self.log.info("Extracting Geometry ...")
+        self.log.debug("Extracting Geometry ...")
 
         stagingdir = self.staging_dir(instance)
         filename = "{name}.obj".format(**instance.data)
diff --git a/openpype/hosts/max/plugins/publish/extract_pointcache.py b/openpype/hosts/max/plugins/publish/extract_pointcache.py
index 5a99a8b845..c3de623bc0 100644
--- a/openpype/hosts/max/plugins/publish/extract_pointcache.py
+++ b/openpype/hosts/max/plugins/publish/extract_pointcache.py
@@ -54,8 +54,6 @@ class ExtractAlembic(publish.Extractor):
         start = float(instance.data.get("frameStartHandle", 1))
         end = float(instance.data.get("frameEndHandle", 1))
 
-        container = instance.data["instance_node"]
-
         self.log.debug("Extracting pointcache ...")
 
         parent_dir = self.staging_dir(instance)
diff --git a/openpype/hosts/max/plugins/publish/extract_redshift_proxy.py b/openpype/hosts/max/plugins/publish/extract_redshift_proxy.py
index ab569ecbcb..f67ed30c6b 100644
--- a/openpype/hosts/max/plugins/publish/extract_redshift_proxy.py
+++ b/openpype/hosts/max/plugins/publish/extract_redshift_proxy.py
@@ -16,11 +16,10 @@ class ExtractRedshiftProxy(publish.Extractor):
     families = ["redshiftproxy"]
 
     def process(self, instance):
-        container = instance.data["instance_node"]
         start = int(instance.context.data.get("frameStart"))
         end = int(instance.context.data.get("frameEnd"))
 
-        self.log.info("Extracting Redshift Proxy...")
+        self.log.debug("Extracting Redshift Proxy...")
         stagingdir = self.staging_dir(instance)
         rs_filename = "{name}.rs".format(**instance.data)
         rs_filepath = os.path.join(stagingdir, rs_filename)
diff --git a/openpype/hosts/maya/api/plugin.py b/openpype/hosts/maya/api/plugin.py
index 3f383fafb8..4032618afb 100644
--- a/openpype/hosts/maya/api/plugin.py
+++ b/openpype/hosts/maya/api/plugin.py
@@ -683,7 +683,6 @@ class ReferenceLoader(Loader):
             loaded_containers.append(container)
             self._organize_containers(nodes, container)
             c += 1
-            namespace = None
 
         return loaded_containers
 
diff --git a/openpype/hosts/maya/plugins/inventory/import_reference.py b/openpype/hosts/maya/plugins/inventory/import_reference.py
index ecc424209d..3f3b85ba6c 100644
--- a/openpype/hosts/maya/plugins/inventory/import_reference.py
+++ b/openpype/hosts/maya/plugins/inventory/import_reference.py
@@ -12,7 +12,6 @@ class ImportReference(InventoryAction):
     color = "#d8d8d8"
 
     def process(self, containers):
-        references = cmds.ls(type="reference")
         for container in containers:
             if container["loader"] != "ReferenceLoader":
                 print("Not a reference, skipping")
diff --git a/openpype/hosts/maya/plugins/load/load_multiverse_usd.py b/openpype/hosts/maya/plugins/load/load_multiverse_usd.py
index d08fcd904e..cad42b55f9 100644
--- a/openpype/hosts/maya/plugins/load/load_multiverse_usd.py
+++ b/openpype/hosts/maya/plugins/load/load_multiverse_usd.py
@@ -43,8 +43,6 @@ class MultiverseUsdLoader(load.LoaderPlugin):
         import multiverse
 
         # Create the shape
-        shape = None
-        transform = None
         with maintained_selection():
             cmds.namespace(addNamespace=namespace)
             with namespaced(namespace, new=False):
diff --git a/openpype/hosts/maya/plugins/load/load_reference.py b/openpype/hosts/maya/plugins/load/load_reference.py
index 91767249e0..61f337f501 100644
--- a/openpype/hosts/maya/plugins/load/load_reference.py
+++ b/openpype/hosts/maya/plugins/load/load_reference.py
@@ -205,7 +205,7 @@ class ReferenceLoader(openpype.hosts.maya.api.plugin.ReferenceLoader):
                 cmds.setAttr("{}.selectHandleZ".format(group_name), cz)
 
             if family == "rig":
-                self._post_process_rig(name, namespace, context, options)
+                self._post_process_rig(namespace, context, options)
             else:
                 if "translate" in options:
                     if not attach_to_root and new_nodes:
@@ -229,7 +229,7 @@ class ReferenceLoader(openpype.hosts.maya.api.plugin.ReferenceLoader):
         members = get_container_members(container)
         self._lock_camera_transforms(members)
 
-    def _post_process_rig(self, name, namespace, context, options):
+    def _post_process_rig(self, namespace, context, options):
 
         nodes = self[:]
         create_rig_animation_instance(
diff --git a/openpype/hosts/maya/plugins/load/load_xgen.py b/openpype/hosts/maya/plugins/load/load_xgen.py
index 323f8d7eda..2ad6ad55bc 100644
--- a/openpype/hosts/maya/plugins/load/load_xgen.py
+++ b/openpype/hosts/maya/plugins/load/load_xgen.py
@@ -53,8 +53,6 @@ class XgenLoader(openpype.hosts.maya.api.plugin.ReferenceLoader):
         )
 
         # Reference xgen. Xgen does not like being referenced in under a group.
-        new_nodes = []
-
         with maintained_selection():
             nodes = cmds.file(
                 maya_filepath,
diff --git a/openpype/hosts/maya/plugins/publish/collect_multiverse_look.py b/openpype/hosts/maya/plugins/publish/collect_multiverse_look.py
index f05fb76d48..bcb979edfc 100644
--- a/openpype/hosts/maya/plugins/publish/collect_multiverse_look.py
+++ b/openpype/hosts/maya/plugins/publish/collect_multiverse_look.py
@@ -281,7 +281,6 @@ class CollectMultiverseLookData(pyblish.api.InstancePlugin):
                                         long=True)
             nodes.update(nodes_of_interest)
 
-        files = []
         sets = {}
         instance.data["resources"] = []
         publishMipMap = instance.data["publishMipMap"]
diff --git a/openpype/hosts/maya/tools/mayalookassigner/widgets.py b/openpype/hosts/maya/tools/mayalookassigner/widgets.py
index f2df17e68c..82c37e2104 100644
--- a/openpype/hosts/maya/tools/mayalookassigner/widgets.py
+++ b/openpype/hosts/maya/tools/mayalookassigner/widgets.py
@@ -90,15 +90,13 @@ class AssetOutliner(QtWidgets.QWidget):
     def get_all_assets(self):
         """Add all items from the current scene"""
 
-        items = []
         with preserve_expanded_rows(self.view):
             with preserve_selection(self.view):
                 self.clear()
                 nodes = commands.get_all_asset_nodes()
                 items = commands.create_items_from_nodes(nodes)
                 self.add_items(items)
-
-        return len(items) > 0
+                return len(items) > 0
 
     def get_selected_assets(self):
         """Add all selected items from the current scene"""
diff --git a/openpype/hosts/nuke/plugins/load/load_camera_abc.py b/openpype/hosts/nuke/plugins/load/load_camera_abc.py
index fec4ee556e..2939ceebae 100644
--- a/openpype/hosts/nuke/plugins/load/load_camera_abc.py
+++ b/openpype/hosts/nuke/plugins/load/load_camera_abc.py
@@ -112,8 +112,6 @@ class AlembicCameraLoader(load.LoaderPlugin):
         version_doc = get_version_by_id(project_name, representation["parent"])
 
         object_name = container['objectName']
-        # get corresponding node
-        camera_node = nuke.toNode(object_name)
 
         # get main variables
         version_data = version_doc.get("data", {})
diff --git a/openpype/hosts/nuke/plugins/publish/extract_review_data_lut.py b/openpype/hosts/nuke/plugins/publish/extract_review_data_lut.py
index e4b7b155cd..2a26ed82fb 100644
--- a/openpype/hosts/nuke/plugins/publish/extract_review_data_lut.py
+++ b/openpype/hosts/nuke/plugins/publish/extract_review_data_lut.py
@@ -20,7 +20,6 @@ class ExtractReviewDataLut(publish.Extractor):
     hosts = ["nuke"]
 
     def process(self, instance):
-        families = instance.data["families"]
         self.log.info("Creating staging dir...")
         if "representations" in instance.data:
             staging_dir = instance.data[
diff --git a/openpype/hosts/nuke/plugins/publish/extract_thumbnail.py b/openpype/hosts/nuke/plugins/publish/extract_thumbnail.py
index d57d55f85d..b20df4ffe2 100644
--- a/openpype/hosts/nuke/plugins/publish/extract_thumbnail.py
+++ b/openpype/hosts/nuke/plugins/publish/extract_thumbnail.py
@@ -91,8 +91,6 @@ class ExtractThumbnail(publish.Extractor):
 
         if collection:
             # get path
-            fname = os.path.basename(collection.format(
-                "{head}{padding}{tail}"))
             fhead = collection.format("{head}")
 
             thumb_fname = list(collection)[mid_frame]
diff --git a/openpype/hosts/photoshop/plugins/publish/collect_auto_image.py b/openpype/hosts/photoshop/plugins/publish/collect_auto_image.py
index f1d8419608..77f1a3e91f 100644
--- a/openpype/hosts/photoshop/plugins/publish/collect_auto_image.py
+++ b/openpype/hosts/photoshop/plugins/publish/collect_auto_image.py
@@ -16,7 +16,6 @@ class CollectAutoImage(pyblish.api.ContextPlugin):
     targets = ["automated"]
 
     def process(self, context):
-        family = "image"
         for instance in context:
             creator_identifier = instance.data.get("creator_identifier")
             if creator_identifier and creator_identifier == "auto_image":
diff --git a/openpype/hosts/resolve/api/plugin.py b/openpype/hosts/resolve/api/plugin.py
index 59c27f29da..e2bd76ffa2 100644
--- a/openpype/hosts/resolve/api/plugin.py
+++ b/openpype/hosts/resolve/api/plugin.py
@@ -413,8 +413,6 @@ class ClipLoader:
         if self.with_handles:
             source_in -= handle_start
             source_out += handle_end
-            handle_start = 0
-            handle_end = 0
 
         # make track item from source in bin as item
         timeline_item = lib.create_timeline_item(
@@ -433,14 +431,6 @@ class ClipLoader:
             self.data["path"], self.active_bin)
         _clip_property = media_pool_item.GetClipProperty
 
-        # get handles
-        handle_start = self.data["versionData"].get("handleStart")
-        handle_end = self.data["versionData"].get("handleEnd")
-        if handle_start is None:
-            handle_start = int(self.data["assetData"]["handleStart"])
-        if handle_end is None:
-            handle_end = int(self.data["assetData"]["handleEnd"])
-
         source_in = int(_clip_property("Start"))
         source_out = int(_clip_property("End"))
 
diff --git a/openpype/hosts/standalonepublisher/plugins/publish/extract_thumbnail.py b/openpype/hosts/standalonepublisher/plugins/publish/extract_thumbnail.py
index b99503b3c8..a2afd160fa 100644
--- a/openpype/hosts/standalonepublisher/plugins/publish/extract_thumbnail.py
+++ b/openpype/hosts/standalonepublisher/plugins/publish/extract_thumbnail.py
@@ -49,8 +49,6 @@ class ExtractThumbnailSP(pyblish.api.InstancePlugin):
         else:
             first_filename = files
 
-        staging_dir = None
-
         # Convert to jpeg if not yet
         full_input_path = os.path.join(
             thumbnail_repre["stagingDir"], first_filename
diff --git a/openpype/hosts/tvpaint/plugins/load/load_reference_image.py b/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
index edc116a8e4..3707ef97aa 100644
--- a/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
+++ b/openpype/hosts/tvpaint/plugins/load/load_reference_image.py
@@ -171,7 +171,7 @@ class LoadImage(plugin.Loader):
         george_script = "\n".join(george_script_lines)
         execute_george_through_file(george_script)
 
-    def _remove_container(self, container, members=None):
+    def _remove_container(self, container):
         if not container:
             return
         representation = container["representation"]
diff --git a/openpype/hosts/tvpaint/plugins/publish/extract_sequence.py b/openpype/hosts/tvpaint/plugins/publish/extract_sequence.py
index 8a610cf388..a13a91de46 100644
--- a/openpype/hosts/tvpaint/plugins/publish/extract_sequence.py
+++ b/openpype/hosts/tvpaint/plugins/publish/extract_sequence.py
@@ -63,7 +63,6 @@ class ExtractSequence(pyblish.api.Extractor):
             "ignoreLayersTransparency", False
         )
 
-        family_lowered = instance.data["family"].lower()
         mark_in = instance.context.data["sceneMarkIn"]
         mark_out = instance.context.data["sceneMarkOut"]
 
@@ -76,11 +75,9 @@ class ExtractSequence(pyblish.api.Extractor):
 
         # Frame start/end may be stored as float
         frame_start = int(instance.data["frameStart"])
-        frame_end = int(instance.data["frameEnd"])
 
         # Handles are not stored per instance but on Context
         handle_start = instance.context.data["handleStart"]
-        handle_end = instance.context.data["handleEnd"]
 
         scene_bg_color = instance.context.data["sceneBgColor"]
 
diff --git a/openpype/hosts/unreal/plugins/publish/extract_uasset.py b/openpype/hosts/unreal/plugins/publish/extract_uasset.py
index 48b62faa97..0dd7ff4a0d 100644
--- a/openpype/hosts/unreal/plugins/publish/extract_uasset.py
+++ b/openpype/hosts/unreal/plugins/publish/extract_uasset.py
@@ -19,9 +19,8 @@ class ExtractUAsset(publish.Extractor):
             "umap" if "umap" in instance.data.get("families") else "uasset")
         ar = unreal.AssetRegistryHelpers.get_asset_registry()
 
-        self.log.info("Performing extraction..")
+        self.log.debug("Performing extraction..")
         staging_dir = self.staging_dir(instance)
-        filename = f"{instance.name}.{extension}"
 
         members = instance.data.get("members", [])
 

From e79f1ef4b9b48f085f42ac43344404580522d10c Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 23:30:17 +0200
Subject: [PATCH 41/49] Replacing value only if value is in `str` is same as
 just replacing it

---
 .../hosts/houdini/plugins/publish/collect_vray_rop.py    | 9 ++-------
 openpype/hosts/maya/api/lib_rendersettings.py            | 9 ++-------
 2 files changed, 4 insertions(+), 14 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/publish/collect_vray_rop.py b/openpype/hosts/houdini/plugins/publish/collect_vray_rop.py
index d4fe37f993..277f922ba4 100644
--- a/openpype/hosts/houdini/plugins/publish/collect_vray_rop.py
+++ b/openpype/hosts/houdini/plugins/publish/collect_vray_rop.py
@@ -80,14 +80,9 @@ class CollectVrayROPRenderProducts(pyblish.api.InstancePlugin):
     def get_beauty_render_product(self, prefix, suffix="<reName>"):
         """Return the beauty output filename if render element enabled
         """
+        # Remove aov suffix from the product: `prefix.aov_suffix` -> `prefix`
         aov_parm = ".{}".format(suffix)
-        beauty_product = None
-        if aov_parm in prefix:
-            beauty_product = prefix.replace(aov_parm, "")
-        else:
-            beauty_product = prefix
-
-        return beauty_product
+        return prefix.replace(aov_parm, "")
 
     def get_render_element_name(self, node, prefix, suffix="<reName>"):
         """Return the output filename using the AOV prefix and suffix
diff --git a/openpype/hosts/maya/api/lib_rendersettings.py b/openpype/hosts/maya/api/lib_rendersettings.py
index f54633c04d..42cf29d0a7 100644
--- a/openpype/hosts/maya/api/lib_rendersettings.py
+++ b/openpype/hosts/maya/api/lib_rendersettings.py
@@ -177,12 +177,7 @@ class RenderSettings(object):
         # list all the aovs
         all_rs_aovs = cmds.ls(type='RedshiftAOV')
         for rs_aov in redshift_aovs:
-            rs_layername = rs_aov
-            if " " in rs_aov:
-                rs_renderlayer = rs_aov.replace(" ", "")
-                rs_layername = "rsAov_{}".format(rs_renderlayer)
-            else:
-                rs_layername = "rsAov_{}".format(rs_aov)
+            rs_layername = "rsAov_{}".format(rs_aov.replace(" ", ""))
             if rs_layername in all_rs_aovs:
                 continue
             cmds.rsCreateAov(type=rs_aov)
@@ -317,7 +312,7 @@ class RenderSettings(object):
             separators = [cmds.menuItem(i, query=True, label=True) for i in items]  # noqa: E501
             try:
                 sep_idx = separators.index(aov_separator)
-            except ValueError as e:
+            except ValueError:
                 six.reraise(
                     CreatorError,
                     CreatorError(

From c1b305a4462b6e58eede2853a5cd0623cf4ef925 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 23:31:33 +0200
Subject: [PATCH 42/49] Fix code (`if not found` should not have been nest into
 the for loop) + simplify logic

---
 openpype/hosts/max/api/lib_rendersettings.py | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/openpype/hosts/max/api/lib_rendersettings.py b/openpype/hosts/max/api/lib_rendersettings.py
index afde5008d5..26e176aa8d 100644
--- a/openpype/hosts/max/api/lib_rendersettings.py
+++ b/openpype/hosts/max/api/lib_rendersettings.py
@@ -37,13 +37,10 @@ class RenderSettings(object):
     def set_render_camera(self, selection):
         for sel in selection:
             # to avoid Attribute Error from pymxs wrapper
-            found = False
             if rt.classOf(sel) in rt.Camera.classes:
-                found = True
                 rt.viewport.setCamera(sel)
-                break
-            if not found:
-                raise RuntimeError("Active Camera not found")
+                return
+        raise RuntimeError("Active Camera not found")
 
     def render_output(self, container):
         folder = rt.maxFilePath

From 60334621988ee749369d222d11ecd972871a2573 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 23:32:53 +0200
Subject: [PATCH 43/49] Use asset doc and project doc from instance/context +
 tweak logic for values

---
 .../publish/validate_resolution_setting.py    | 29 ++++++-------------
 1 file changed, 9 insertions(+), 20 deletions(-)

diff --git a/openpype/hosts/max/plugins/publish/validate_resolution_setting.py b/openpype/hosts/max/plugins/publish/validate_resolution_setting.py
index 5fcb843b20..5ac41b10a0 100644
--- a/openpype/hosts/max/plugins/publish/validate_resolution_setting.py
+++ b/openpype/hosts/max/plugins/publish/validate_resolution_setting.py
@@ -6,11 +6,6 @@ from openpype.pipeline import (
 from pymxs import runtime as rt
 from openpype.hosts.max.api.lib import reset_scene_resolution
 
-from openpype.pipeline.context_tools import (
-    get_current_project_asset,
-    get_current_project
-)
-
 
 class ValidateResolutionSetting(pyblish.api.InstancePlugin,
                                 OptionalPyblishPluginMixin):
@@ -43,22 +38,16 @@ class ValidateResolutionSetting(pyblish.api.InstancePlugin,
                                          "on asset or shot.")
 
     def get_db_resolution(self, instance):
-        data = ["data.resolutionWidth", "data.resolutionHeight"]
-        project_resolution = get_current_project(fields=data)
-        project_resolution_data = project_resolution["data"]
-        asset_resolution = get_current_project_asset(fields=data)
-        asset_resolution_data = asset_resolution["data"]
-        # Set project resolution
-        project_width = int(
-            project_resolution_data.get("resolutionWidth", 1920))
-        project_height = int(
-            project_resolution_data.get("resolutionHeight", 1080))
-        width = int(
-            asset_resolution_data.get("resolutionWidth", project_width))
-        height = int(
-            asset_resolution_data.get("resolutionHeight", project_height))
+        asset_doc = instance.data["assetEntity"]
+        project_doc = instance.context.data["projectEntity"]
+        for data in [asset_doc["data"], project_doc["data"]]:
+            if "resolutionWidth" in data and "resolutionHeight" in data:
+                width = data["resolutionWidth"]
+                height = data["resolutionHeight"]
+                return int(width), int(height)
 
-        return width, height
+        # Defaults if not found in asset document or project document
+        return 1920, 1080
 
     @classmethod
     def repair(cls, instance):

From 91a4ae5dfd39d64db3ded9e98b750b8d1afe22bc Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 6 Sep 2023 23:34:01 +0200
Subject: [PATCH 44/49] Remove (not yet deprecated?) `system_settings` from
 `apply_settings`

---
 .../hosts/maya/plugins/publish/extract_import_reference.py    | 4 ++--
 openpype/hosts/maya/plugins/publish/validate_maya_units.py    | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/openpype/hosts/maya/plugins/publish/extract_import_reference.py b/openpype/hosts/maya/plugins/publish/extract_import_reference.py
index 9d2ff1a3eb..1fdee28d0c 100644
--- a/openpype/hosts/maya/plugins/publish/extract_import_reference.py
+++ b/openpype/hosts/maya/plugins/publish/extract_import_reference.py
@@ -30,8 +30,8 @@ class ExtractImportReference(publish.Extractor,
     tmp_format = "_tmp"
 
     @classmethod
-    def apply_settings(cls, project_setting, system_settings):
-        cls.active = project_setting["deadline"]["publish"]["MayaSubmitDeadline"]["import_reference"] # noqa
+    def apply_settings(cls, project_settings):
+        cls.active = project_settings["deadline"]["publish"]["MayaSubmitDeadline"]["import_reference"] # noqa
 
     def process(self, instance):
         if not self.is_active(instance.data):
diff --git a/openpype/hosts/maya/plugins/publish/validate_maya_units.py b/openpype/hosts/maya/plugins/publish/validate_maya_units.py
index 1d5619795f..ae6dc093a9 100644
--- a/openpype/hosts/maya/plugins/publish/validate_maya_units.py
+++ b/openpype/hosts/maya/plugins/publish/validate_maya_units.py
@@ -37,7 +37,7 @@ class ValidateMayaUnits(pyblish.api.ContextPlugin):
     )
 
     @classmethod
-    def apply_settings(cls, project_settings, system_settings):
+    def apply_settings(cls, project_settings):
         """Apply project settings to creator"""
         settings = (
             project_settings["maya"]["publish"]["ValidateMayaUnits"]

From b601da0dbba469c1a6920622218d74ada0f589f5 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 7 Sep 2023 11:40:59 +0200
Subject: [PATCH 45/49] do not fix folder in representation context

---
 openpype/client/server/conversion_utils.py | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/openpype/client/server/conversion_utils.py b/openpype/client/server/conversion_utils.py
index a6c190a0fc..2dca9ad57d 100644
--- a/openpype/client/server/conversion_utils.py
+++ b/openpype/client/server/conversion_utils.py
@@ -663,8 +663,8 @@ def convert_v4_representation_to_v3(representation):
         if isinstance(context, six.string_types):
             context = json.loads(context)
 
-        if "folder" in context:
-            _c_folder = context.pop("folder")
+        if "asset" not in context and "folder" in context:
+            _c_folder = context["folder"]
             context["asset"] = _c_folder["name"]
 
         if "product" in context:
@@ -959,9 +959,11 @@ def convert_create_representation_to_v4(representation, con):
     converted_representation["files"] = new_files
 
     context = representation["context"]
-    context["folder"] = {
-        "name": context.pop("asset", None)
-    }
+    if "folder" not in context:
+        context["folder"] = {
+            "name": context.get("asset")
+        }
+
     context["product"] = {
         "type": context.pop("family", None),
         "name": context.pop("subset", None),
@@ -1285,7 +1287,7 @@ def convert_update_representation_to_v4(
 
     if "context" in update_data:
         context = update_data["context"]
-        if "asset" in context:
+        if "folder" not in context and "asset" in context:
             context["folder"] = {"name": context.pop("asset")}
 
         if "family" in context or "subset" in context:

From 7cde83933a22437d3ba4a570f3c2a77f0396e810 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 7 Sep 2023 11:50:16 +0200
Subject: [PATCH 46/49] autofix wrong representations

---
 openpype/client/server/conversion_utils.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/openpype/client/server/conversion_utils.py b/openpype/client/server/conversion_utils.py
index 2dca9ad57d..f67a1ef9c4 100644
--- a/openpype/client/server/conversion_utils.py
+++ b/openpype/client/server/conversion_utils.py
@@ -667,6 +667,9 @@ def convert_v4_representation_to_v3(representation):
             _c_folder = context["folder"]
             context["asset"] = _c_folder["name"]
 
+        elif "asset" in context and "folder" not in context:
+            context["folder"] = {"name": context["asset"]}
+
         if "product" in context:
             _c_product = context.pop("product")
             context["family"] = _c_product["type"]

From a492addc54691fd23ce462cd06284809a3edeaa0 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 7 Sep 2023 15:55:28 +0200
Subject: [PATCH 47/49] Reduce artist-facing logs for component integration for
 Ftrack + tweak "Comment is not set" log also for Kitsu to debug level

---
 .../ftrack/plugins/publish/integrate_ftrack_api.py   | 12 ++++++------
 .../plugins/publish/integrate_ftrack_description.py  |  2 +-
 .../ftrack/plugins/publish/integrate_ftrack_note.py  |  6 +++---
 .../kitsu/plugins/publish/integrate_kitsu_note.py    |  2 +-
 4 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
index 4d474fab10..858c0bb2d6 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_api.py
@@ -27,8 +27,8 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
     def process(self, instance):
         component_list = instance.data.get("ftrackComponentsList")
         if not component_list:
-            self.log.info(
-                "Instance don't have components to integrate to Ftrack."
+            self.log.debug(
+                "Instance doesn't have components to integrate to Ftrack."
                 " Skipping."
             )
             return
@@ -37,7 +37,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
         task_entity, parent_entity = self.get_instance_entities(
             instance, context)
         if parent_entity is None:
-            self.log.info((
+            self.log.debug((
                 "Skipping ftrack integration. Instance \"{}\" does not"
                 " have specified ftrack entities."
             ).format(str(instance)))
@@ -323,7 +323,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
             "type_id": asset_type_id,
             "context_id": parent_id
         }
-        self.log.info("Created new Asset with data: {}.".format(asset_data))
+        self.log.debug("Created new Asset with data: {}.".format(asset_data))
         session.create("Asset", asset_data)
         session.commit()
         return self._query_asset(session, asset_name, asset_type_id, parent_id)
@@ -384,7 +384,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
             if comment:
                 new_asset_version_data["comment"] = comment
 
-            self.log.info("Created new AssetVersion with data {}".format(
+            self.log.debug("Created new AssetVersion with data {}".format(
                 new_asset_version_data
             ))
             session.create("AssetVersion", new_asset_version_data)
@@ -555,7 +555,7 @@ class IntegrateFtrackApi(pyblish.api.InstancePlugin):
                 location=location
             )
             data["component"] = component_entity
-            self.log.info(
+            self.log.debug(
                 (
                     "Created new Component with path: {0}, data: {1},"
                     " metadata: {2}, location: {3}"
diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
index 6ed02bc8b6..ceaff8ff54 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_description.py
@@ -40,7 +40,7 @@ class IntegrateFtrackDescription(pyblish.api.InstancePlugin):
 
         comment = instance.data["comment"]
         if not comment:
-            self.log.info("Comment is not set.")
+            self.log.debug("Comment is not set.")
         else:
             self.log.debug("Comment is set to `{}`".format(comment))
 
diff --git a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
index 6e82897d89..10b7932cdf 100644
--- a/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
+++ b/openpype/modules/ftrack/plugins/publish/integrate_ftrack_note.py
@@ -47,7 +47,7 @@ class IntegrateFtrackNote(pyblish.api.InstancePlugin):
         app_label = context.data["appLabel"]
         comment = instance.data["comment"]
         if not comment:
-            self.log.info("Comment is not set.")
+            self.log.debug("Comment is not set.")
         else:
             self.log.debug("Comment is set to `{}`".format(comment))
 
@@ -127,14 +127,14 @@ class IntegrateFtrackNote(pyblish.api.InstancePlugin):
 
             note_text = StringTemplate.format_template(template, format_data)
             if not note_text.solved:
-                self.log.warning((
+                self.log.debug((
                     "Note template require more keys then can be provided."
                     "\nTemplate: {}\nMissing values for keys:{}\nData: {}"
                 ).format(template, note_text.missing_keys, format_data))
                 continue
 
             if not note_text:
-                self.log.info((
+                self.log.debug((
                     "Note for AssetVersion {} would be empty. Skipping."
                     "\nTemplate: {}\nData: {}"
                 ).format(asset_version["id"], template, format_data))
diff --git a/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py b/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
index 6e5dd056f3..b66e1f01e0 100644
--- a/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
+++ b/openpype/modules/kitsu/plugins/publish/integrate_kitsu_note.py
@@ -121,7 +121,7 @@ class IntegrateKitsuNote(pyblish.api.ContextPlugin):
                 publish_comment = self.format_publish_comment(instance)
 
             if not publish_comment:
-                self.log.info("Comment is not set.")
+                self.log.debug("Comment is not set.")
             else:
                 self.log.debug("Comment is `{}`".format(publish_comment))
 

From 3a02964af5e4c2c9aac5da4770e59aeef913f98a Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 7 Sep 2023 15:58:28 +0200
Subject: [PATCH 48/49] Do not show debug log about ffmpeg probe in
 artist-facing report

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

diff --git a/openpype/lib/transcoding.py b/openpype/lib/transcoding.py
index 2bae28786e..6e323f55c1 100644
--- a/openpype/lib/transcoding.py
+++ b/openpype/lib/transcoding.py
@@ -724,7 +724,7 @@ def get_ffprobe_data(path_to_file, logger=None):
     """
     if not logger:
         logger = logging.getLogger(__name__)
-    logger.info(
+    logger.debug(
         "Getting information about input \"{}\".".format(path_to_file)
     )
     ffprobe_args = get_ffmpeg_tool_args("ffprobe")

From 2919d241da617229efa47c9ae52854633e0100c4 Mon Sep 17 00:00:00 2001
From: Mustafa-Zarkash <mustafataherzaky@outlook.com>
Date: Thu, 7 Sep 2023 19:30:10 +0300
Subject: [PATCH 49/49] move import inside

---
 .../hosts/houdini/plugins/create/create_review.py    | 12 ++++--------
 .../plugins/publish/validate_review_colorspace.py    |  2 +-
 2 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/openpype/hosts/houdini/plugins/create/create_review.py b/openpype/hosts/houdini/plugins/create/create_review.py
index c087c54f6c..60c34a358b 100644
--- a/openpype/hosts/houdini/plugins/create/create_review.py
+++ b/openpype/hosts/houdini/plugins/create/create_review.py
@@ -2,7 +2,6 @@
 """Creator plugin for creating openGL reviews."""
 from openpype.hosts.houdini.api import plugin
 from openpype.lib import EnumDef, BoolDef, NumberDef
-from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
 import os
 import hou
@@ -87,7 +86,8 @@ class CreateReview(plugin.HoudiniCreator):
 
         # Set OCIO Colorspace to the default output colorspace
         #  if there's OCIO
-        self.set_colorcorrect_to_default_view_space(instance_node)
+        if os.getenv("OCIO"):
+            self.set_colorcorrect_to_default_view_space(instance_node)
 
         to_lock = ["id", "family"]
 
@@ -134,13 +134,9 @@ class CreateReview(plugin.HoudiniCreator):
     def set_colorcorrect_to_default_view_space(self,
                                                instance_node):
         """Set ociocolorspace to the default output space."""
+        from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
-        if os.getenv("OCIO") is None:
-            # No OCIO, skip setting ociocolorspace
-            return
-
-        # if there's OCIO then set Color Correction parameter
-        # to OpenColorIO
+        # set Color Correction parameter to OpenColorIO
         instance_node.setParms({"colorcorrect": 2})
 
         # Get default view space for ociocolorspace parm.
diff --git a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
index 61c3a755d0..03ecd1b052 100644
--- a/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
+++ b/openpype/hosts/houdini/plugins/publish/validate_review_colorspace.py
@@ -6,7 +6,6 @@ from openpype.pipeline import (
 )
 from openpype.pipeline.publish import RepairAction
 from openpype.hosts.houdini.api.action import SelectROPAction
-from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
 import os
 import hou
@@ -69,6 +68,7 @@ class ValidateReviewColorspace(pyblish.api.InstancePlugin,
         It is a helper action more than a repair action,
         used to set colorspace on opengl node to the default view.
         """
+        from openpype.hosts.houdini.api.colorspace import get_default_display_view_colorspace  # noqa
 
         rop_node = hou.node(instance.data["instance_node"])