Skip to content

Commit 9b6be08

Browse files
committed
AnticipatedNetworkTransform corrections
1 parent 2793ec8 commit 9b6be08

File tree

2 files changed

+37
-11
lines changed

2 files changed

+37
-11
lines changed

com.unity.netcode.gameobjects/Runtime/Components/AnticipatedNetworkTransform.cs

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -286,6 +286,9 @@ private void ProcessSmoothing()
286286
// TODO: This does not handle OnFixedUpdate
287287
// This requires a complete overhaul in this class to switch between using
288288
// NetworkRigidbody's position and rotation values.
289+
/// <summary>
290+
/// Processes updates for the network transform, including smoothing interpolation
291+
/// </summary>
289292
public override void OnUpdate()
290293
{
291294
ProcessSmoothing();
@@ -422,6 +425,9 @@ protected internal override void InternalOnNetworkPostSpawn()
422425
}
423426
}
424427

428+
/// <summary>
429+
/// Called when the NetworkObject is spawned to initialize network transform functionality
430+
/// </summary>
425431
public override void OnNetworkSpawn()
426432
{
427433
if (NetworkManager.DistributedAuthorityMode)
@@ -445,6 +451,9 @@ public override void OnNetworkSpawn()
445451
NetworkManager.AnticipationSystem.AllAnticipatedObjects.Add(m_AnticipatedObject);
446452
}
447453

454+
/// <summary>
455+
/// Called when the NetworkObject is despawned to cleanup network transform resources
456+
/// </summary>
448457
public override void OnNetworkDespawn()
449458
{
450459
if (m_AnticipatedObject != null)
@@ -459,6 +468,9 @@ public override void OnNetworkDespawn()
459468
base.OnNetworkDespawn();
460469
}
461470

471+
/// <summary>
472+
/// Called when the component is being destroyed to perform final cleanup
473+
/// </summary>
462474
public override void OnDestroy()
463475
{
464476
if (m_AnticipatedObject != null)
@@ -510,19 +522,30 @@ public void Smooth(TransformState from, TransformState to, float durationSeconds
510522
m_CurrentSmoothTime = 0;
511523
}
512524

525+
/// <summary>
526+
/// Called before the transform state is updated with new network data
527+
/// </summary>
513528
protected override void OnBeforeUpdateTransformState()
514529
{
515530
// this is called when new data comes from the server
516531
m_LastAuthorityUpdateCounter = NetworkManager.AnticipationSystem.LastAnticipationAck;
517532
m_OutstandingAuthorityChange = true;
518533
}
519534

535+
/// <summary>
536+
/// Called when new transform state data is received from the network
537+
/// </summary>
538+
/// <param name="oldState">The previous network transform state</param>
539+
/// <param name="newState">The new network transform state to apply</param>
520540
protected override void OnNetworkTransformStateUpdated(ref NetworkTransformState oldState, ref NetworkTransformState newState)
521541
{
522542
base.OnNetworkTransformStateUpdated(ref oldState, ref newState);
523543
ApplyAuthoritativeState();
524544
}
525545

546+
/// <summary>
547+
/// Called after the transform has been updated with new values
548+
/// </summary>
526549
protected override void OnTransformUpdated()
527550
{
528551
if (CanCommitToTransform || m_AnticipatedObject == null)

pvpExceptions.json

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,47 +7,50 @@
77
},
88
"PVP-151-1": {
99
"errors": [
10-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnUpdate(): undocumented",
11-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnNetworkSpawn(): undocumented",
12-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnNetworkDespawn(): undocumented",
13-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnDestroy(): undocumented",
14-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnBeforeUpdateTransformState(): undocumented",
15-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnNetworkTransformStateUpdated(ref NetworkTransformState, ref NetworkTransformState): undocumented",
16-
"Unity.Netcode.Components.AnticipatedNetworkTransform: void OnTransformUpdated(): undocumented",
17-
"Unity.Netcode.Components.AnticipatedNetworkTransform.TransformState: undocumented",
18-
"Unity.Netcode.Components.AnticipatedNetworkTransform.TransformState: Position: undocumented",
19-
"Unity.Netcode.Components.AnticipatedNetworkTransform.TransformState: Rotation: undocumented",
20-
"Unity.Netcode.Components.AnticipatedNetworkTransform.TransformState: Scale: undocumented",
2110
"Unity.Netcode.ConnectionEventData: EventType: undocumented",
11+
2212
"Unity.Netcode.RpcException: undocumented",
2313
"Unity.Netcode.RpcException: .ctor(string): undocumented",
14+
2415
"Unity.Netcode.NetworkManager: CurrentSessionOwner: undocumented",
2516
"Unity.Netcode.NetworkManager: void NetworkUpdate(NetworkUpdateStage): undocumented",
2617
"Unity.Netcode.NetworkManager.ReanticipateDelegate: undocumented",
18+
2719
"Unity.Netcode.NetworkObject: void SetSceneObjectStatus(bool): undocumented",
20+
2821
"Unity.Netcode.CustomMessagingManager.HandleNamedMessageDelegate: missing <param name=\"senderClientId\">",
2922
"Unity.Netcode.CustomMessagingManager.HandleNamedMessageDelegate: missing <param name=\"messagePayload\">",
23+
3024
"Unity.Netcode.GenerateSerializationForGenericParameterAttribute: .ctor(int): undocumented",
25+
3126
"Unity.Netcode.GenerateSerializationForTypeAttribute: .ctor(Type): undocumented",
27+
3228
"Unity.Netcode.NetworkList<T>: void Finalize(): undocumented",
29+
3330
"Unity.Netcode.NetworkVariable<T>: CheckExceedsDirtinessThreshold: undocumented",
3431
"Unity.Netcode.NetworkVariable<T>: bool ExceedsDirtinessThreshold(): undocumented",
3532
"Unity.Netcode.NetworkVariable<T>: void OnInitialize(): undocumented",
3633
"Unity.Netcode.NetworkVariable<T>: bool CheckDirtyState(bool): missing <returns>",
3734
"Unity.Netcode.NetworkVariable<T>: void Dispose(): undocumented",
3835
"Unity.Netcode.NetworkVariable<T>: void Finalize(): undocumented",
3936
"Unity.Netcode.NetworkVariable<T>.CheckExceedsDirtinessThresholdDelegate: undocumented",
37+
4038
"Unity.Netcode.NetworkVariableUpdateTraits: undocumented",
4139
"Unity.Netcode.NetworkVariableUpdateTraits: MinSecondsBetweenUpdates: undocumented",
4240
"Unity.Netcode.NetworkVariableUpdateTraits: MaxSecondsBetweenUpdates: undocumented",
41+
4342
"Unity.Netcode.NetworkVariableBase: NetworkBehaviour GetBehaviour(): undocumented",
4443
"Unity.Netcode.NetworkVariableBase: void MarkNetworkBehaviourDirty(): undocumented",
44+
4545
"Unity.Netcode.NetworkVariableSerialization<T>.EqualsDelegate: missing <param name=\"a\">",
4646
"Unity.Netcode.NetworkVariableSerialization<T>.EqualsDelegate: missing <param name=\"b\">",
4747
"Unity.Netcode.NetworkVariableSerialization<T>.EqualsDelegate: missing <returns>",
48+
4849
"Unity.Netcode.UserNetworkVariableSerialization<T>.WriteDeltaDelegate: missing <param name=\"previousValue\">",
4950
"Unity.Netcode.UserNetworkVariableSerialization<T>.DuplicateValueDelegate: missing <param name=\"duplicatedValue\">",
51+
5052
"Unity.Netcode.NetworkSpawnManager: void Finalize(): undocumented",
53+
5154
"Unity.Netcode.NetworkTransport.TransportEventDelegate: missing <param name=\"eventType\">",
5255
"Unity.Netcode.NetworkTransport.TransportEventDelegate: missing <param name=\"clientId\">",
5356
"Unity.Netcode.NetworkTransport.TransportEventDelegate: missing <param name=\"payload\">",

0 commit comments

Comments
 (0)