design icon indicating copy to clipboard operation
design copied to clipboard

finish EBNF for topic and service names

Open wjwwood opened this issue 9 years ago • 3 comments
trafficstars

See: https://github.com/ros2/design/pull/90#issuecomment-247329344

wjwwood avatar Oct 29 '16 01:10 wjwwood

In design article it say :

must not have an underscore (_) followed by a forward slash (/), i.e. _/

https://github.com/ros2/design/blob/gh-pages/articles/115_topic_and_service_name_mapping.md#ros-2-topic-and-service-name-constraints

but in valid example :

foo/_/bar

https://github.com/ros2/design/blob/gh-pages/articles/115_topic_and_service_name_mapping.md#ros-2-name-examples

Can you clarify ?

Theosakamg avatar Nov 11 '16 18:11 Theosakamg

ind design acticle it say :

must not contain any number of repeated underscores (_)

https://github.com/ros2/design/blob/gh-pages/articles/115_topic_and_service_name_mapping.md#ros-2-topic-and-service-name-constraints

but in current implementation of Parameter service : node_->get_name() + "__get_parameters", https://github.com/ros2/rclcpp/blob/5894a9cd4eb0371e380dcbac73b74efae353fde2/rclcpp/src/rclcpp/parameter_service.cpp

It is excluded of Service name of parameter stack ?

Theosakamg avatar Nov 11 '16 18:11 Theosakamg

@Theosakamg looks like I missed that one (foo/_/bar) when updating the rules. Thanks for pointing it out! I'll open a pr to fix it.

As for the current implementation, it doesn't follow this design doc yet (the implementation is lagging behind the document). We'll replace that line with something like this:

node_->get_name() + "/_get_parameters",

Or something similar, so that it uses the / namespace separator and is "hidden".

If you have further comments, please open a new issue or comment on the original pr (https://github.com/ros2/design/pull/90). This issue is specifically about completing the EBNF (candidate implementation is in #96).

wjwwood avatar Nov 11 '16 22:11 wjwwood