Skip to content

remove BoundingBox2D message #4

@minhnh

Description

@minhnh

As there exists the message sensor_msgs/RegionOfInterest, we should remove our mas_perception_msgs/BoundingBox2D in favor of the built-in option.

This will affect mas_perception_msgs/Plane, and potentially mdr_place_action, since it uses the limits field in mas_perception_msgs/Plane of type mas_perception_msgs/BoundingBox2D.

Since the values from this limits can be calculated from the convex_hull field which contains 3D points, I think it's ok to simply remove limits as the plane may not always be horizontal.

Metadata

Metadata

Labels

Status: StaleThe PR or issue has been abandoned.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions