Closed
Conversation
9f516bc to
ba2a2f8
Compare
This was referenced May 30, 2018
42d6998 to
a3e73e0
Compare
Contributor
Author
nnmm
pushed a commit
to ApexAI/rclcpp
that referenced
this pull request
Jul 9, 2022
* Migrate to '--ros-args ... [--]'-based ROS args extraction Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Extend rcl arguments API to retrieve unparsed ROS args. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Adapt rcl arguments tests. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Complete unparsed ROS args functionality. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Address peer review comments. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Extend rcl arguments test coverage. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Adapt more rcl tests to use --ros-args. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Address peer review comments. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com> * Ignore duplicate --ros-args flags. Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Work in progress towards initializing parameters.
Current state:
When the node is constructed yaml files passed as CLI args are parsed, and the parameters within are set on the node.
RCL
rclfunction to get yaml file name from command line arguments (Support passing yaml parameter files via commandline rcl#253)Questions/Assumptions
rclcppneeds to call methods to parse yaml becausercldoing it would cause a circular dependency between it andrcl_yaml_parse_paramsAssumingHas method for getting all yaml paths specifiedrclwill have a method for getting a path to a yaml file from anrcl_arguments_tDoesYesrcl_yaml_param_parserdependency need to be exported?rcl_yaml_param_parsernamespace + nodename include a leading slash? currently it does notParameterInitializationMapfor? Why notstd::vector<ParameterVariant>?TODO
Bare minimum:
NodeParametersshould start parameter services in its constructor (Autostart parameter services #478)Nodeinstance toNodeParametersto give toParameterServiceNodeParametersneeds to get yaml file from CLI arguments (Initialize params via yaml from command line #488)rcl_arguments_tpassed to node options is not zero initialized, then get yaml files from itnode optionsuse_global_argumentsisTrue` then get yaml files from global arguments.Desired:
ParameterVariantintoParameterwhich has aParameterValue(Split ParameterVariant into Parameter and ParameterValue #481)rclcppmethod to convertrcl_params_ttostd::vector<Parameter>(orstd::map<string, ParameterValue>orstd::map<string, Parameter>? ) (Convert rcl_params_t to ParameterMap #485)std::vector<Parameter>intoNodeParametersconstructor. (Pass initial parameter values to node constructor #486)node::create_parameter(name, value)std::vector<Parameter> node::unknown_initial_parameters()void node::raise_if_unknown_initial_parameters()