Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Simulated IMU #17

Open
ruffsl opened this issue Nov 10, 2016 · 3 comments
Open

Simulated IMU #17

ruffsl opened this issue Nov 10, 2016 · 3 comments
Assignees

Comments

@ruffsl
Copy link

ruffsl commented Nov 10, 2016

I was testing out this package with gazebo and noticed that there was no imu messages being publish like for the real platform: http://docs.fetchrobotics.com/robot_hardware.html#imu

It'd be nice in general to add the IMUs in the gazebo models so I could test out projects like cartographer that use senors fusion with IMUs in simulation. I suppose I could do a PR, but I'm not sure of the IMUs' origin wrt the robot's baselink and arm link.

@ruffsl
Copy link
Author

ruffsl commented Feb 1, 2017

Related issue pending on this enhancement:
cartographer-project/cartographer_fetch#2 (comment)

@moriarty
Copy link
Contributor

moriarty commented Apr 5, 2019

Sorry @ruffsl this issue has been neglected a long time.

It's still very low priority but I've tagged @erelson and @ajzeller because they should be able to get the IMUs' origin wrt the robot's base link.

@erelson
Copy link
Collaborator

erelson commented Apr 8, 2019

For future reference, internal ticket: HP-2127

If anyone wants to make a PR with dummy values, demonstrating what components of the URDF the they think the IMUs should be linked to, that may make this happen sooner :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

6 participants