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

Jump to bottom

Enable IMU? #2

Closed
ruffsl opened this issue Jan 28, 2017 · 1 comment
Closed

Enable IMU? #2

ruffsl opened this issue Jan 28, 2017 · 1 comment
Assignees
@damonkohler
Labels
question

Comments

@ruffsl
Copy link

ruffsl commented Jan 28, 2017 •
edited
Loading

I see use of the IMU has been left disabled. Are there any current blockers, or it just a matter of testing it? freight.lua#L38-L39

I'd be happy help test it. Enabling it just quickly I didn't notice anything drastically different, but I do see the node is subscribed to the /imu topic, and should be being used: sensor_bridge.cc#L71
@damonkohler
Copy link
Contributor

It's just a matter of testing it out. I only have access to the simulated robot where IMU isn't supported.

If you'd like to file a PR to enable it, please ensure there's also a configuration for the simulated robot.

@ruffsl ruffsl mentioned this issue Feb 1, 2017
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@damonkohler damonkohler

Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@damonkohler @ruffsl