Maya: Update frame range of instances when clicking on "Reset frame range"

openpype/hosts/maya/api/lib.py

@@ -35,7 +35,7 @@
 )
 from openpype.pipeline.create import (
     legacy_create,
-    get_legacy_creator_by_name,
+    get_legacy_creator_by_name
 )
 from openpype.pipeline.context_tools import (
     get_current_asset_name,
@@ -2205,10 +2205,8 @@ def get_frame_range(include_animation_range=False):
         include_animation_range (bool, optional): Whether to include
             `animationStart` and `animationEnd` keys to define the outer
             range of the timeline. It is excluded by default.
-
     Returns:
         dict: Asset's expected frame range values.
-
     """
 
     # Set frame start/end
@@ -2232,6 +2230,7 @@ def get_frame_range(include_animation_range=False):
         "handleStart": handle_start,
         "handleEnd": handle_end
     }
+
     if include_animation_range:
         # The animation range values are only included to define whether
         # the Maya time slider should include the handles or not.
@@ -2261,7 +2260,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 +2269,10 @@ 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. Even when True this will only reset instances if
+            `update_publishable_frame_range` is also enabled
+            in project settings.
     """
     if fps:
         fps = convert_to_maya_fps(
@@ -2300,6 +2303,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 +3138,52 @@ def remove_render_layer_observer():
         pass
 
 
-def update_content_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'
     """
-    This will update scene content to match new asset on context change
-    """
-    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/settings/defaults/project_settings/maya.json

@@ -453,6 +453,9 @@
             "destination-path": []
         }
     },
+    "update_publishable_frame_range": {
+        "enabled": false
+    },
     "include_handles": {
         "include_handles_default": false,
         "per_task_type": []

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",