refactor(ar_tag_based_localizer): refactor pub/sub and so on

[SakodaShintaro]

Description

Refactored ar_tag_based_localizer.

• Changed launch nest
  • Before /localization/pose_estimator/ar_tag_based_localizer/ar_tag_based_localizer
  • After /localization/pose_estimator/ar_tag_based_localizer
  • Previously, landmark_parser was running as a node, so it was necessary to create a namespace and launch nodes within it, but that is no longer the case.
• Fixed launch.xml to remap correctly
• Removed image_transport
  • it has allowed to remove setup function also
• Fixed the code of the qos variable (the contents remain unchanged)
• Fixed cam_info_callback
• Fixed test

Tests performed

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

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: 16 lines in your changes are missing coverage. Please review.

Comparison is base (dd3050a) 15.31% compared to head (966e952) 15.31%.
Report is 7 commits behind head on main.

❗ Current head 966e952 differs from pull request most recent head 1a27e43. Consider uploading reports for the commit 1a27e43 to get more accurate results

Files	Patch %	Lines
...tag_based_localizer/src/ar_tag_based_localizer.cpp	20.00%	9 Missing and 7 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5854   +/-   ##
=======================================
  Coverage   15.31%   15.31%           
=======================================
  Files        1740     1740           
  Lines      119879   119859   -20     
  Branches    36450    36444    -6     
=======================================
- Hits        18355    18353    -2     
+ Misses      80992    80978   -14     
+ Partials    20532    20528    -4     

Flag	Coverage Δ		*Carryforward flag
differential	8.52% <20.00%> (?)
total	15.31% <ø> (+<0.01%)	⬆️	Carriedforward from dd3050a

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

[KYabuuchi]

There is a trivial point 🙏
Now this package does not depend on image_transport, so you can remove <depend>image_transport</depend> from package.xml.

No other problems. 🙆‍♂️

[SakodaShintaro]

@KYabuuchi
Thanks. It makes perfect sense.
Fixed in 1a27e43

[KYabuuchi]

LGTM 🙆‍♂️