reeseschultz / ReeseUnityDemos

Unity packages and demos—emphasizing ECS, jobs and the Burst compiler—by Reese and others.
https://reese.codes
MIT License
516 stars 45 forks source link

Update PhysicsVelocity when available instead of Translation #43

Closed reeseschultz closed 2 years ago

reeseschultz commented 3 years ago

Is the improvement related to a problem? Please describe.

Yeah, physical agents ought to move physically (as in Unity.Physics) to avoid weirdness.

Describe the solution you'd prefer:

If there is a PhysicsVelocity available, modify it instead of the Translation in the NavInterpolationSystem.

Describe alternatives or workarounds you've considered:

The alternative/workaround would be forcing users to change my code to suit their needs, which doesn't make a lot of sense, especially when they update the nav package.

reeseschultz commented 2 years ago

Closing per #95.