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

Extending ros_bridge to playback MRPT datasets #83

Merged
merged 9 commits into from Aug 29, 2017

Conversation

raghavendersahdev
Copy link
Contributor

@raghavendersahdev raghavendersahdev commented Aug 25, 2017

This pull request is for Google Summer of Code 2017 with Mobile Robotics Programming Toolkit (work done by Raghavender Sahdev)

Extension of the ROS package mrpt_bridge

Extending the ROS package mrpt_bridge to include additional mrpt_ros conversions for the following types:

  • mrpt::obs::CObservationImage <--> sensor_msgs/Image
  • mrpt::obs::CObservationIMU <--> sensor_msgs/IMU
  • mrpt::obs::CObservationGPS <--> sensor_msgs/NavSatFix
  • mrpt::obs::CObservationRange <--> sensor_msgs/Range
  • mrpt::obs::CObservationStereoImages <--> 3xsensor_msgs/Image for left, right and disparity
  • added test_Bridge file to test the conversions

* mrpt::obs::CObservationStereoImages <--> 3xsensor_Image for left, right and disparity
* also added the file test_Bridge to test the above conversions and publishing on respective ROS topics
* Contributors: Raghavender Sahdev

Copy link
Member

Choose a reason for hiding this comment

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

Thanks for updating the CHANGELOG!

However, normally this file is updated automatically as part of using the automated "bloom" scripts from ROS.
But your content will be useful to assist in the cleanup of the automatically-generated changelog, when time comes for such a new release! ;-)

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