NVIDIAGameWorks / PhysX-3.4

NVIDIA PhysX SDK 3.4
https://www.nvidia.com/
2.35k stars 274 forks source link

Revolute Joint Velocity #44

Closed GregSlazinski closed 6 years ago

GregSlazinski commented 6 years ago

https://github.com/NVIDIAGameWorks/PhysX-3.4/blob/d5334b8a11bcdec853a038cc67e818dec8f70b4f/PhysX_3.4/Source/PhysXExtensions/src/ExtRevoluteJoint.cpp#L85

I think this should be getRelativeAngularVelocity().magnitude() instead of getRelativeAngularVelocity().x ?

In the tests I've done in an older version of PhysX, the ".x" solution worked only if revolute joint axis was (1,0,0), but once it was changed to another direction, then I had to use ".magnitude()".

Please check,

Thanks

AlesBorovicka commented 6 years ago

Hi, thanks for the report we will fix this issue in next patch. Regards, Ales