refactor(external_cmd_selector): rework parameters

[batuhanbeytekin]

Description

Implement the ROS Node configuration layout described in https://github.com/orgs/autowarefoundation/discussions/3371

Tests performed

Package built and launch locally.

Effects on system behavior

More reliable and faster parameter configuration file creation.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.
• The PR has been properly tested.
• The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.
• The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

[github-actions]

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

• You've checked our contribution guidelines.
• Your PR follows our pull request guidelines.
• All required CI checks pass before marking the PR ready for review.

[github-actions]

Documentation URL: https://autowarefoundation.github.io/autoware.universe/pr-8198/
Modified URLs:

• https://autowarefoundation.github.io/autoware.universe/pr-8198/control/autoware_external_cmd_selector/

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 23.90%. Comparing base (a742e82) to head (59f844a).
Report is 31 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8198      +/-   ##
==========================================
- Coverage   23.90%   23.90%   -0.01%     
==========================================
  Files        1380     1380              
  Lines      101770   101747      -23     
  Branches    38731    38713      -18     
==========================================
- Hits        24332    24323       -9     
+ Misses      74992    74918      -74     
- Partials     2446     2506      +60     

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <ø> (?)
total	23.90% <ø> (-0.01%)	⬇️	Carriedforward from 5e400de

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mitsudome-r]

Generated documentation page is complaining about the path for json file.

The path for this function should be the relative path from the root directory so please remove the first '/'.

Suggested change

	{{json_to_markdown("/control/autoware_external_cmd_selector/schema/external_cmd_selector.schema.json")}}
	{{json_to_markdown("control/autoware_external_cmd_selector/schema/external_cmd_selector.schema.json")}}

[mitsudome-r]

@batuhanbeytekin It seems like you resolved the conversation, but it seems like the issue exists. Could you remove the first / in the path?

[batuhanbeytekin]

Hi @TakaHoribe, @rej55, @satoshi-ota, @shmpwk, @taikitanaka3, @takayuki5168, @tkimura4,

One approval is needed to merge this PR. Could you please review it?

[shmpwk]

LGTM