Skip to content

fix: NetworkRigidbodyBase 2D rigid body issues#4012

Open
NoelStephensUnity wants to merge 6 commits into
develop-2.0.0from
fix/rigidbody-2D-4009-3924
Open

fix: NetworkRigidbodyBase 2D rigid body issues#4012
NoelStephensUnity wants to merge 6 commits into
develop-2.0.0from
fix/rigidbody-2D-4009-3924

Conversation

@NoelStephensUnity

@NoelStephensUnity NoelStephensUnity commented Jun 8, 2026

Copy link
Copy Markdown
Member

WIP -- Writing test for 4009

Purpose of this PR

  • Fixing issue where Rigidbody2D was not applying rotation correctly. (4009)
  • Fixing issue where NetworkRigidbodyBase was always checking the 3D rigid body's interpolation mode when determining if it is kinematic and needs to put the rigid body to sleep and then switch to interpolation. (3924)

fix: #4009, #3924

Jira ticket

(wip)

Changelog

  • Fixed: Issue where NetworkRigidbodyBase was not applying rotation correctly when using Rigidbody2D.
  • Fixed: Issue where NetworkRigidbodyBase was always checking the 3D rigid body's interpolation mode when determining if it is kinematic and needs to put the rigid body to sleep and then switch to interpolation.

Documentation

  • No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

  • Manual testing done

Automated tests:

Does the change require QA team to:

  • Review automated tests?
  • Execute manual tests?
  • Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Backports

Specific to v2.x.x+. No backport required.

Fixing issue where Rigidbody2D was not applying rotation correctly. (4009)
Fixing issue where `NetworkRigidbodyBase` was always checking the 3D rigid body's interpolation mode when determining if it is kinematic and needs to put the rigid body to sleep and then switch to interpolation. (3924)
Removing static using directive (not used).
Missed the return...
Did a bit of an overhaul on the original NetworkRigidbodyTest. It should now run with distributed authority tests and includes a 2D rigidbody in the tests as well.
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review June 8, 2026 23:53
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner June 8, 2026 23:53
Removing trailing space on comment
Updating the NetworkRigidbodyTest to do a 2nd spawn pass that swaps the session owner and non-session owner clients to validate it works when a non-session authority client is the spawn authority.
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.

NetworkRigidBody2D with "Use Rigid Body for motion" breaks rotation sync

1 participant