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

Add sim for ada_imu #14

Closed
amalnanavati opened this issue Sep 6, 2023 · 0 comments · Fixed by #16
Closed

Add sim for ada_imu #14

amalnanavati opened this issue Sep 6, 2023 · 0 comments · Fixed by #16
Assignees

Comments

@amalnanavati
Copy link
Contributor

#11 currently breaks ADA MoveIt in sim, because it results in two disconnected TF trees: world->root and root_tilt->{entire robot}. It is missing root->robot_tile->root_tilt, which is published by the IMU joint state publisher.

This Issue is for the following:

  1. Add a sim argument to ada_imu.launch.py and a corresponding parameter to imu_jointstate_publisher.py. This argument should be able to take on values of either mock or real.
  2. In imu_jointstate_publisher.py, if sim!="real", then don't load the serial port and always assume the imu_angle is 0 (here).
  3. In ada_moveit demo.launch.py, include ada_imu.launch.py with the sim argument set to the same value that is passed in to the moveit launch file.
@amalnanavati amalnanavati self-assigned this Sep 6, 2023
@amalnanavati amalnanavati mentioned this issue Sep 6, 2023
7 tasks
@amalnanavati amalnanavati linked a pull request Sep 6, 2023 that will close this issue
7 tasks
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 a pull request may close this issue.

1 participant