From 52f4b75b62b3776f7b1a3c4d65690b3e5b53cace Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Thu, 23 Mar 2023 22:30:23 +0100
Subject: [PATCH] Flame: skip colorspace management if imageio is not enabled

also set enabled to True by default since some productions are already using it
---
 openpype/hosts/flame/hooks/pre_flame_setup.py | 21 +++++++++++++++----
 .../defaults/project_settings/flame.json      |  2 +-
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/openpype/hosts/flame/hooks/pre_flame_setup.py b/openpype/hosts/flame/hooks/pre_flame_setup.py
index 713daf1031..6367e132de 100644
--- a/openpype/hosts/flame/hooks/pre_flame_setup.py
+++ b/openpype/hosts/flame/hooks/pre_flame_setup.py
@@ -47,6 +47,13 @@ class FlamePrelaunch(PreLaunchHook):
 
         imageio_flame = project_settings["flame"]["imageio"]
 
+        colormanaged = True
+        # check if host settings are having enabled key and if it is False
+        if imageio_flame.get("enabled") and imageio_flame["enabled"] is False:
+            # if host settings are disabled return False because
+            # it is expected that no colorspace management is needed
+            colormanaged = False
+
         # get user name and host name
         user_name = get_openpype_username()
         user_name = user_name.replace(".", "_")
@@ -68,9 +75,7 @@ class FlamePrelaunch(PreLaunchHook):
             "FrameWidth": int(width),
             "FrameHeight": int(height),
             "AspectRatio": float((width / height) * _db_p_data["pixelAspect"]),
-            "FrameRate": self._get_flame_fps(fps),
-            "FrameDepth": str(imageio_flame["project"]["frameDepth"]),
-            "FieldDominance": str(imageio_flame["project"]["fieldDominance"])
+            "FrameRate": self._get_flame_fps(fps)
         }
 
         data_to_script = {
@@ -78,7 +83,6 @@ class FlamePrelaunch(PreLaunchHook):
             "host_name": _env.get("FLAME_WIRETAP_HOSTNAME") or hostname,
             "volume_name": volume_name,
             "group_name": _env.get("FLAME_WIRETAP_GROUP"),
-            "color_policy": str(imageio_flame["project"]["colourPolicy"]),
 
             # from project
             "project_name": project_name,
@@ -86,6 +90,15 @@ class FlamePrelaunch(PreLaunchHook):
             "project_data": project_data
         }
 
+        # add color management data
+        if colormanaged:
+            project_data.update({
+                "FrameDepth": str(imageio_flame["project"]["frameDepth"]),
+                "FieldDominance": str(imageio_flame["project"]["fieldDominance"])
+            })
+            data_to_script["color_policy"] = str(
+                imageio_flame["project"]["colourPolicy"])
+
         self.log.info(pformat(dict(_env)))
         self.log.info(pformat(data_to_script))
 
diff --git a/openpype/settings/defaults/project_settings/flame.json b/openpype/settings/defaults/project_settings/flame.json
index f323af7496..5eb6ec2d2a 100644
--- a/openpype/settings/defaults/project_settings/flame.json
+++ b/openpype/settings/defaults/project_settings/flame.json
@@ -1,6 +1,6 @@
 {
     "imageio": {
-        "enabled": false,
+        "enabled": true,
         "ocio_config": {
             "enabled": false,
             "filepath": []