AprilRobotics / apriltag_ros

A ROS wrapper of the AprilTag 3 visual fiducial detector
Other
358 stars 338 forks source link

Add names to tag detections #150

Open jack-slip opened 1 year ago

jack-slip commented 1 year ago

Can be useful for hashing tag bundles for consumers of a /tag_detection stream

jack-slip commented 1 year ago

Thank you for the PR. While I agree with the usefulness of having the names, we don't want to change messages of released packages (it would make rosbags etc. incompatible) - sorry

Understood. Would you be in favor of the default behavior being the current message structure for backwards compatibility, but have an optional param that can enable publishing of a new NamedAprilTagDetection(Array) type? Or something along those lines