Skip to content
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

Mujoco tests #59

Closed
wants to merge 0 commits into from
Closed

Mujoco tests #59

wants to merge 0 commits into from

Conversation

studywolf
Copy link
Contributor

Added in parameter to allow loading of a state into Mujoco config before calculating the dynamics or kinematics functions.

Added testing and fixed a bunch of bugs

Can also now get the transforms etc for any of the objects not just geoms

@studywolf studywolf force-pushed the mujoco_tests branch 6 times, most recently from 5d9bcf0 to a76bcf9 Compare October 9, 2019 22:35
Copy link
Collaborator

@p3jawors p3jawors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some tests are failing and a few examples should probably be removed. More detailed notes left in review and last commit

abr_control/arms/tests/test_mujoco_config.py Outdated Show resolved Hide resolved
abr_control/arms/tests/test_mujoco_config.py Outdated Show resolved Hide resolved
abr_control/arms/tests/test_mujoco_config.py Outdated Show resolved Hide resolved
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.

2 participants