Unity-Technologies / URDF-Importer

URDF importer
Apache License 2.0
232 stars 71 forks source link

AIRO-1703 Inertia tensor rotation set to zero if NaN values produced during inertia matrix conversion. #182

Closed vidurvij-Unity closed 2 years ago

vidurvij-Unity commented 2 years ago

Proposed change(s)

Change has been made to UrdfInertial.cs. The importer will set the inertia tensor rotation to zero if the inertia tensor rotation produced from the inertial matrix has NaN values.

Useful links (GitHub issues, JIRA tickets, forum threads, etc.)

https://github.com/Unity-Technologies/Unity-Robotics-Hub/issues/343

Types of change(s)

Testing and Verification

Please describe the tests that you ran to verify your changes. Please also provide instructions, ROS packages, and Unity project files as appropriate so we can reproduce the test environment.

Test Configuration:

Checklist

Other comments

vidurvij-Unity commented 2 years ago

Sonaqqube and Unit tests passed. Not showing up here.