perf(trajectory_follower_nodes): change longitudinal control to use period parameter

[Sharrrrk]

Description

To solve issue #701

Related links

Tests performed

On vehicle test is performed to check the control stability.

1. Plan a path
2. Engage vehicle
3. Check the speed and acceleration control stability.

Notes for reviewers

tier4_control_launch package is also updated accordingly, and duplicated parameters removed the launch.py.

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.

[Sharrrrk]

@TakaHoribe would you please help me to review this PR?

[codecov]

Codecov Report

Merging #763 (e7183b3) into main (d1cb17e) will increase coverage by 0.15%.
The diff coverage is 42.85%.

@@           Coverage Diff            @@
##            main    #763      +/-   ##
========================================
+ Coverage   9.66%   9.81%   +0.15%     
========================================
  Files        908     907       -1     
  Lines      56560   56717     +157     
  Branches    6830    6786      -44     
========================================
+ Hits        5465    5569     +104     
- Misses     46593   46623      +30     
- Partials    4502    4525      +23     

Flag	Coverage Δ		*Carryforward flag
differential	34.69% <50.00%> (?)
total	9.63% <50.00%> (-0.03%)	⬇️	Carriedforward from b03d565

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

Impacted Files	Coverage Δ
...ion/include/fault_injection/diagnostic_storage.hpp	53.84% <ø> (ø)
...ollower_nodes/src/longitudinal_controller_node.cpp	49.35% <42.85%> (+7.96%)	⬆️
...lision_checker_node/obstacle_collision_checker.cpp	0.00% <0.00%> (-8.14%)	⬇️
...nning/behavior_velocity_planner/test/src/utils.hpp	85.71% <0.00%> (-2.10%)	⬇️
control/trajectory_follower/src/pid.cpp	91.48% <0.00%> (ø)
perception/shape_estimation/src/node.cpp	0.00% <0.00%> (ø)
control/trajectory_follower/test/test_mpc.cpp	53.75% <0.00%> (ø)
planning/behavior_velocity_planner/src/node.cpp	0.00% <0.00%> (ø)
...erception/shape_estimation/lib/corrector/utils.cpp	0.00% <0.00%> (ø)
...eption/shape_estimation/lib/model/bounding_box.cpp	0.00% <0.00%> (ø)
... and 15 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdbff23...e7183b3. Read the comment docs.

[TakaHoribe]

@Sharrrrk Sure. Thank you for your great contribution!

[maxime-clem]

Tested in the planning simulator and everything works fine.
Code changes look good and I only have a few minor comments.