From ddf90da4fdc2284b3ef05eed265d730fad4db23b Mon Sep 17 00:00:00 2001
From: Toke Stuart Jepsen <tokejepsen@gmail.com>
Date: Wed, 10 Apr 2024 10:13:59 +0100
Subject: [PATCH 01/11] Working version

---
 client/ayon_core/hosts/hiero/api/events.py    |  4 +
 client/ayon_core/hosts/hiero/api/lib.py       | 82 +++++++++++++++----
 client/ayon_core/hosts/hiero/api/tags.py      |  2 +-
 client/ayon_core/hosts/nuke/api/lib.py        | 75 +++++++++--------
 client/ayon_core/hosts/nuke/api/plugin.py     |  1 -
 server_addon/hiero/server/settings/imageio.py | 18 ++--
 server_addon/hiero/server/version.py          |  2 +-
 server_addon/nuke/server/settings/imageio.py  | 39 +++++++--
 server_addon/nuke/server/version.py           |  2 +-
 9 files changed, 158 insertions(+), 67 deletions(-)

diff --git a/client/ayon_core/hosts/hiero/api/events.py b/client/ayon_core/hosts/hiero/api/events.py
index 304605e24e..663004abd2 100644
--- a/client/ayon_core/hosts/hiero/api/events.py
+++ b/client/ayon_core/hosts/hiero/api/events.py
@@ -8,6 +8,7 @@ from .lib import (
     sync_avalon_data_to_workfile,
     launch_workfiles_app,
     before_project_save,
+    apply_colorspace_project
 )
 from .tags import add_tags_to_workfile
 from .menu import update_menu_task_label
@@ -44,6 +45,8 @@ def afterNewProjectCreated(event):
         # reset workfiles startup not to open any more in session
         os.environ["WORKFILES_STARTUP"] = "0"
 
+    apply_colorspace_project()
+
 
 def beforeProjectLoad(event):
     log.info("before project load event...")
@@ -122,6 +125,7 @@ def register_hiero_events():
     except RuntimeError:
         pass
 
+
 def register_events():
     """
     Adding all callbacks.
diff --git a/client/ayon_core/hosts/hiero/api/lib.py b/client/ayon_core/hosts/hiero/api/lib.py
index 8682ff7780..aaf99546c7 100644
--- a/client/ayon_core/hosts/hiero/api/lib.py
+++ b/client/ayon_core/hosts/hiero/api/lib.py
@@ -11,7 +11,6 @@ import warnings
 import json
 import ast
 import secrets
-import shutil
 import hiero
 
 from qtpy import QtWidgets, QtCore
@@ -36,9 +35,6 @@ from .constants import (
     DEFAULT_SEQUENCE_NAME,
     DEFAULT_BIN_NAME
 )
-from ayon_core.pipeline.colorspace import (
-    get_imageio_config
-)
 
 
 class _CTX:
@@ -105,9 +101,9 @@ def flatten(list_):
 
 
 def get_current_project(remove_untitled=False):
-    projects = flatten(hiero.core.projects())
+    projects = hiero.core.projects()
     if not remove_untitled:
-        return next(iter(projects))
+        return projects[0]
 
     # if remove_untitled
     for proj in projects:
@@ -1050,18 +1046,68 @@ def _set_hrox_project_knobs(doc, **knobs):
 
 
 def apply_colorspace_project():
-    project_name = get_current_project_name()
-    # get path the the active projects
-    project = get_current_project(remove_untitled=True)
-    current_file = project.path()
-
-    # close the active project
-    project.close()
+    """Apply colorspaces from settings.
 
+    Due to not being able to set the project settings through the Python API,
+    we need to do use some dubious code to find the widgets and set them. It is
+    possible to set the project settings without traversing through the widgets
+    but it involves reading the hrox files from disk with XML, so no in-memory
+    support. See https://community.foundry.com/discuss/topic/137771/change-a-project-s-default-color-transform-with-python  # noqa
+    for more details.
+    """
     # get presets for hiero
+    project_name = get_current_project_name()
     imageio = get_project_settings(project_name)["hiero"]["imageio"]
     presets = imageio.get("workfile")
 
+    # Open Project Settings UI.
+    for act in hiero.ui.registeredActions():
+        if act.objectName() == "foundry.project.settings":
+            act.trigger()
+
+    # Find widgets from their sibling label.
+    labels = {
+        "Working Space:": "workingSpace",
+        "Viewer:": "viewerLut",
+        "Thumbnails:": "thumbnailLut",
+        "Monitor Out:": "monitorOutLut",
+        "8 Bit Files:": "eightBitLut",
+        "16 Bit Files:": "sixteenBitLut",
+        "Log Files:": "logLut",
+        "Floating Point Files:": "floatLut"
+    }
+    widgets = {x: None for x in labels.values()}
+
+    def _recursive_children(widget, labels, widgets):
+        children = widget.children()
+        for count, child in enumerate(children):
+            if isinstance(child, QtWidgets.QLabel):
+                if child.text() in labels.keys():
+                    widgets[labels[child.text()]] = children[count + 1]
+            _recursive_children(child, labels, widgets)
+
+    app = QtWidgets.QApplication.instance()
+    title = "Project Settings"
+    for widget in app.topLevelWidgets():
+        if isinstance(widget, QtWidgets.QMainWindow):
+            if widget.windowTitle() != title:
+                continue
+            _recursive_children(widget, labels, widgets)
+            widget.close()
+
+    msg = "Setting value \"{}\" is not a valid option for \"{}\""
+    for key, widget in widgets.items():
+        options = [widget.itemText(i) for i in range(widget.count())]
+        setting_value = presets[key]
+        assert setting_value in options, msg.format(setting_value, key)
+        widget.setCurrentText(presets[key])
+
+    # This code block is for setting up project colorspaces for files on disk.
+    # Due to not having Python API access to set the project settings, the
+    # Foundry recommended way is to modify the hrox files on disk with XML. See
+    # this forum thread for more details;
+    # https://community.foundry.com/discuss/topic/137771/change-a-project-s-default-color-transform-with-python  # noqa
+    '''
     # backward compatibility layer
     # TODO: remove this after some time
     config_data = get_imageio_config(
@@ -1074,6 +1120,13 @@ def apply_colorspace_project():
             "ocioConfigName": "custom"
         })
 
+    # get path the the active projects
+    project = get_current_project()
+    current_file = project.path()
+
+    msg = "The project needs to be saved to disk to apply colorspace settings."
+    assert current_file, msg
+
     # save the workfile as subversion "comment:_colorspaceChange"
     split_current_file = os.path.splitext(current_file)
     copy_current_file = current_file
@@ -1116,6 +1169,7 @@ def apply_colorspace_project():
 
     # open the file as current project
     hiero.core.openProject(copy_current_file)
+    '''
 
 
 def apply_colorspace_clips():
@@ -1125,10 +1179,8 @@ def apply_colorspace_clips():
 
     # get presets for hiero
     imageio = get_project_settings(project_name)["hiero"]["imageio"]
-    from pprint import pprint
 
     presets = imageio.get("regexInputs", {}).get("inputs", {})
-    pprint(presets)
     for clip in clips:
         clip_media_source_path = clip.mediaSource().firstpath()
         clip_name = clip.name()
diff --git a/client/ayon_core/hosts/hiero/api/tags.py b/client/ayon_core/hosts/hiero/api/tags.py
index 5abfee75d0..d4acb23493 100644
--- a/client/ayon_core/hosts/hiero/api/tags.py
+++ b/client/ayon_core/hosts/hiero/api/tags.py
@@ -144,7 +144,7 @@ def add_tags_to_workfile():
     # Get project task types.
     project_name = get_current_project_name()
     project_entity = ayon_api.get_project(project_name)
-    task_types = project_entity["taskType"]
+    task_types = project_entity["taskTypes"]
     nks_pres_tags["[Tasks]"] = {}
     log.debug("__ tasks: {}".format(task_types))
     for task_type in task_types:
diff --git a/client/ayon_core/hosts/nuke/api/lib.py b/client/ayon_core/hosts/nuke/api/lib.py
index 78cbe85097..e3505a16f2 100644
--- a/client/ayon_core/hosts/nuke/api/lib.py
+++ b/client/ayon_core/hosts/nuke/api/lib.py
@@ -1495,18 +1495,28 @@ class WorkfileSettings(object):
 
         filter_knobs = [
             "viewerProcess",
-            "wipe_position"
+            "wipe_position",
+            "monitorOutOutputTransform"
         ]
 
+        display, viewer = get_viewer_config_from_string(
+            viewer_dict["viewerProcess"]
+        )
+        viewer_process = create_viewer_profile_string(
+            viewer, display, path_like=False
+        )
+        display, viewer = get_viewer_config_from_string(
+            viewer_dict["output_transform"]
+        )
+        output_transform = create_viewer_profile_string(
+            viewer, display, path_like=False
+        )
         erased_viewers = []
         for v in nuke.allNodes(filter="Viewer"):
             # set viewProcess to preset from settings
-            v["viewerProcess"].setValue(
-                str(viewer_dict["viewerProcess"])
-            )
+            v["viewerProcess"].setValue(viewer_process)
 
-            if str(viewer_dict["viewerProcess"]) \
-                    not in v["viewerProcess"].value():
+            if viewer_process not in v["viewerProcess"].value():
                 copy_inputs = v.dependencies()
                 copy_knobs = {k: v[k].value() for k in v.knobs()
                               if k not in filter_knobs}
@@ -1524,11 +1534,11 @@ class WorkfileSettings(object):
 
                 # set copied knobs
                 for k, v in copy_knobs.items():
-                    print(k, v)
                     nv[k].setValue(v)
 
                 # set viewerProcess
-                nv["viewerProcess"].setValue(str(viewer_dict["viewerProcess"]))
+                nv["viewerProcess"].setValue(viewer_process)
+                nv["monitorOutOutputTransform"].setValue(output_transform)
 
         if erased_viewers:
             log.warning(
@@ -1547,7 +1557,6 @@ class WorkfileSettings(object):
             host_name="nuke"
         )
 
-        viewer_process_settings = imageio_host["viewer"]["viewerProcess"]
         workfile_settings = imageio_host["workfile"]
         color_management = workfile_settings["color_management"]
         native_ocio_config = workfile_settings["native_ocio_config"]
@@ -1574,29 +1583,6 @@ class WorkfileSettings(object):
                     residual_path
                 ))
 
-        # get monitor lut from settings respecting Nuke version differences
-        monitor_lut = workfile_settings["thumbnail_space"]
-        monitor_lut_data = self._get_monitor_settings(
-            viewer_process_settings, monitor_lut
-        )
-        monitor_lut_data["workingSpaceLUT"] = (
-            workfile_settings["working_space"]
-        )
-
-        # then set the rest
-        for knob, value_ in monitor_lut_data.items():
-            # skip unfilled ocio config path
-            # it will be dict in value
-            if isinstance(value_, dict):
-                continue
-            # skip empty values
-            if not value_:
-                continue
-            if self._root_node[knob].value() not in value_:
-                self._root_node[knob].setValue(str(value_))
-                log.debug("nuke.root()['{}'] changed to: {}".format(
-                    knob, value_))
-
         # set ocio config path
         if config_data:
             config_path = config_data["path"].replace("\\", "/")
@@ -1611,6 +1597,31 @@ class WorkfileSettings(object):
             if correct_settings:
                 self._set_ocio_config_path_to_workfile(config_data)
 
+        # get monitor lut from settings respecting Nuke version differences
+        monitor_lut_data = self._get_monitor_settings(
+            workfile_settings["monitor_out_lut"],
+            workfile_settings["monitor_lut"]
+        )
+        monitor_lut_data.update({
+            "workingSpaceLUT": workfile_settings["working_space"],
+            "int8Lut": workfile_settings["int_8_lut"],
+            "int16Lut": workfile_settings["int_16_lut"],
+            "logLut": workfile_settings["log_lut"],
+            "floatLut": workfile_settings["float_lut"]
+        })
+
+        # then set the rest
+        for knob, value_ in monitor_lut_data.items():
+            # skip unfilled ocio config path
+            # it will be dict in value
+            if isinstance(value_, dict):
+                continue
+            # skip empty values
+            if not value_:
+                continue
+            self._root_node[knob].setValue(str(value_))
+            log.debug("nuke.root()['{}'] changed to: {}".format(knob, value_))
+
     def _get_monitor_settings(self, viewer_lut, monitor_lut):
         """ Get monitor settings from viewer and monitor lut
 
diff --git a/client/ayon_core/hosts/nuke/api/plugin.py b/client/ayon_core/hosts/nuke/api/plugin.py
index 6aa098c558..56c30a8ff5 100644
--- a/client/ayon_core/hosts/nuke/api/plugin.py
+++ b/client/ayon_core/hosts/nuke/api/plugin.py
@@ -1151,7 +1151,6 @@ def _remove_old_knobs(node):
         "OpenpypeDataGroup", "OpenpypeDataGroup_End", "deadlinePriority",
         "deadlineChunkSize", "deadlineConcurrentTasks", "Deadline"
     ]
-    print(node.name())
 
     # remove all old knobs
     for knob in node.allKnobs():
diff --git a/server_addon/hiero/server/settings/imageio.py b/server_addon/hiero/server/settings/imageio.py
index f2bc71ac33..9e15e15597 100644
--- a/server_addon/hiero/server/settings/imageio.py
+++ b/server_addon/hiero/server/settings/imageio.py
@@ -149,15 +149,15 @@ class ImageIOSettings(BaseSettingsModel):
 
 DEFAULT_IMAGEIO_SETTINGS = {
     "workfile": {
-        "ocioConfigName": "nuke-default",
-        "workingSpace": "linear",
-        "viewerLut": "sRGB",
-        "eightBitLut": "sRGB",
-        "sixteenBitLut": "sRGB",
-        "logLut": "Cineon",
-        "floatLut": "linear",
-        "thumbnailLut": "sRGB",
-        "monitorOutLut": "sRGB"
+        "ocioConfigName": "aces_1.2",
+        "workingSpace": "role_scene_linear",
+        "viewerLut": "ACES/sRGB",
+        "eightBitLut": "role_matte_paint",
+        "sixteenBitLut": "role_texture_paint",
+        "logLut": "role_compositing_log",
+        "floatLut": "role_scene_linear",
+        "thumbnailLut": "ACES/sRGB",
+        "monitorOutLut": "ACES/sRGB"
     },
     "regexInputs": {
         "inputs": [
diff --git a/server_addon/hiero/server/version.py b/server_addon/hiero/server/version.py
index b3f4756216..ae7362549b 100644
--- a/server_addon/hiero/server/version.py
+++ b/server_addon/hiero/server/version.py
@@ -1 +1 @@
-__version__ = "0.1.2"
+__version__ = "0.1.3"
diff --git a/server_addon/nuke/server/settings/imageio.py b/server_addon/nuke/server/settings/imageio.py
index 1b84457133..9cdb0bf1d7 100644
--- a/server_addon/nuke/server/settings/imageio.py
+++ b/server_addon/nuke/server/settings/imageio.py
@@ -97,8 +97,23 @@ class WorkfileColorspaceSettings(BaseSettingsModel):
     working_space: str = SettingsField(
         title="Working Space"
     )
-    thumbnail_space: str = SettingsField(
-        title="Thumbnail Space"
+    monitor_lut: str = SettingsField(
+        title="Thumbnails"
+    )
+    monitor_out_lut: str = SettingsField(
+        title="Monitor Out"
+    )
+    int_8_lut: str = SettingsField(
+        title="8-bit Files"
+    )
+    int_16_lut: str = SettingsField(
+        title="16-bit Files"
+    )
+    log_lut: str = SettingsField(
+        title="Log Files"
+    )
+    float_lut: str = SettingsField(
+        title="Float Files"
     )
 
 
@@ -120,6 +135,9 @@ class ViewProcessModel(BaseSettingsModel):
     viewerProcess: str = SettingsField(
         title="Viewer Process Name"
     )
+    output_transform: str = SettingsField(
+        title="Output Transform"
+    )
 
 
 class ImageIOConfigModel(BaseSettingsModel):
@@ -214,16 +232,23 @@ class ImageIOSettings(BaseSettingsModel):
 
 DEFAULT_IMAGEIO_SETTINGS = {
     "viewer": {
-        "viewerProcess": "sRGB (default)"
+        "viewerProcess": "ACES/sRGB",
+        "output_transform": "ACES/sRGB"
     },
     "baking": {
-        "viewerProcess": "rec709 (default)"
+        "viewerProcess": "ACES/Rec.709",
+        "output_transform": "ACES/Rec.709"
     },
     "workfile": {
         "color_management": "OCIO",
-        "native_ocio_config": "nuke-default",
-        "working_space": "scene_linear",
-        "thumbnail_space": "sRGB (default)",
+        "native_ocio_config": "aces_1.2",
+        "working_space": "role_scene_linear",
+        "monitor_lut": "ACES/sRGB",
+        "monitor_out_lut": "ACES/sRGB",
+        "int_8_lut": "role_matte_paint",
+        "int_16_lut": "role_texture_paint",
+        "log_lut": "role_compositing_log",
+        "float_lut": "role_scene_linear"
     },
     "nodes": {
         "required_nodes": [
diff --git a/server_addon/nuke/server/version.py b/server_addon/nuke/server/version.py
index 569b1212f7..0c5c30071a 100644
--- a/server_addon/nuke/server/version.py
+++ b/server_addon/nuke/server/version.py
@@ -1 +1 @@
-__version__ = "0.1.10"
+__version__ = "0.1.11"

From d7b20dff37693492a16e6742d7e679808c4fe7a0 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Wed, 17 Apr 2024 23:53:20 +0200
Subject: [PATCH 02/11] Workfile templates: add event system to Workfile
 Template Builder

---
 .../workfile/workfile_template_builder.py     | 68 +++++++++++++++++++
 1 file changed, 68 insertions(+)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index 5e63ba444a..f53aee6341 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -36,6 +36,7 @@ from ayon_core.lib import (
     filter_profiles,
     attribute_definitions,
 )
+from ayon_core.lib.events import EventSystem
 from ayon_core.lib.attribute_definitions import get_attributes_keys
 from ayon_core.pipeline import Anatomy
 from ayon_core.pipeline.load import (
@@ -124,6 +125,8 @@ class AbstractTemplateBuilder(object):
         self._current_task_entity = _NOT_SET
         self._linked_folder_entities = _NOT_SET
 
+        self._event_system = EventSystem()
+
     @property
     def project_name(self):
         if isinstance(self._host, HostBase):
@@ -244,6 +247,14 @@ class AbstractTemplateBuilder(object):
             self._log = Logger.get_logger(repr(self))
         return self._log
 
+    @property
+    def event_system(self):
+        """Event System of the Workfile templatee builder.
+        Returns:
+            EventSystem: The event system.
+        """
+        return self._event_system
+
     def refresh(self):
         """Reset cached data."""
 
@@ -257,6 +268,8 @@ class AbstractTemplateBuilder(object):
 
         self._project_settings = None
 
+        self._event_system = EventSystem()
+
         self.clear_shared_data()
         self.clear_shared_populate_data()
 
@@ -729,6 +742,16 @@ class AbstractTemplateBuilder(object):
 
                 placeholder.set_finished()
 
+            # Trigger on_depth_processed event
+            self.event_system.emit(
+                topic="template.depth_processed",
+                data={
+                    "depth": iter_counter,
+                    "placeholders_by_scene_id": placeholder_by_scene_id
+                },
+                source="builder"
+            )
+
             # Clear shared data before getting new placeholders
             self.clear_shared_populate_data()
 
@@ -747,6 +770,16 @@ class AbstractTemplateBuilder(object):
                 placeholder_by_scene_id[identifier] = placeholder
                 placeholders.append(placeholder)
 
+        # Trigger on_finished event
+        self.event_system.emit(
+            topic="template.finished",
+            data={
+                "depth": iter_counter,
+                "placeholders_by_scene_id": placeholder_by_scene_id,
+            },
+            source="builder"
+        )
+
         self.refresh()
 
     def _get_build_profiles(self):
@@ -1102,6 +1135,41 @@ class PlaceholderPlugin(object):
         plugin_data[key] = value
         self.builder.set_shared_populate_data(self.identifier, plugin_data)
 
+    def register_on_finished_callback(
+            self, placeholder, callback, order=None
+    ):
+        self.register_callback(
+            placeholder,
+            topic="template.finished",
+            callback=callback,
+            order=order
+        )
+
+    def register_on_depth_processed_callback(
+            self, placeholder, callback, order=0
+    ):
+        self.register_callback(
+            placeholder,
+            topic="template.depth_processed",
+            callback=callback,
+            order=order
+        )
+
+    def register_callback(self, placeholder, topic, callback, order=None):
+
+        if order is None:
+            # Match placeholder order by default
+            order = placeholder.order
+
+        # We must persist the callback over time otherwise it will be removed
+        # by the event system as a valid function reference. We do that here
+        # always just so it's easier to develop plugins where callbacks might
+        # be partials or lambdas
+        placeholder.data.setdefault("callbacks", []).append(callback)
+        self.log.debug("Registering '%s' callback: %s", topic, callback)
+        self.builder.event_system.add_callback(topic, callback, order=order)
+
+
 
 class PlaceholderItem(object):
     """Item representing single item in scene that is a placeholder to process.

From edee279f15fa16b549db68e3adb6aba14fc2f541 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 18 Apr 2024 12:04:24 +0200
Subject: [PATCH 03/11] Do not store in placeholder data - it's up to the
 registering code itself to persist or use the `weakref_partial`
 implementation from the event system

---
 .../pipeline/workfile/workfile_template_builder.py          | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index f53aee6341..d08c951b36 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -1161,16 +1161,10 @@ class PlaceholderPlugin(object):
             # Match placeholder order by default
             order = placeholder.order
 
-        # We must persist the callback over time otherwise it will be removed
-        # by the event system as a valid function reference. We do that here
-        # always just so it's easier to develop plugins where callbacks might
-        # be partials or lambdas
-        placeholder.data.setdefault("callbacks", []).append(callback)
         self.log.debug("Registering '%s' callback: %s", topic, callback)
         self.builder.event_system.add_callback(topic, callback, order=order)
 
 
-
 class PlaceholderItem(object):
     """Item representing single item in scene that is a placeholder to process.
 

From 2befe843dc6fe9168b156ccf9501c99ddd48fe9e Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 18 Apr 2024 12:05:06 +0200
Subject: [PATCH 04/11] Do not force order of the placeholder, allow it to be
 `None`

---
 .../pipeline/workfile/workfile_template_builder.py         | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index d08c951b36..b27b614579 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -1146,7 +1146,7 @@ class PlaceholderPlugin(object):
         )
 
     def register_on_depth_processed_callback(
-            self, placeholder, callback, order=0
+            self, placeholder, callback, order=None
     ):
         self.register_callback(
             placeholder,
@@ -1156,11 +1156,6 @@ class PlaceholderPlugin(object):
         )
 
     def register_callback(self, placeholder, topic, callback, order=None):
-
-        if order is None:
-            # Match placeholder order by default
-            order = placeholder.order
-
         self.log.debug("Registering '%s' callback: %s", topic, callback)
         self.builder.event_system.add_callback(topic, callback, order=order)
 

From 47c7e8634c9deafe307e25cfe4bb654851fbbb63 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Thu, 18 Apr 2024 12:10:45 +0200
Subject: [PATCH 05/11] Do not expose the `event_system` on the builder
 directly - but expose the register and trigger event methods

---
 .../workfile/workfile_template_builder.py     | 60 ++++++++-----------
 1 file changed, 26 insertions(+), 34 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index b27b614579..22a4c984bc 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -247,14 +247,6 @@ class AbstractTemplateBuilder(object):
             self._log = Logger.get_logger(repr(self))
         return self._log
 
-    @property
-    def event_system(self):
-        """Event System of the Workfile templatee builder.
-        Returns:
-            EventSystem: The event system.
-        """
-        return self._event_system
-
     def refresh(self):
         """Reset cached data."""
 
@@ -743,7 +735,7 @@ class AbstractTemplateBuilder(object):
                 placeholder.set_finished()
 
             # Trigger on_depth_processed event
-            self.event_system.emit(
+            self.trigger_event(
                 topic="template.depth_processed",
                 data={
                     "depth": iter_counter,
@@ -771,7 +763,7 @@ class AbstractTemplateBuilder(object):
                 placeholders.append(placeholder)
 
         # Trigger on_finished event
-        self.event_system.emit(
+        self.trigger_event(
             topic="template.finished",
             data={
                 "depth": iter_counter,
@@ -905,6 +897,30 @@ class AbstractTemplateBuilder(object):
             "create_first_version": create_first_version
         }
 
+    def trigger_event(self, topic, data=None, source=None):
+        self._event_system.emit(topic, data, source)
+
+    def register_event_callback(self, topic, callback, order=None):
+        self._event_system.add_callback(topic, callback, order=order)
+
+    def register_on_finished_callback(
+            self, callback, order=None
+    ):
+        self.register_event_callback(
+            topic="template.finished",
+            callback=callback,
+            order=order
+        )
+
+    def register_on_depth_processed_callback(
+            self, callback, order=None
+    ):
+        self.register_event_callback(
+            topic="template.depth_processed",
+            callback=callback,
+            order=order
+        )
+
 
 @six.add_metaclass(ABCMeta)
 class PlaceholderPlugin(object):
@@ -1135,30 +1151,6 @@ class PlaceholderPlugin(object):
         plugin_data[key] = value
         self.builder.set_shared_populate_data(self.identifier, plugin_data)
 
-    def register_on_finished_callback(
-            self, placeholder, callback, order=None
-    ):
-        self.register_callback(
-            placeholder,
-            topic="template.finished",
-            callback=callback,
-            order=order
-        )
-
-    def register_on_depth_processed_callback(
-            self, placeholder, callback, order=None
-    ):
-        self.register_callback(
-            placeholder,
-            topic="template.depth_processed",
-            callback=callback,
-            order=order
-        )
-
-    def register_callback(self, placeholder, topic, callback, order=None):
-        self.log.debug("Registering '%s' callback: %s", topic, callback)
-        self.builder.event_system.add_callback(topic, callback, order=order)
-
 
 class PlaceholderItem(object):
     """Item representing single item in scene that is a placeholder to process.

From da5abf836773283ccb4d1296a35258cc9106e443 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Fri, 19 Apr 2024 22:24:44 +0200
Subject: [PATCH 06/11] Update
 client/ayon_core/pipeline/workfile/workfile_template_builder.py

---
 .../pipeline/workfile/workfile_template_builder.py   | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index 22a4c984bc..a4fa2b4ddd 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -897,25 +897,25 @@ class AbstractTemplateBuilder(object):
             "create_first_version": create_first_version
         }
 
-    def trigger_event(self, topic, data=None, source=None):
+    def emit_event(self, topic, data=None, source=None):
         self._event_system.emit(topic, data, source)
 
-    def register_event_callback(self, topic, callback, order=None):
+    def add_event_callback(self, topic, callback, order=None):
         self._event_system.add_callback(topic, callback, order=order)
 
-    def register_on_finished_callback(
+    def add_on_finished_callback(
             self, callback, order=None
     ):
-        self.register_event_callback(
+        self.add_event_callback(
             topic="template.finished",
             callback=callback,
             order=order
         )
 
-    def register_on_depth_processed_callback(
+    def add_on_depth_processed_callback(
             self, callback, order=None
     ):
-        self.register_event_callback(
+        self.add_event_callback(
             topic="template.depth_processed",
             callback=callback,
             order=order

From 8194a7b07b1799ffafe960b25e0d86c2feaff258 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Fri, 19 Apr 2024 22:26:54 +0200
Subject: [PATCH 07/11] Return the values of the called functions

---
 .../workfile/workfile_template_builder.py        | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index a4fa2b4ddd..d189e3e2d6 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -36,7 +36,7 @@ from ayon_core.lib import (
     filter_profiles,
     attribute_definitions,
 )
-from ayon_core.lib.events import EventSystem
+from ayon_core.lib.events import EventSystem, EventCallback, Event
 from ayon_core.lib.attribute_definitions import get_attributes_keys
 from ayon_core.pipeline import Anatomy
 from ayon_core.pipeline.load import (
@@ -897,16 +897,16 @@ class AbstractTemplateBuilder(object):
             "create_first_version": create_first_version
         }
 
-    def emit_event(self, topic, data=None, source=None):
-        self._event_system.emit(topic, data, source)
+    def emit_event(self, topic, data=None, source=None) -> Event:
+        return self._event_system.emit(topic, data, source)
 
     def add_event_callback(self, topic, callback, order=None):
-        self._event_system.add_callback(topic, callback, order=order)
+        return self._event_system.add_callback(topic, callback, order=order)
 
     def add_on_finished_callback(
             self, callback, order=None
-    ):
-        self.add_event_callback(
+    ) -> EventCallback:
+        return self.add_event_callback(
             topic="template.finished",
             callback=callback,
             order=order
@@ -914,8 +914,8 @@ class AbstractTemplateBuilder(object):
 
     def add_on_depth_processed_callback(
             self, callback, order=None
-    ):
-        self.add_event_callback(
+    ) -> EventCallback:
+        return self.add_event_callback(
             topic="template.depth_processed",
             callback=callback,
             order=order

From 72028abc2c70955fb25a63e6628cdaa656e97a50 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 29 Apr 2024 15:55:36 +0200
Subject: [PATCH 08/11] Fix import

---
 .../ayon_core/hosts/maya/tools/mayalookassigner/vray_proxies.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/ayon_core/hosts/maya/tools/mayalookassigner/vray_proxies.py b/client/ayon_core/hosts/maya/tools/mayalookassigner/vray_proxies.py
index 88ef4b201a..c1d9f019e4 100644
--- a/client/ayon_core/hosts/maya/tools/mayalookassigner/vray_proxies.py
+++ b/client/ayon_core/hosts/maya/tools/mayalookassigner/vray_proxies.py
@@ -7,7 +7,7 @@ from maya import cmds
 import ayon_api
 
 from ayon_core.pipeline import get_current_project_name
-import ayon_core.hosts.maya.lib as maya_lib
+import ayon_core.hosts.maya.api.lib as maya_lib
 from . import lib
 from .alembic import get_alembic_ids_cache
 

From b5f4a843d516bf505a73455a956bb81b3e5dbe31 Mon Sep 17 00:00:00 2001
From: Kayla Man <moonyuet2008@hotmail.com>
Date: Tue, 30 Apr 2024 13:31:30 +0800
Subject: [PATCH 09/11] add more fps value support & supports to reset fps
 value based on the task entity

---
 client/ayon_core/hosts/maya/api/lib.py | 35 ++++++++++++++++++--------
 1 file changed, 24 insertions(+), 11 deletions(-)

diff --git a/client/ayon_core/hosts/maya/api/lib.py b/client/ayon_core/hosts/maya/api/lib.py
index 321bcbc0b5..017d0cd2c4 100644
--- a/client/ayon_core/hosts/maya/api/lib.py
+++ b/client/ayon_core/hosts/maya/api/lib.py
@@ -2520,7 +2520,17 @@ def set_scene_fps(fps, update=True):
     """
 
     fps_mapping = {
+        # 2, 3, 4, 5, 6, 8, 10, 12, 16
+        '2': '2fps',
+        '3': '3fps',
+        '4': '4fps',
+        '5': '5fps',
+        '6': '6fps',
+        '8': '8fps',
+        '10': '10fps',
+        '12': '12fps',
         '15': 'game',
+        '16': '16fps',
         '24': 'film',
         '25': 'pal',
         '30': 'ntsc',
@@ -2612,21 +2622,24 @@ def get_fps_for_current_context():
     Returns:
         Union[int, float]: FPS value.
     """
-
-    project_name = get_current_project_name()
-    folder_path = get_current_folder_path()
-    folder_entity = ayon_api.get_folder_by_path(
-        project_name, folder_path, fields={"attrib.fps"}
-    ) or {}
-    fps = folder_entity.get("attrib", {}).get("fps")
+    task_entity = get_current_task_entity(fields={"attrib"})
+    fps = task_entity.get("attrib", {}).get("fps")
     if not fps:
-        project_entity = ayon_api.get_project(
-            project_name, fields=["attrib.fps"]
+        project_name = get_current_project_name()
+        folder_path = get_current_folder_path()
+        folder_entity = ayon_api.get_folder_by_path(
+            project_name, folder_path, fields={"attrib.fps"}
         ) or {}
-        fps = project_entity.get("attrib", {}).get("fps")
 
+        fps = folder_entity.get("attrib", {}).get("fps")
         if not fps:
-            fps = 25
+            project_entity = ayon_api.get_project(
+                project_name, fields=["attrib.fps"]
+            ) or {}
+            fps = project_entity.get("attrib", {}).get("fps")
+
+            if not fps:
+                fps = 25
 
     return convert_to_maya_fps(fps)
 

From 444b0d8fc7bacf5ad4517b061cf05c55ce998643 Mon Sep 17 00:00:00 2001
From: Kayla Man <64118225+moonyuet@users.noreply.github.com>
Date: Tue, 30 Apr 2024 15:38:32 +0800
Subject: [PATCH 10/11] Update client/ayon_core/hosts/maya/api/lib.py

Co-authored-by: Toke Jepsen <tokejepsen@gmail.com>
---
 client/ayon_core/hosts/maya/api/lib.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/client/ayon_core/hosts/maya/api/lib.py b/client/ayon_core/hosts/maya/api/lib.py
index 017d0cd2c4..b8c9bedc60 100644
--- a/client/ayon_core/hosts/maya/api/lib.py
+++ b/client/ayon_core/hosts/maya/api/lib.py
@@ -2520,7 +2520,6 @@ def set_scene_fps(fps, update=True):
     """
 
     fps_mapping = {
-        # 2, 3, 4, 5, 6, 8, 10, 12, 16
         '2': '2fps',
         '3': '3fps',
         '4': '4fps',

From c50bd1498e820ed9af035578fbfe8f6c8ffb8854 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Tue, 30 Apr 2024 10:34:31 +0200
Subject: [PATCH 11/11] Apply suggestions from code review

Co-authored-by: Jakub Trllo <43494761+iLLiCiTiT@users.noreply.github.com>
---
 .../pipeline/workfile/workfile_template_builder.py        | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/client/ayon_core/pipeline/workfile/workfile_template_builder.py b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
index 64f2ad1c18..fc4ba3fe98 100644
--- a/client/ayon_core/pipeline/workfile/workfile_template_builder.py
+++ b/client/ayon_core/pipeline/workfile/workfile_template_builder.py
@@ -741,7 +741,7 @@ class AbstractTemplateBuilder(object):
                 placeholder.set_finished()
 
             # Trigger on_depth_processed event
-            self.trigger_event(
+            self.emit_event(
                 topic="template.depth_processed",
                 data={
                     "depth": iter_counter,
@@ -769,7 +769,7 @@ class AbstractTemplateBuilder(object):
                 placeholders.append(placeholder)
 
         # Trigger on_finished event
-        self.trigger_event(
+        self.emit_event(
             topic="template.finished",
             data={
                 "depth": iter_counter,
@@ -912,7 +912,7 @@ class AbstractTemplateBuilder(object):
         return self._event_system.add_callback(topic, callback, order=order)
 
     def add_on_finished_callback(
-            self, callback, order=None
+        self, callback, order=None
     ) -> EventCallback:
         return self.add_event_callback(
             topic="template.finished",
@@ -921,7 +921,7 @@ class AbstractTemplateBuilder(object):
         )
 
     def add_on_depth_processed_callback(
-            self, callback, order=None
+        self, callback, order=None
     ) -> EventCallback:
         return self.add_event_callback(
             topic="template.depth_processed",