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.

Sign up for GitHub

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

Jump to bottom

Port tf2_sensor_msgs #132

Closed
bpwilcox opened this issue Oct 2, 2018 · 6 comments
Closed

Port tf2_sensor_msgs #132

bpwilcox opened this issue Oct 2, 2018 · 6 comments
Assignees
@SteveMacenski
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@bpwilcox
Copy link

bpwilcox commented Oct 2, 2018 •
edited
Loading

In the costmap_2d package (#129), the observation_buffer requires the tf2_sensor_msgs package which is not yet ported in the geometry2 metapackage.
@mkhansenbot mkhansenbot added the enhancement New feature or request label Oct 4, 2018
@mjeronimo mjeronimo added the help wanted Extra attention is needed label Oct 18, 2018
@IanTheEngineer
Copy link

Is this the port, or is there additional work required?
https://github.com/ros2/geometry2/tree/ros2/tf2_sensor_msgs

@bpwilcox
Copy link
Author

Is this the port, or is there additional work required?
https://github.com/ros2/geometry2/tree/ros2/tf2_sensor_msgs

It hasn't been ported yet and is currently being ignored (AMENT_IGNORE) by the compiler. It is a small package, however.

@SteveMacenski
Copy link
Member

#192 (comment)

@mkhansenbot mkhansenbot mentioned this issue Oct 23, 2018
Closed
@mkhansenbot
Copy link
Collaborator

See issue: ros2/geometry2#59

@mkhansenbot mkhansenbot mentioned this issue Oct 23, 2018
Closed
@SteveMacenski SteveMacenski mentioned this issue Oct 23, 2018
Closed
@SteveMacenski
Copy link
Member

Ported: ros2/geometry2#74

@SteveMacenski SteveMacenski mentioned this issue Oct 23, 2018
Merged
@SteveMacenski SteveMacenski self-assigned this Oct 23, 2018
@SteveMacenski
Copy link
Member

done.

Once geometry2 merges my PR (its been approved) I'll remove my dep fork of it and then the ros2_ws geometry2 will contain it

ghost pushed a commit to logivations/navigation2 that referenced this issue Mar 7, 2022
@jplapp
AMR-1011 set putdown field in putdown (ros-navigation#132)
b5c2721
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SteveMacenski SteveMacenski

Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@IanTheEngineer @mjeronimo @mkhansenbot @SteveMacenski @bpwilcox