Merge branch 'develop-2.0.0' into chore/optimize-destroy-object-message

# Conflicts:
#	com.unity.netcode.gameobjects/CHANGELOG.md

Author: noellie-velez

com.unity.netcode.gameobjects/CHANGELOG.md

@@ -13,11 +13,13 @@ Additional documentation and release notes are available at [Multiplayer Documen
 - It is now possible to control which port clients will bind to using the `UnityTransport.ConnectionData.ClientBindPort` field. If not set, clients will bind to an ephemeral port (same as before this change). (#3764)
 - Added a flag to override command-line arguments (port and ip) in `SetConnectionData`. (#3760)
 - Added a command-line singleton to parse environment command-line arguments. (#3760)
-
+- Added `NetworkAnimator.AuthorityMode` which allows you to select whether the `NetworkAnimator` will use a server or owner authority model for state updates (like `NetworkTransform`). (#3586)
+- Added the ability to select which `Animator` parameters the authority `NetworkAnimator` instance should synchronize. This can be done via the inspector view interface or during runtime via `NetworkAnimator.EnableParameterSynchronization`. (#3586)
 
 ### Changed
 
 - Improve performance of `DestroyObjectMessage`. (#3801)
+- Changed NetworkAnimator to use the `RpcAttribute` along with the appropriate `SendTo` parameter. (#3586)
 - Improve performance of `NetworkTransformState`. (#3770)
 
 
@@ -33,6 +35,7 @@ Additional documentation and release notes are available at [Multiplayer Documen
 - Fixed issue where invoking an RPC, on another `NetworkBehaviour` associated with the same `NetworkObject` that is ordered before the `NetworkBehaviour` invoking the RPC, during `OnNetworkSpawn` could throw an exception if scene management is disabled. (#3782)
 - Fixed issue where the `Axis to Synchronize` toggles didn't work with multi object editing in `NetworkTransform`. (#3781)
 - Fixed issue where using the dedicated server package would override all attempts to change the port by code. (#3760)
+- Fixed issue with authority animator instance sending itself RPCs. (#3586)
 
 ### Security
 

com.unity.netcode.gameobjects/Documentation~/components/core/networkobject.md

@@ -112,9 +112,9 @@ Similar to [`NetworkObject.ActiveSceneSynchronization`](#active-scene-synchroniz
 
 `NetworkObject.ActiveSceneSynchronization` can be used with `NetworkObject.SceneMigrationSynchronization` as long as you take into consideration that if you migrate a NetworkObject into a non-active scene via `SceneManager.MoveGameObjectToScene` and later change the active scene, then the NetworkObject instance will be automatically migrated to the newly set active scene.
 
-:::warning
-Scene migration synchronization is enabled by default. For NetworkObjects that don't require it, such as those that generate static environmental objects like trees, it's recommended to disable scene migration synchronization to avoid additional processing overheads.
-:::
+> [!NOTE]
+> The `NetworkObject.SceneMigrationSynchronization` field is enabled by default. If your project can have many (600-1000+) spawned objects at any given time, this setting can cause performance issues. Scene migration synchronization only provides the ability to automatically synchronize the scene a NetworkObject is migrated to while it is spawned. If your NetworkObject doesn't change scenes and you have no need for automatic scene migration synchronization (_only applies when the integration scene management is enabled_), then it's recommended to disable this property to avoid additional processing overheads.
+
 
 ## Additional resources
 

com.unity.netcode.gameobjects/Documentation~/components/helper/networkanimator.md

@@ -0,0 +1,96 @@
+
+#if COM_UNITY_MODULES_ANIMATION
+using Unity.Netcode.Components;
+using UnityEditor;
+using UnityEngine;
+
+namespace Unity.Netcode.Editor
+{
+    [CustomPropertyDrawer(typeof(NetworkAnimator.AnimatorParametersListContainer))]
+    internal class NetworkAnimatorParameterEntryDrawer : PropertyDrawer
+    {
+        // Draw the property inside the given rect
+        public override void OnGUI(Rect position, SerializedProperty property, GUIContent label)
+        {
+            EditorGUI.BeginProperty(position, label, property);
+
+            // Draw the foldout for the list
+            SerializedProperty items = property.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntries.ParameterEntries));
+            position.height = EditorGUIUtility.singleLineHeight;
+            property.isExpanded = EditorGUI.Foldout(position, property.isExpanded, label);
+
+            if (property.isExpanded)
+            {
+                // Set the indention level down
+                EditorGUI.indentLevel++;
+                for (int i = 0; i < items.arraySize; i++)
+                {
+                    position.y += EditorGUIUtility.singleLineHeight + 2;
+                    SerializedProperty element = items.GetArrayElementAtIndex(i);
+                    var nameField = element.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntry.name));
+                    // Draw the foldout for the item
+                    element.isExpanded = EditorGUI.Foldout(position, element.isExpanded, nameField.stringValue);
+                    if (!element.isExpanded)
+                    {
+                        continue;
+                    }
+                    // Draw the contents of the item
+                    position.y += EditorGUIUtility.singleLineHeight + 2;
+                    // Set the indention level down
+                    EditorGUI.indentLevel++;
+                    // Calculate rects
+                    var nameHashRect = new Rect(position.x, position.y, position.width, EditorGUIUtility.singleLineHeight);
+                    position.y += EditorGUIUtility.singleLineHeight + 2;
+                    var paramRect = new Rect(position.x, position.y, position.width, EditorGUIUtility.singleLineHeight);
+                    position.y += EditorGUIUtility.singleLineHeight + 2;
+                    var syncRect = new Rect(position.x, position.y, position.width, EditorGUIUtility.singleLineHeight);
+
+                    // Get the three properties we want to visualize in the inspector view
+                    var synchronizeField = element.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntry.Synchronize));
+                    var nameHashField = element.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntry.NameHash));
+                    var parameterTypeField = element.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntry.ParameterType));
+
+                    // Draw the read only fields
+                    GUI.enabled = false;
+                    EditorGUI.PropertyField(nameHashRect, nameHashField);
+                    EditorGUI.PropertyField(paramRect, parameterTypeField);
+                    GUI.enabled = true;
+                    // Draw the read/write fields
+                    EditorGUI.PropertyField(syncRect, synchronizeField);
+                    // Set the indention level up
+                    EditorGUI.indentLevel--;
+                }
+                // Set the indention level up
+                EditorGUI.indentLevel--;
+            }
+            EditorGUI.EndProperty();
+        }
+
+        public override float GetPropertyHeight(SerializedProperty property, GUIContent label)
+        {
+            var totalHeight = EditorGUIUtility.singleLineHeight;
+            if (!property.isExpanded)
+            {
+                return totalHeight;
+            }
+            var singleLineWithSpace = EditorGUIUtility.singleLineHeight + 2;
+            SerializedProperty items = property.FindPropertyRelative(nameof(NetworkAnimator.AnimatorParameterEntries.ParameterEntries));
+
+            totalHeight += singleLineWithSpace;
+            for (int i = 0; i < items.arraySize; i++)
+            {
+                SerializedProperty element = items.GetArrayElementAtIndex(i);
+                if (element.isExpanded)
+                {
+                    totalHeight += (singleLineWithSpace * 4);
+                }
+                else
+                {
+                    totalHeight += EditorGUIUtility.singleLineHeight;
+                }
+            }
+            return totalHeight;
+        }
+    }
+}
+#endif