-
Notifications
You must be signed in to change notification settings - Fork 419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Physics tests: Install and run Habitat-sim with Bullet physics #213
Conversation
Introduced dynamics test and corresponding habitat-sim with bullet install/test path
I'd like to get this test (especially the build part) into master before the release. I'll write a separate PR to add assertions and additional tests once we have force/torque caching and more interesting time-stepping schemes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding physics testing. In future we need to expose physics enabling options to Python layer that we can test different configurations with pytest
parameter sweeping.
18c9356
to
2967c4c
Compare
…ebookresearch#213) * Eval on val and add an option to not take the checkpoints config
…ookresearch#213) * Download object test dataset and install Bullet physics with circleCI * Introduced test_dynamics to model physics phenomena and assert expected simulation results
Motivation and Context
Test installation of Bullet physics and habitat-sim with Bullet physics in circleCI.
Test python exposed dynamics with various physics implementations (only Bullet currently).
Leaving TODOs for later PR additions. We have no caching of dynamic state currently so I'm not querying or asserting anything about that yet. This PR is primarily to test bullet install, build with bullet, and run the simulator with dynamics.
How Has This Been Tested
Local (MacOS) and devfair (Ubuntu) pytest.
Types of changes
Checklist