ros-planning / navigation_msgs

Message packages required by the navigation stack
42 stars 53 forks source link

SetMapProjections and get MapProjectionInfo have odd semantics #16

Open tfoote opened 4 years ago

tfoote commented 4 years ago

In the pre Foxy API review it was flagged that these services appear to be named the opposite of what would be semantically expected.

https://github.com/ros2/common_interfaces/pull/86

https://github.com/ros2/common_interfaces/pull/86/files#diff-4fa5d39b533930879a61949f36d238cbR574

https://github.com/ros2/common_interfaces/pull/86/files#diff-4fa5d39b533930879a61949f36d238cbR590