ipa320 / cob_perception_common

Apache License 2.0
13 stars 59 forks source link

[Question] Why is PoseStamped used instead of Pose in Detection.msg? #96

Closed cagbal closed 5 years ago

cagbal commented 5 years ago

Why do we use geometry_msgs/PoseStamped instead of just geormetry_msgs/Pose inside Detection.msg? Is there a special reason that this message has 2 headers(1 in detection, 1 in detection/posestamped)?

You can see the line below: https://github.com/ipa320/cob_perception_common/blob/e55f4a95559b613383dafc5306d00fb5a6f5a667/cob_perception_msgs/msg/Detection.msg#L7

fmessmer commented 5 years ago

I don't know where it comes from....but I highly suggest to leave it this way, as this message is vastly used in various downstream nodes and a message change would likely break a lot of downstream functionality...

if you need one with only one header, then please create a new one in addition to Detection.msg

cagbal commented 5 years ago

Ok then I will close this.