refactor(ndt_scan_matcher): modularize map_callback

[kminoda]

Description

Aiming at integrating dynamic map loading feature in ndt_scan_matcher smoothly, I would like to modularize ndt_scan_matcher into the following modules:

• module that execute NDT scan matching (ndt_scan_matching_module) (refactor(ndt_scan_matcher): modularize callback_sensor_points and some related functions #2224, would be opened later)
• module that load pointcloud map (map_module) (This PR)
• module that execute the pose initialization (initial_pose_module)

As a first step, I would like to modularize callback_map_points.

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.

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.

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

[codecov]

Codecov Report

Base: 11.13% // Head: 11.14% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (d7e4c7a) compared to base (ed6b5c8).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff            @@
##             main    #2249    +/-   ##
========================================
  Coverage   11.13%   11.14%            
========================================
  Files        1200     1201     +1     
  Lines       86211    86335   +124     
  Branches    20802    20787    -15     
========================================
+ Hits         9601     9620    +19     
- Misses      66478    66590   +112     
+ Partials    10132    10125     -7     

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	11.13% <ø> (+0.02%)	⬆️	Carriedforward from 179f692

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

Impacted Files	Coverage Δ
localization/ndt_scan_matcher/src/map_module.cpp	0.00% <0.00%> (ø)
...ion/ndt_scan_matcher/src/ndt_scan_matcher_core.cpp	0.00% <0.00%> (ø)
...rajectory_follower/pid_longitudinal_controller.hpp	0.00% <0.00%> (-28.58%)	⬇️
planning/rtc_interface/src/rtc_interface.cpp	0.00% <0.00%> (ø)
.../trajectory_follower_nodes/src/controller_node.cpp	42.02% <0.00%> (ø)
...ctory_follower/include/trajectory_follower/mpc.hpp	46.66% <0.00%> (ø)
...on_planner/src/mission_planner/mission_planner.cpp	0.00% <0.00%> (ø)
...pi_helpers/ad_api_adaptors/src/routing_adaptor.cpp	0.00% <0.00%> (ø)
...ocity_planner/src/scene_module/stop_line/scene.cpp	0.00% <0.00%> (ø)
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[KYabuuchi]

Your code seems to be fine.
I added just one comment, please check it out.

[KYabuuchi]

LGTM