Add optional bounds type field to GetEntityBounds request#29
Add optional bounds type field to GetEntityBounds request#29ayushgnv wants to merge 2 commits intoros-simulation:mainfrom
Conversation
|
Is there some real use case for autoselection of the bounds type? I can't really imagine downstream code asking for "some" bounds. But I agree with the addition of the type to the request |
@peci1 If we agree as a standard to support a default type for all simulators, I'm happy with setting this option to that default type moving forward. |
Adds an optional type field to GetEntityBounds.srv so callers can specify their preferred bounding box representation (box, sphere, convex hull). If unset, it defaults to TYPE_UNSPECIFIED (255), preserving existing behavior where the simulator picks the type.
Also adds the TYPE_UNSPECIFIED constant to Bounds.msg as a clean sentinel value for use in requests, distinct from the existing response types.
Closes #26.