Merge feature from #4793 and #4996 to set frame range on instances

openpype/hosts/maya/api/lib.py

@@ -2261,7 +2261,7 @@ def get_frame_range(include_animation_range=False):
     return frame_range
 
 
-def reset_frame_range(playback=True, render=True, fps=True):
+def reset_frame_range(playback=True, render=True, fps=True, instances=True):
     """Set frame range to current asset
 
     Args:
@@ -2270,6 +2270,8 @@ def reset_frame_range(playback=True, render=True, fps=True):
         render (bool, Optional): Whether to set the maya render frame range.
             Defaults to True.
         fps (bool, Optional): Whether to set scene FPS. Defaults to True.
+        instances (bool, Optional): Whether to update publishable instances.
+            Defaults to True.
     """
     if fps:
         fps = convert_to_maya_fps(
@@ -2300,6 +2302,12 @@ def reset_frame_range(playback=True, render=True, fps=True):
         cmds.setAttr("defaultRenderGlobals.startFrame", animation_start)
         cmds.setAttr("defaultRenderGlobals.endFrame", animation_end)
 
+    if instances:
+        project_name = get_current_project_name()
+        settings = get_project_settings(project_name)
+        if settings["maya"]["update_publishable_frame_range"]["enabled"]:
+            update_instances_frame_range()
+
 
 def reset_scene_resolution():
     """Apply the scene resolution  from the project definition
@@ -3129,31 +3137,52 @@ def remove_render_layer_observer():
         pass
 
 
-def update_content_on_context_change():
-    """
-    This will update scene content to match new asset on context change
+def update_instances_frame_range():
+    """Update 'frameStart', 'frameEnd', 'handleStart', 'handleEnd' and 'asset'
+    attributes of sets that got one, execpt if instance family is 'render'
     """
-    scene_sets = cmds.listSets(allSets=True)
+    collected_instances = cmds.ls(
+        "*.id",
+        long=True,
+        type="objectSet",
+        recursive=True,
+        objectsOnly=True
+    )
     asset_doc = get_current_project_asset()
-    new_asset = asset_doc["name"]
-    new_data = asset_doc["data"]
-    for s in scene_sets:
-        try:
-            if cmds.getAttr("{}.id".format(s)) == "pyblish.avalon.instance":
-                attr = cmds.listAttr(s)
-                print(s)
-                if "asset" in attr:
-                    print("  - setting asset to: [ {} ]".format(new_asset))
-                    cmds.setAttr("{}.asset".format(s),
-                                 new_asset, type="string")
-                if "frameStart" in attr:
-                    cmds.setAttr("{}.frameStart".format(s),
-                                 new_data["frameStart"])
-                if "frameEnd" in attr:
-                    cmds.setAttr("{}.frameEnd".format(s),
-                                 new_data["frameEnd"],)
-        except ValueError:
-            pass
+
+    asset_data = asset_doc["data"]
+    frames_attributes = {
+        'frameStart': asset_data["frameStart"],
+        'frameEnd': asset_data["frameEnd"],
+        'handleStart': asset_data["handleStart"],
+        'handleEnd': asset_data["handleEnd"],
+        'asset': asset_doc['name']
+    }
+
+    for instance in collected_instances:
+        id_attr = "{}.id".format(instance)
+        if cmds.getAttr(id_attr) != "pyblish.avalon.instance":
+            continue
+
+        if (
+            cmds.attributeQuery('family', node=instance, exists=True) and
+            cmds.getAttr("{}.family".format(instance)) == "render"
+        ):
+            continue
+
+        for key, value in frames_attributes.items():
+            if cmds.attributeQuery(key, node=instance, exists=True):
+                if key == 'asset':
+                    cmds.setAttr(
+                        "{}.{}".format(instance, key),
+                        value,
+                        type="string"
+                    )
+                else:
+                    cmds.setAttr(
+                        "{}.{}".format(instance, key),
+                        value
+                    )
 
 
 def show_message(title, msg):

openpype/hosts/maya/api/lib_rendersettings.py

@@ -158,7 +158,12 @@ def _set_arnold_settings(self, width, height):
         cmds.setAttr(
             "defaultArnoldDriver.mergeAOVs", multi_exr)
         self._additional_attribs_setter(additional_options)
-        reset_frame_range(playback=False, fps=False, render=True)
+        reset_frame_range(
+            playback=False,
+            fps=False,
+            render=True,
+            instances=False
+        )
 
     def _set_redshift_settings(self, width, height):
         """Sets settings for Redshift."""

openpype/hosts/maya/api/pipeline.py

@@ -653,7 +653,7 @@ def on_task_changed():
 
     with lib.suspended_refresh():
         lib.set_context_settings()
-        lib.update_content_on_context_change()
+        lib.update_instances_frame_range()
 
     msg = "  project: {}\n  asset: {}\n  task:{}".format(
         legacy_io.active_project(),

openpype/hosts/maya/api/workfile_template_builder.py

@@ -14,7 +14,7 @@
     WorkfileBuildPlaceholderDialog,
 )
 
-from .lib import read, imprint, get_main_window
+from .lib import read, imprint, reset_frame_range, get_main_window
 
 PLACEHOLDER_SET = "PLACEHOLDERS_SET"
 
@@ -251,6 +251,7 @@ def cleanup_placeholder(self, placeholder, failed):
         cmds.sets(node, addElement=PLACEHOLDER_SET)
         cmds.hide(node)
         cmds.setAttr(node + ".hiddenInOutliner", True)
+        reset_frame_range(playback=False, render=False, fps=False)
 
     def load_succeed(self, placeholder, container):
         self._parent_in_hierarchy(placeholder, container)

openpype/settings/defaults/project_settings/maya.json

@@ -457,6 +457,9 @@
         "include_handles_default": false,
         "per_task_type": []
     },
+    "update_publishable_frame_range": {
+        "enabled": true
+    },
     "scriptsmenu": {
         "name": "OpenPype Tools",
         "definition": [

openpype/settings/entities/schemas/projects_schema/schema_project_maya.json

@@ -197,6 +197,24 @@
                 }
             ]
         },
+        {
+            "type": "dict",
+            "collapsible": true,
+            "key": "update_publishable_frame_range",
+            "label": "Update publishable instances on Reset Frame Range",
+            "checkbox_key": "enabled",
+            "children": [
+                {
+                    "type": "label",
+                    "label": "If enabled, the frame range and the handles of all the publishable instances will be updated when using the 'Reset Frame Range' functionality"
+                },
+                {
+                    "type": "boolean",
+                    "key": "enabled",
+                    "label": "Enabled"
+                }
+            ]
+        },
         {
             "type": "dict",
             "key": "include_handles",