fix(ndt_scan_matcher): add a bool return indicating whether ndt has been updated in update_ndt

[SakodaShintaro]

Description

Context

In the current implementation, the process continues even if there is no change in update_ndt, then the pointer is switched, and the original ndt object is destroyed.
I'm not sure which thread calls the NDT destructor when it is destroyed, but it has a negative impact on the overall operation of NDT.

Changes

• Add a bool return indicating whether ndt has been updated in update_ndt.
• Use this return value to switch the behavior in update_map.

Tests performed

It has been confirmed that the logging_simulator runs with the same accuracy as before on AWSIM data with GT.

• single pcd
• multi pcd

I confirmed that it works properly with AWSIM as well.

Effects on system behavior

There are no effects on system behavior.

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

Attention: Patch coverage is 0% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 14.72%. Comparing base (4b784b4) to head (0134e4b).
Report is 3 commits behind head on main.

Files	Patch %	Lines
...ization/ndt_scan_matcher/src/map_update_module.cpp	0.00%	5 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #6502      +/-   ##
==========================================
- Coverage   14.72%   14.72%   -0.01%     
==========================================
  Files        1900     1900              
  Lines      130361   130363       +2     
  Branches    38381    38382       +1     
==========================================
  Hits        19198    19198              
- Misses      89647    89649       +2     
  Partials    21516    21516              

Flag	Coverage Δ		*Carryforward flag
differential	3.65% <0.00%> (?)
total	14.72% <ø> (+<0.01%)	⬆️	Carriedforward from 4b784b4

*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.