From 3ee386543bc6b91ee7a0ab3c95424ac7955d7d98 Mon Sep 17 00:00:00 2001
From: Jakub Jezek <jakubjezek001@gmail.com>
Date: Wed, 9 Nov 2022 11:43:55 +0100
Subject: [PATCH] hiero: adding animated knobs also making track per subset

---
 .../hosts/hiero/plugins/load/load_effects.py  | 27 +++++++++++++++++--
 1 file changed, 25 insertions(+), 2 deletions(-)

diff --git a/openpype/hosts/hiero/plugins/load/load_effects.py b/openpype/hosts/hiero/plugins/load/load_effects.py
index fab426e58d..0819d1d1b7 100644
--- a/openpype/hosts/hiero/plugins/load/load_effects.py
+++ b/openpype/hosts/hiero/plugins/load/load_effects.py
@@ -13,6 +13,7 @@ from openpype.pipeline import (
     get_representation_path
 )
 from openpype.hosts.hiero import api as phiero
+from openpype.lib import Logger
 
 
 class LoadEffects(load.LoaderPlugin):
@@ -26,6 +27,8 @@ class LoadEffects(load.LoaderPlugin):
     icon = "cc"
     color = "white"
 
+    log = Logger.get_logger(__name__)
+
     def load(self, context, name, namespace, data):
         """
         Loading function to get the soft effects to particular read node
@@ -41,7 +44,7 @@ class LoadEffects(load.LoaderPlugin):
         """
         active_sequence = phiero.get_current_sequence()
         active_track = phiero.get_current_track(
-            active_sequence, "LoadedEffects")
+            active_sequence, "Loaded_{}".format(name))
 
         # get main variables
         namespace = namespace or context["asset"]["name"]
@@ -119,7 +122,27 @@ class LoadEffects(load.LoaderPlugin):
                     continue
 
                 try:
-                    node[knob_name].setValue(knob_value)
+                    # assume list means animation
+                    # except 4 values could be RGBA or vector
+                    if isinstance(knob_value, list) and len(knob_value) > 4:
+                        node[knob_name].setAnimated()
+                        for i, value in enumerate(knob_value):
+                            if isinstance(value, list):
+                                # list can have vector animation
+                                for ci, cv in enumerate(value):
+                                    node[knob_name].setValueAt(
+                                        cv,
+                                        (clip_in + i),
+                                        ci
+                                    )
+                            else:
+                                # list is single values
+                                node[knob_name].setValueAt(
+                                    value,
+                                    (clip_in + i)
+                                )
+                    else:
+                        node[knob_name].setValue(knob_value)
                 except NameError:
                     self.log.warning("Knob: {} cannot be set".format(
                         knob_name))