search

ros-perception / vision_msgs

Algorithm-agnostic computer vision message types for ROS.
Apache License 2.0
149 stars 72 forks source link

Suggestion for new msg type: vision_msgs/Point #66

Closed ijnek closed 2 years ago

ijnek commented 2 years ago

Along with the changes in https://github.com/ros-perception/vision_msgs/pull/64, I want to suggest separating the position (x, y) from vision_msgs/Pose into a new msg vision_msgs/Position.

Often, there are cases where I want to store a point in an image (where heading doesn't make sense), and there is no appropriate message type for it. vision_msgs seems like the appropriate place for this sort of msg.

This is similar to how position is a field in geometry_msgs/Pose.msg

SteveMacenski commented 2 years ago

Seems reasonable to me if you update all the appropriate locations