New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SetMapProjections and get MapProjectionInfo have odd semantics #16

Open

tfoote opened this issue Apr 1, 2020 · 0 comments

Member

tfoote commented Apr 1, 2020

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

ros2/common_interfaces#86

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

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

tfoote mentioned this issue

Aggregate Foxy Message API Review comments. ros2/common_interfaces#86

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment