feat(behavior_path_planner)!: remove macro

[satoshi-ota]

Description

We decided to transit new bpp manager. Therefore, the behavior tree manager is no longer needed. So, I remove macro USE_OLD_ARCHITECTURE in this PR.

Related link:

• https://github.com/orgs/autowarefoundation/discussions/3097
• https://github.com/orgs/autowarefoundation/discussions/3600

Tests performed

• PASS TIER IV INTERNAL SCENARIOS

Effects on system behavior

Nothing.

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

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (df8a43a) 14.36% compared to head (1b771b3) 14.36%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4106      +/-   ##
==========================================
- Coverage   14.36%   14.36%   -0.01%     
==========================================
  Files        1567     1567              
  Lines      107880   107873       -7     
  Branches    31252    31248       -4     
==========================================
- Hits        15499    15496       -3     
+ Misses      75528    75522       -6     
- Partials    16853    16855       +2     

Flag	Coverage Δ		*Carryforward flag
differential	13.64% <0.00%> (?)
total	14.36% <0.00%> (-0.01%)	⬇️	Carriedforward from 2246931

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

Impacted Files	Coverage Δ
...ception/tensorrt_yolox/src/tensorrt_yolox_node.cpp	0.00% <ø> (ø)
...lanner/scene_module/avoidance/avoidance_module.hpp	0.00% <ø> (ø)
...ule/dynamic_avoidance/dynamic_avoidance_module.hpp	0.00% <ø> (ø)
.../scene_module/goal_planner/goal_planner_module.hpp	20.00% <ø> (ø)
...th_planner/scene_module/lane_change/base_class.hpp	16.27% <ø> (ø)
...r_path_planner/scene_module/lane_change/normal.hpp	0.00% <ø> (ø)
...th_planner/scene_module/scene_module_interface.hpp	16.53% <ø> (ø)
...nner/scene_module/side_shift/side_shift_module.hpp	0.00% <ø> (ø)
...cene_module/start_planner/start_planner_module.hpp	0.00% <ø> (ø)
.../utils/goal_planner/default_fixed_goal_planner.hpp	100.00% <ø> (ø)
... and 10 more

... and 1 file with indirect coverage changes

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