Skip to content

Conversation

@fernando-cortez
Copy link
Contributor

@fernando-cortez fernando-cortez commented Sep 10, 2024

Description

Fix to the jitter issue on connected bodies on a FixedJoint. Changing to ForceMode.Force properly propagates movement to the child Rigidbody & FixedJoint.
Forces have been modified on the player's locomotion ScriptableObject to accommodate the introduction of mass to the force equation.

To test: pick up crate object and sprint with character -> notice no jitter.

Issue Number(s)

MTT-9071

Contribution checklist

  • [ N/A ] Tests have been added for the project and/or any internal package
  • [ N/A ] Release notes have been added to the project changelog file
  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • JIRA ticket ID is in the PR title or at least one commit message
  • Include the ticket ID number within the body message of the PR to create a hyperlink

@fernando-cortez fernando-cortez requested a review from a team as a code owner September 10, 2024 19:17
@fernando-cortez fernando-cortez changed the title fix: Distributed Authority Sample - Jitter on FixedJoint connected body fix: Distributed Authority Sample - Jitter on FixedJoint connected body [MTT-9071] Sep 10, 2024
@Elfi0Kuhndorf
Copy link
Contributor

Running with a crate looks way better now, awesome!

One thing I did notice though is that the box goes crazy when walking into another object. Try running e.g. into a bush and you will notice what I mean.

@fernando-cortez
Copy link
Contributor Author

Running with a crate looks way better now, awesome!

One thing I did notice though is that the box goes crazy when walking into another object. Try running e.g. into a bush and you will notice what I mean.

That's coming soon in an upcoming PR!

@fernando-cortez fernando-cortez merged commit 56845f9 into develop Sep 11, 2024
@fernando-cortez fernando-cortez deleted the fix/distributed-authority-sample/jitter-fixedjoint-forces-fix branch September 11, 2024 14:07
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