Skip to content

Conversation

NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Aug 21, 2025

Purpose of this PR

This PR resolves an issue where a client, under above average latency and packet loss conditions, could receive multiple NetworkTransform state updates in one frame and when processing the state updates only the last state update would be applied to the transform.

fix: #3571

Jira ticket

MTTB-1530

Changelog

  • Fixed: Issue where a client, under above average latency and packet loss conditions, could receive multiple NetworkTransform state updates in one frame and when processing the state updates only the last state update would be applied to the transform if interpolation was disabled.

Documentation

  • No documentation changes or additions were necessary.

Testing & QA

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

  • Covered by existing automated tests
  • Covered by new automated tests

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?

If any boxes above are checked, please add QA as a PR reviewer.

Backport

This is an NGO v2.x only issue.

This resolves issue #3571.
This test validates the fix for #3571.
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner August 21, 2025 21:27
@NoelStephensUnity NoelStephensUnity marked this pull request as draft August 21, 2025 21:27
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review August 26, 2025 15:28
@NoelStephensUnity NoelStephensUnity enabled auto-merge (squash) August 26, 2025 15:28
@NoelStephensUnity NoelStephensUnity merged commit 15263c7 into develop-2.0.0 Aug 27, 2025
25 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/networktransform-fails-to-sync-multiple-state-updates branch August 27, 2025 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants