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

Initial messages for review. #1

Merged
merged 2 commits into from
Apr 18, 2014
Merged

Conversation

mikepurvis
Copy link
Member

Please review, @rgariepy, @ibaranov-cp

Additional MCU msgs/srvs (to be documented on the jackal_firmware ROS wiki page):

nmea_msgs/Sentence -> raw strings to and from globaltop module
sensor_msgs/IMU -> raw data from accelerometer and gyro
geometry_msgs/Vector3Stamped -> raw magnetometer data
std_srvs/Empty -> enable_motors, stop_motors services
std_msgs/Bool (?) -> wifi connected status

MCU parameters to be reviewed separately.


Header header

DriveFeedback[2] drivers
Copy link
Member Author

Choose a reason for hiding this comment

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

Provide an indication when the system has switched from user control to current-limit control? This is a very real thing which will have to be managed; it'd be great for users to have some visibility into it. It's not per-Drive feedback, though, since the current limit is shared between the two sides (it's the headroom after the PC and user supplies are accounted for).

@rgariepy
Copy link
Contributor

Placeholder for MCU firmware comments:
sensor_msgs/IMU isn't meant for truly raw data. Will need to determine where is best to scale to m/s^2, etc

@mikepurvis
Copy link
Member Author

re: sensor_msgs/IMU

The intent is that the MCU will publish a data message suitable for consumption by the filter, using standard REP-103 units.

So, scaling will occur on the MCU, controlled by rosparams.

@mikepurvis mikepurvis merged commit 545def8 into indigo-devel Apr 18, 2014
@mikepurvis mikepurvis deleted the first-cut-messages branch April 30, 2014 16:49
tonybaltovski pushed a commit that referenced this pull request Feb 5, 2020
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.

None yet

2 participants