From 44ed77a9f655c03ed585596b08eafc2e6dd2a402 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 25 Mar 2024 14:20:14 +0100
Subject: [PATCH 1/3] Houdini: Implement USD loader to SOPs via `usdimport`

---
 .../houdini/plugins/load/load_usd_sop.py      | 91 +++++++++++++++++++
 1 file changed, 91 insertions(+)
 create mode 100644 client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py

diff --git a/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
new file mode 100644
index 0000000000..b32210e399
--- /dev/null
+++ b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
@@ -0,0 +1,91 @@
+import os
+
+from ayon_core.pipeline import load
+from ayon_core.hosts.houdini.api import pipeline
+
+
+class SopUsdImportLoader(load.LoaderPlugin):
+    """Load USD to SOPs via `usdimport`"""
+
+    label = "Load USD to SOPs"
+    product_types = {"*"}
+    representations = ["usd"]
+    order = -6
+    icon = "code-fork"
+    color = "orange"
+
+    def load(self, context, name=None, namespace=None, data=None):
+        import hou
+
+        # Format file name, Houdini only wants forward slashes
+        file_path = self.filepath_from_context(context)
+        file_path = os.path.normpath(file_path)
+        file_path = file_path.replace("\\", "/")
+
+        # Get the root node
+        obj = hou.node("/obj")
+
+        # Define node name
+        namespace = namespace if namespace else context["folder"]["name"]
+        node_name = "{}_{}".format(namespace, name) if namespace else name
+
+        # Create a new geo node
+        container = obj.createNode("geo", node_name=node_name)
+
+        # Remove the file node, it only loads static meshes
+        # Houdini 17 has removed the file node from the geo node
+        file_node = container.node("file1")
+        if file_node:
+            file_node.destroy()
+
+        # Create a usdimport node
+        usdimport = container.createNode("usdimport", node_name=node_name)
+        usdimport.setParms({"filepath1": file_path})
+
+        # Ensure display flag is on the first input node and not on the OUT
+        # node to optimize "debug" displaying in the viewport.
+        usdimport.setDisplayFlag(True)
+
+        # Set new position for unpack node else it gets cluttered
+        nodes = [container, usdimport]
+        for nr, node in enumerate(nodes):
+            node.setPosition([0, (0 - nr)])
+
+        self[:] = nodes
+
+        return pipeline.containerise(
+            node_name,
+            namespace,
+            nodes,
+            context,
+            self.__class__.__name__,
+            suffix="",
+        )
+
+    def update(self, container, context):
+
+        node = container["node"]
+        try:
+            usdimport_node = next(
+                n for n in node.children() if n.type().name() == "usdimport"
+            )
+        except StopIteration:
+            self.log.error("Could not find node of type `usdimport`")
+            return
+
+        # Update the file path
+        file_path = self.filepath_from_context(context)
+        file_path = file_path.replace("\\", "/")
+
+        usdimport_node.setParms({"filepath1": file_path})
+
+        # Update attribute
+        node.setParms({"representation": context["representation"]["id"]})
+
+    def remove(self, container):
+
+        node = container["node"]
+        node.destroy()
+
+    def switch(self, container, representation):
+        self.update(container, representation)

From 24d87bf92a8be56fe837013860c2bb944f7520f6 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 25 Mar 2024 14:22:27 +0100
Subject: [PATCH 2/3] Remove redundant logic

---
 .../hosts/houdini/plugins/load/load_usd_sop.py       | 12 ------------
 1 file changed, 12 deletions(-)

diff --git a/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
index b32210e399..3607987e15 100644
--- a/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
+++ b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
@@ -32,24 +32,12 @@ class SopUsdImportLoader(load.LoaderPlugin):
         # Create a new geo node
         container = obj.createNode("geo", node_name=node_name)
 
-        # Remove the file node, it only loads static meshes
-        # Houdini 17 has removed the file node from the geo node
-        file_node = container.node("file1")
-        if file_node:
-            file_node.destroy()
-
         # Create a usdimport node
         usdimport = container.createNode("usdimport", node_name=node_name)
         usdimport.setParms({"filepath1": file_path})
 
-        # Ensure display flag is on the first input node and not on the OUT
-        # node to optimize "debug" displaying in the viewport.
-        usdimport.setDisplayFlag(True)
-
         # Set new position for unpack node else it gets cluttered
         nodes = [container, usdimport]
-        for nr, node in enumerate(nodes):
-            node.setPosition([0, (0 - nr)])
 
         self[:] = nodes
 

From 135dd15870feb230231ce888949f5a4c56ce8749 Mon Sep 17 00:00:00 2001
From: Roy Nieterau <roy_nieterau@hotmail.com>
Date: Mon, 25 Mar 2024 14:26:18 +0100
Subject: [PATCH 3/3] Remove redundant assignment to `self[:]`

---
 client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
index 3607987e15..5b7e022e73 100644
--- a/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
+++ b/client/ayon_core/hosts/houdini/plugins/load/load_usd_sop.py
@@ -39,8 +39,6 @@ class SopUsdImportLoader(load.LoaderPlugin):
         # Set new position for unpack node else it gets cluttered
         nodes = [container, usdimport]
 
-        self[:] = nodes
-
         return pipeline.containerise(
             node_name,
             namespace,