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

ROS wrapper #1911

Merged
merged 15 commits into from
May 13, 2019
Merged

ROS wrapper #1911

merged 15 commits into from
May 13, 2019

Conversation

madratman
Copy link
Contributor

@madratman madratman commented Apr 25, 2019

This is a draft pull request, which means it is a WIP, and is not ready for a merge.

Things needed to be done:

  • Auto import list of cameras and lidars from settings.json. Add a parameter publish_to_ros and automatically setup publishers in the wrapper.
    Chose to add a sensor.yml instead, which is imported as a rosparam
  • Refactor into a nodelet
    nodelets weren't needed as rosmsg's pointers were being used already and things were zero copy.
    Quite weirdly, somehow simgetimage() calls to airsim's C++ client are slower than Python client. Needs more looking into.
  • Add an IMU API and expose it. Related: Correct way to get IMU data - Code snippet and Explanation [not bug] #1909
  • Lidar publisher
  • Enable multiple drones
  • Client package / nodes in C++ and Python making it easiers for users to get started
  • State estimation, mapping, path, and trajectory planning demos.

@madratman madratman force-pushed the PR/ros_release branch 5 times, most recently from 5e2936b to 76cd5d9 Compare April 27, 2019 00:43
@madratman madratman force-pushed the PR/ros_release branch 2 times, most recently from 7d5525e to 8d8b51c Compare April 30, 2019 04:23
@madratman madratman force-pushed the PR/ros_release branch 9 times, most recently from 9fc5f42 to e119962 Compare May 7, 2019 19:52
@madratman madratman marked this pull request as ready for review May 7, 2019 19:52
@madratman madratman force-pushed the PR/ros_release branch 11 times, most recently from 92ec2ab to 2f33f9a Compare May 11, 2019 05:50
@madratman madratman force-pushed the PR/ros_release branch 2 times, most recently from a7b625e to 14ab2d9 Compare May 13, 2019 23:05
@madratman
Copy link
Contributor Author

Merging current version now. Adding more demos and tutorials in next PR

@madratman madratman merged commit cc8d6aa into microsoft:master May 13, 2019
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.

1 participant