charleslemaux/Mirror
1
0
Fork 0
mirror of https://github.com/MirrorNetworking/Mirror.git synced 2024-11-17 18:40:33 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix: #3588 NetworkTransform OnTeleport doesn't call Reset() anymore, causing NetworkTransformReliable's delta compression to get out of sync before

Browse Source
This commit is contained in:
mischa 2023-11-01 15:15:52 +01:00
parent a432b7f43d
commit 631b8e0eac
2 changed files with 27 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
Assets/Mirror/Components/NetworkTransform/NetworkTransformBase.cs
View File

@ -313,13 +313,22 @@ void RpcReset()
// common Teleport code for client->server and server->client // common Teleport code for client->server and server->client
protected virtual void OnTeleport(Vector3 destination) protected virtual void OnTeleport(Vector3 destination)
{ {
// reset any in-progress interpolation & buffers
Reset();
// set the new position. // set the new position.
// interpolation will automatically continue. // interpolation will automatically continue.
target.position = destination; target.position = destination;
// reset interpolation to immediately jump to the new position.
// do not call Reset() here, this would cause delta compression to
// get out of sync for NetworkTransformReliable because NTReliable's
// 'override Reset()' resets lastDe/SerializedPosition:
// https://github.com/MirrorNetworking/Mirror/issues/3588
// because client's next OnSerialize() will delta compress,
// but server's last delta will have been reset, causing offsets.
//
// instead, simply clear snapshots.
serverSnapshots.Clear();
clientSnapshots.Clear();
// TODO // TODO
// what if we still receive a snapshot from before the interpolation? // what if we still receive a snapshot from before the interpolation?
// it could easily happen over unreliable. // it could easily happen over unreliable.
@ -329,14 +338,23 @@ protected virtual void OnTeleport(Vector3 destination)
// common Teleport code for client->server and server->client // common Teleport code for client->server and server->client
protected virtual void OnTeleport(Vector3 destination, Quaternion rotation) protected virtual void OnTeleport(Vector3 destination, Quaternion rotation)
{ {
// reset any in-progress interpolation & buffers
Reset();
// set the new position. // set the new position.
// interpolation will automatically continue. // interpolation will automatically continue.
target.position = destination; target.position = destination;
target.rotation = rotation; target.rotation = rotation;
// reset interpolation to immediately jump to the new position.
// do not call Reset() here, this would cause delta compression to
// get out of sync for NetworkTransformReliable because NTReliable's
// 'override Reset()' resets lastDe/SerializedPosition:
// https://github.com/MirrorNetworking/Mirror/issues/3588
// because client's next OnSerialize() will delta compress,
// but server's last delta will have been reset, causing offsets.
//
// instead, simply clear snapshots.
serverSnapshots.Clear();
clientSnapshots.Clear();
// TODO // TODO
// what if we still receive a snapshot from before the interpolation? // what if we still receive a snapshot from before the interpolation?
// it could easily happen over unreliable. // it could easily happen over unreliable.

3
Assets/Mirror/Components/NetworkTransform/NetworkTransformReliable.cs
View File

@ -405,6 +405,9 @@ static void RewriteHistory(
); );
} }
// reset state for next session.
// do not ever call this during a session (i.e. after teleport).
// calling this will break delta compression.
public override void Reset() public override void Reset()
{ {
base.Reset(); base.Reset();