Skip to content

Use new spawner support for allow_substs#1079

Open
saikishor wants to merge 1 commit intomasterfrom
allow/parsing_param_files/spawner
Open

Use new spawner support for allow_substs#1079
saikishor wants to merge 1 commit intomasterfrom
allow/parsing_param_files/spawner

Conversation

@saikishor
Copy link
Copy Markdown
Member

No description provided.

@saikishor saikishor force-pushed the allow/parsing_param_files/spawner branch from a79a52b to 8bf86b7 Compare March 23, 2026 09:58
Copy link
Copy Markdown
Contributor

@fmauch fmauch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Loving it :-)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we remove this and add it directly to launch? As it is so short? And basically there are no myna parameters for CM - so probably adding from launch files might be better for simplicity.

Comment on lines 43 to 47
parameters=[
PathSubstitution(FindPackageShare("ros2_control_demo_example_1"))
/ "config"
/ "rrbot_controllers.yaml"
/ "controller_manager.yaml"
],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple as that:

parameters=[{"update_rate": 10}]

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

Labels

hold Wait for a release of upstream packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants