Collect frame range for all rop nodes

openpype/hosts/houdini/plugins/publish/collect_frames.py

@@ -11,15 +11,13 @@ from openpype.hosts.houdini.api import lib
 class CollectFrames(pyblish.api.InstancePlugin):
     """Collect all frames which would be saved from the ROP nodes"""
 
-    order = pyblish.api.CollectorOrder
+    order = pyblish.api.CollectorOrder + 0.01
     label = "Collect Frames"
     families = ["vdbcache", "imagesequence", "ass", "redshiftproxy", "review"]
 
     def process(self, instance):
 
         ropnode = hou.node(instance.data["instance_node"])
-        frame_data = lib.get_frame_data(ropnode)
-        instance.data.update(frame_data)
 
         start_frame = instance.data.get("frameStart", None)
         end_frame = instance.data.get("frameEnd", None)

openpype/hosts/houdini/plugins/publish/collect_instances.py

@@ -70,16 +70,10 @@ class CollectInstances(pyblish.api.ContextPlugin):
             if "active" in data:
                 data["publish"] = data["active"]
 
-            data.update(self.get_frame_data(node))
-
             # Create nice name if the instance has a frame range.
             label = data.get("name", node.name())
             label += " (%s)" % data["asset"]  # include asset in name
 
-            if "frameStart" in data and "frameEnd" in data:
-                frames = "[{frameStart} - {frameEnd}]".format(**data)
-                label = "{} {}".format(label, frames)
-
             instance = context.create_instance(label)
 
             # Include `families` using `family` data

openpype/hosts/houdini/plugins/publish/collect_rop_frame_range.py

@@ -0,0 +1,41 @@
+# -*- coding: utf-8 -*-
+"""Collector plugin for frames data on ROP instances."""
+import hou  # noqa
+import pyblish.api
+from openpype.hosts.houdini.api import lib
+
+
+class CollectRopFrameRange(pyblish.api.InstancePlugin):
+    """Collect all frames which would be saved from the ROP nodes"""
+
+    order = pyblish.api.CollectorOrder
+    label = "Collect RopNode Frame Range"
+
+    def process(self, instance):
+
+        node_path = instance.data.get("instance_node")
+        if node_path is None:
+            # Instance without instance node like a workfile instance
+            return
+
+        ropnode = hou.node(node_path)
+        frame_data = lib.get_frame_data(ropnode)
+
+        if "frameStart" in frame_data and "frameEnd" in frame_data:
+
+            # Log artist friendly message about the collected frame range
+            message = (
+                "Frame range {0[frameStart]} - {0[frameEnd]}"
+            ).format(frame_data)
+            if frame_data.get("step", 1.0) != 1.0:
+                message += " with step {0[step]}".format(frame_data)
+            self.log.info(message)
+
+            instance.data.update(frame_data)
+
+            # Add frame range to label if the instance has a frame range.
+            label = instance.data.get("label", instance.data["name"])
+            instance.data["label"] = (
+                "{0} [{1[frameStart]} - {1[frameEnd]}]".format(label,
+                                                               frame_data)
+            )