Skip to content
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.

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

Use wildcard syntax in generated parameter YAML files #35

Merged
merged 1 commit into from
Jun 24, 2019

Conversation

cottsay
Copy link
Member

@cottsay cottsay commented Jun 14, 2019

This will eliminate the requirement that a node name override be specified when passing a dictionary of parameters from launch.

Requires ros2/rclcpp#762
Requires ros2/rclpy#370

Resolves #4

@cottsay cottsay added enhancement New feature or request in review labels Jun 14, 2019
@cottsay cottsay self-assigned this Jun 14, 2019
@cottsay
Copy link
Member Author

cottsay commented Jun 14, 2019

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

This will eliminate the requirement that a node name override be
specified when passing a dictionary of parameters from launch.

Signed-off-by: Scott K Logan <logans@cottsay.net>
@cottsay cottsay merged commit 37929ee into master Jun 24, 2019
@delete-merged-branch delete-merged-branch bot deleted the wildcard_param_yaml branch June 24, 2019 22:55
@cottsay cottsay restored the wildcard_param_yaml branch July 29, 2019 23:29
cottsay added a commit that referenced this pull request Jul 31, 2019
This will eliminate the requirement that a node name override be
specified when passing a dictionary of parameters from launch.

This is a backport of #35 for Dashing.

Signed-off-by: Scott K Logan <logans@cottsay.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request in review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't require node name to be specified if parameter dictionary passed to Node action
2 participants